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

Resolved #4185 where the page scrolled up when the file field modal window was opened #4195

Open
wants to merge 2 commits into
base: 7.dev
Choose a base branch
from

Conversation

Yulyaswan
Copy link
Collaborator

Resolved #4185 where the page scrolled up when the file field modal window was opened

@Yulyaswan Yulyaswan added the Bug: Accepted Bug has been confirmed, is reproducible, and ready to work on. label Apr 3, 2024
@Yulyaswan Yulyaswan added this to the 7.x milestone Apr 3, 2024
@bryannielsen bryannielsen modified the milestones: 7.x, 7.4.7 Apr 3, 2024
@bryannielsen
Copy link
Contributor

I think this fixes the behavior of bringing the scroll back to the field you were editing but when I try to choose an image I can no longer scroll within the modal for selecting an image further down in my upload location. If I switch back to 7.dev it appears to work again.

@bryannielsen bryannielsen modified the milestones: 7.4.7, 7.x Apr 5, 2024
Copy link
Contributor

@bryannielsen bryannielsen left a comment

Choose a reason for hiding this comment

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

Now I'm having issues scrolling back to the top of the file modal. I can't see the top of my files list or the search/filter bars in the file modal and I can't scroll at all.

My setup is a file field down at the bottom of the channel entry publish screen and a long list of files in my upload location. I'm also making my window fairly narrow to make the scroll area bigger.

@Yulyaswan
Copy link
Collaborator Author

I can't reproduce this

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Bug: Accepted Bug has been confirmed, is reproducible, and ready to work on.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

File field UI issue
2 participants