-
Notifications
You must be signed in to change notification settings - Fork 6
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
🎨 [#1131] Dropdown on mobile partly rendered off-screen #473
🎨 [#1131] Dropdown on mobile partly rendered off-screen #473
Conversation
90f3c94
to
e56e08a
Compare
@@ -11,7 +11,7 @@ | |||
flex-direction: column; | |||
padding: var(--spacing-medium) 0; | |||
position: absolute; | |||
left: 0; | |||
right: 0; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@vaszig I had only updated that for the mobile views in a different PR yes, but we still need this new PR to also make the alignment correct everywhere else (for desktops), as based on the actual design (see my description on top of this page).
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
If this is what we want for the different size screens, the code looks good to me. @alextreme can you please give us your opinion?
Codecov Report
@@ Coverage Diff @@
## develop #473 +/- ##
========================================
Coverage 96.46% 96.46%
========================================
Files 506 506
Lines 18263 18263
========================================
Hits 17617 17617
Misses 646 646
📣 We’re building smart automated test selection to slash your CI/CD build times. Learn more |
issue in Taiga: #1131 Dropdown on mobile partly rendered off-screen
In the original design it already showed the drop down should have been aligned to the right:
(https://projects.invisionapp.com/share/Y211V6JRHPC4#/screens/470665006)