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.

Sign up for GitHub

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

Jump to bottom

[Dialog] Move scroll to content #2977

Open
jeremie-lucca opened this issue Jul 23, 2024 · 1 comment · May be fixed by #3071
Open

[Dialog] Move scroll to content #2977

jeremie-lucca opened this issue Jul 23, 2024 · 1 comment · May be fixed by #3071
Labels
🔨 Technical Doesn't affect the output (refactor, dependencies update, cleaning, etc.) 👥 UI Requires UI/HTML/CSS skills
Milestone
18.3

Comments

@jeremie-lucca
Copy link
Contributor

It was decided that scrollbar should be on content instead of full dialog to avoid ugly UI on Windows.
We don't have much confidence toward breakings, so it should be heavily tested before merging.
@jeremie-lucca jeremie-lucca added 👥 UI Requires UI/HTML/CSS skills 🔨 Technical Doesn't affect the output (refactor, dependencies update, cleaning, etc.) labels Jul 23, 2024
@jeremie-lucca jeremie-lucca added this to the 18.2 milestone Jul 23, 2024
@vvalentin-lucca
Copy link
Contributor

https://codepen.io/collection/LPKYMV

@jeremie-lucca jeremie-lucca modified the milestones: 18.2, 18.3 Aug 9, 2024
@vvalentin-lucca vvalentin-lucca linked a pull request Sep 11, 2024 that will close this issue
Open
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
🔨 Technical Doesn't affect the output (refactor, dependencies update, cleaning, etc.) 👥 UI Requires UI/HTML/CSS skills
Projects
None yet
Milestone
18.3
Development

Successfully merging a pull request may close this issue.

[dialog] Internal scroll LuccaSA/lucca-front
2 participants
@jeremie-lucca @vvalentin-lucca