Skip to content

Conversation

MohanrajRKM
Copy link
Collaborator

Bug description

On the npmci site, the Smart Stock Picks section has a scrollbar even at 100% browser size with the recommended screen resolution of 125%.

Root cause

NA

Reason for not identifying earlier

Find how it was missed in our earlier testing and development by analyzing the below checklist. This will help prevent similar mistakes in the future.

  • Guidelines/documents are not followed

  • Common guidelines / Core team guideline

  • Specification document

  • Requirement document

  • Guidelines/documents are not given

  • Common guidelines / Core team guideline

  • Specification document

  • Requirement document

Reason:

Guidelines/documents are not given - Requirement document

Action taken:

NA

Related areas:

theme

Is it a breaking issue?

NA

Solution description

On the npmci site, the Smart Stock Picks section has a scrollbar even at 100% browser size with the recommended screen resolution of 125%, and the solution is to decrease the grid height to prevent it.

Output screenshots

Before
image

After
image

Areas affected and ensured

Ensured screen resolution at 150% and 125%, tested with browser sizes 100%, 110%, and 125%.

Additional checklist

This may vary for different teams or products. Check with your scrum masters.

  • Did you run the automation against your fix? - NA

  • Is there any API name change? - NA

  • Is there any existing behavior change of other features due to this code change? - NA

  • Does your new code introduce new warnings or binding errors? - NA

  • Does your code pass all FxCop and StyleCop rules? - NA

  • Did you record this case in the unit test or UI test? - NA

  • This issue applicable for blazor? - No

Copy link
Collaborator

@jmfarook7 jmfarook7 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

fine

@jmfarook7 jmfarook7 merged commit 74939eb into master Jan 28, 2025
1 check passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

3 participants