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

sticky close buttons in notify? #541

Open
danielghost opened this issue Nov 27, 2024 · 2 comments
Open

sticky close buttons in notify? #541

danielghost opened this issue Nov 27, 2024 · 2 comments

Comments

@danielghost
Copy link

Should we look into whether elements such as close buttons in notify popups should use position: sticky and adjust the UI accordingly so that when a popup requires scrolling, the close button moves with the scroll to prevent a user from needing to scroll back to the top to use the button (ignoring that it can be closed by clicking outside the popup depending on config and screen size)?

@oliverfoster
Copy link
Member

This seems more of a theme / fed related stylistic choice, can I move to vanilla? @danielghost

@danielghost danielghost transferred this issue from adaptlearning/adapt-contrib-core Jan 8, 2025
@oliverfoster oliverfoster moved this from Awaiting Response to New in adapt_framework: The TODO Board Jan 8, 2025
@oliverfoster
Copy link
Member

Could you do a PR or ask a FED to do it and work out the requirements and problems? @danielghost

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

No branches or pull requests

2 participants