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

Fix user menu text alignment #16596

Merged
merged 2 commits into from
Aug 22, 2024
Merged

Conversation

mvarblow
Copy link
Contributor

Fixes #16595

@hyzx86
Copy link
Contributor

hyzx86 commented Aug 21, 2024

Can you provide screenshots before and after the fix? Thanks!

@hishamco
Copy link
Member

As @hyzx86 said please share before and after screenshoot

Copy link
Contributor

This pull request has merge conflicts. Please resolve those before requesting a review.

@mvarblow
Copy link
Contributor Author

Hi @hishamco, @hyzx86. Thanks for taking a look. I included screen shots on the associated issue. #16595

@mvarblow
Copy link
Contributor Author

It had merge conflicts with my other pull request from today. Those conflicts are resolved now. :)

@Piedone
Copy link
Member

Piedone commented Aug 22, 2024

@gvkries when you approve a PR, and especially for someone who doesn't have merge permissions, please also merge right away after waiting for the functional tests to pass. Or, if you're waiting for a second opinion, make it explicit with a comment (and then merge after like 1-2 days if nobody else chimes in, so we don't have PRs lingering indefinitely).

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Slightly misaligned icons in user menu
5 participants