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

[Bug]: Overflow Menu Styling changed to centered since 8.121.6 (and is still in 8.121.12) #33312

Closed
2 tasks done
chr-sad opened this issue Nov 21, 2024 · 0 comments · Fixed by #33325
Closed
2 tasks done

Comments

@chr-sad
Copy link

chr-sad commented Nov 21, 2024

Package

react

Package version

8.121.12

React version

17.0.1

Environment

System:
OS: Windows 11 10.0.22631
CPU: (16) x64 Intel(R) Core(TM) i9-10885H CPU @ 2.40GHz
Memory: 47.39 GB / 63.72 GB
Browsers:
Edge: Chromium (127.0.2651.105)
Internet Explorer: 11.0.22621.3527
npmPackages:
@fluentui/react: ^8.121.5 => 8.121.12
@types/react: 17.0.45 => 17.0.45
react: 17.0.1 => 17.0.1

Current Behavior

This ticket is related to #33075 (@khmakoto). Currently the overflow menu styling seems incorrect since it changed to centered which I think was not the case before - it was left aligned. For example it could be seen here on the pivot overflow menu but could also be on others:

Image

Expected Behavior

It should be left aligned again.

Reproduction

https://developer.microsoft.com/en-us/fluentui#/controls/web/pivot

Steps to reproduce

Open the Fluent UI Pivot page (https://developer.microsoft.com/en-us/fluentui#/controls/web/pivot) and scroll to Pivot with overflow menu. Force the screen to be small enough to show the 3 dots for the overflow menu and see that the items are center aligned.

Are you reporting an Accessibility issue?

None

Suggested severity

High - No workaround

Products/sites affected

No response

Are you willing to submit a PR to fix?

no

Validations

  • Check that there isn't already an issue that reports the same bug to avoid creating a duplicate.
  • The provided reproduction is a minimal reproducible example of the bug.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants