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 Report][3.2.2] VMenu position incorrect the first time after switching to or from RTL #17314

Open
fahmifitu opened this issue May 7, 2023 · 3 comments · Fixed by #17338
Assignees
Labels
C: VMenu VMenu T: bug Functionality that does not work as intended/expected

Comments

@fahmifitu
Copy link
Contributor

fahmifitu commented May 7, 2023

Environment

Vuetify Version: 3.2.2
Vue Version: 3.2.24
Browsers: Chrome 112.0.0.0
OS: Mac OS 10.15.7

Steps to reproduce

  1. hover over the button then, toggle RTL switch
  2. hover over the button
    The menu is not visible due to incorrect position

Expected Behavior

to position correctly the first

Actual Behavior

it doesn't

Reproduction Link

https://play.vuetifyjs.com/#...

@fahmifitu
Copy link
Contributor Author

fahmifitu commented May 7, 2023

It seems that the issue is in updateLocation function in locationStrategies.ts

Adding requestAnimationFrame(() => { updateLocation() }) fixes this issue

As stated in the code comment "// TODO: overflowing content should only require a single updateLocation call"

@lioneaglesolutions
Copy link
Collaborator

This looks to possibly be a duplicate or very similar to;
#16325
#16628

I think they might be fixed already - check the nightly repro below here

We will leave this open until it's resolved anyway.

@lioneaglesolutions lioneaglesolutions added T: bug Functionality that does not work as intended/expected duplicate The issue has already been reported and removed S: triage labels May 8, 2023
@johnleider johnleider self-assigned this Jul 18, 2023
@KaelWD KaelWD removed the duplicate The issue has already been reported label Jul 25, 2024
@KaelWD KaelWD reopened this Jul 25, 2024
@KaelWD
Copy link
Member

KaelWD commented Jul 25, 2024

#17338 has been reverted due to #19863

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
C: VMenu VMenu T: bug Functionality that does not work as intended/expected
Projects
None yet
Development

Successfully merging a pull request may close this issue.

4 participants