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 dropdown not correctly positioned horizontally when body position is relative #927

Merged
merged 1 commit into from
Jun 18, 2024

Conversation

ArnaudWeyts
Copy link
Contributor

Why

We noticed that in certain use cases the dropdown is not correctly positioned horizontally when the body is set to position: relative and you scroll horizontally. When digging through some issues and the code I discovered this fix for vertical positioning. I believe that we should also implement a similar fix for horizontal positioning. We've patched this dependency in our own app and have verified that it works.

Reproducible use case (scroll to the right, then open a dropdown): https://stackblitz.com/edit/ember-cli-editor-output-1hdarw?file=app%2Ftemplates%2Fapplication.hbs,app%2Fstyles%2Fapp.css,app%2Fapp.js&title=Ember%20Starter

Screenshot 2024-06-18 at 15 37 38

Copy link
Collaborator

@mkszepp mkszepp left a comment

Choose a reason for hiding this comment

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

Thanks, LGTM

@mkszepp mkszepp added the bug label Jun 18, 2024
@mkszepp mkszepp merged commit d1b2320 into cibernox:master Jun 18, 2024
19 checks passed
@ArnaudWeyts ArnaudWeyts deleted the calculate-position-width-fix branch June 18, 2024 15:53
@ArnaudWeyts
Copy link
Contributor Author

Hi @mkszepp I see we didn't create a new release yet that includes this change, any chance we could?

@mkszepp
Copy link
Collaborator

mkszepp commented Jul 26, 2024

Sry i will publish everything on begin of next week

@mkszepp
Copy link
Collaborator

mkszepp commented Jul 31, 2024

@ArnaudWeyts now released in v8.2.0

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

Successfully merging this pull request may close these issues.

2 participants