Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[#12043] Navigation menu got collapsed #12057

Merged

Conversation

NaveenJoyGit
Copy link
Contributor

@NaveenJoyGit NaveenJoyGit commented Feb 5, 2023

Fixes #12043

Outline of Solution

Added flex-nowrap bootstrap class so as to prevent unpredictable wrapping
Please find below screenshots for reference:

  • Before flex-nowrap

navbar_issue_before

  • After flex-nowrap

navbar_issue_after

Added flex-nowrap bootstrap class so as to prevent unpredictable wrapping
Copy link
Contributor

@domlimm domlimm left a comment

Choose a reason for hiding this comment

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

@NaveenJoyGit thanks for your contribution! i have tested and it looks good. tried on prod too by directly adding flex-nowrap and the bug isn't there. lgtm for this solution!

however, do fix your PR description you're missing Fixes #ISSUE_NUMBER at the top. i see that you've ensured everything in the checklist is completed, that's great! you can remove the checklist, and the contents of your PR should follow under Outline of Solution. For a better picture, you could look at other PRs 👍🏻

Copy link
Member

@samuelfangjw samuelfangjw left a comment

Choose a reason for hiding this comment

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

Thanks for the fix!

@samuelfangjw samuelfangjw added s.ToMerge The PR is approved by all reviewers including final reviewer; ready for merging c.Bug Bug/defect report labels Feb 5, 2023
@samuelfangjw samuelfangjw added this to the V8.22.0 milestone Feb 5, 2023
@samuelfangjw samuelfangjw merged commit feb6d8f into TEAMMATES:master Feb 6, 2023
@NaveenJoyGit NaveenJoyGit deleted the 12043-navigation-menu-collapse branch February 6, 2023 07:12
weiquu added a commit to weiquu/teammates that referenced this pull request Feb 18, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
c.Bug Bug/defect report s.ToMerge The PR is approved by all reviewers including final reviewer; ready for merging
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Navigation menu got collapsed
3 participants