-
Notifications
You must be signed in to change notification settings - Fork 3.4k
Add scrollable fade to more info dialog #28314
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
Conversation
|
Thanks @timmo001 for implementing and applying this component on a global scale! Does the |
It's actually just the more info dialog for now, I wasn't sure whether I should use this globally, it would probably make sense to do this, but for the webawesome variants, so bottom sheet would likely come into this. On this note I think we would need to make the ha-wa-dialog swap to bottom sheet on a configurable basis to save on duplicated effort Just made this to discuss: #28341 |
Co-authored-by: Wendelin <[email protected]>
Breaking change
Proposed change
Adds scrollable fade mixin to more info dialog. Also adds a scroll threshold to not instantly show the fade when scrolling through empty content
Screenshots
Type of change
Example configuration
Additional information
Checklist
If user exposed functionality or configuration variables are added/changed: