-
-
Notifications
You must be signed in to change notification settings - Fork 4.1k
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
Responsive app menu #4439
Responsive app menu #4439
Conversation
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.
Can confirm @icewind1991's issue. It also results in a double scrollbar on the right in Firefox, and there are too few items shown by default (should be 8 icons).
(Screenshot is after scrolling, so the files are visible – shows double scrollbar and too few icons.)
Signed-off-by: Julius Härtl <jus@bitgrid.net>
Signed-off-by: Julius Härtl <jus@bitgrid.net>
Signed-off-by: Julius Härtl <jus@bitgrid.net>
07f668d
to
27b19aa
Compare
Codecov Report
@@ Coverage Diff @@
## master #4439 +/- ##
============================================
+ Coverage 53.9% 54.09% +0.19%
+ Complexity 21701 21695 -6
============================================
Files 1272 1331 +59
Lines 75815 83079 +7264
Branches 0 1315 +1315
============================================
+ Hits 40870 44944 +4074
- Misses 34945 38135 +3190
|
Ready for review @nextcloud/designers |
@icewind1991 Thanks for having a look. Should be fixed now, there was some leftover css rule from development. 🙈 |
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.
Tested and works (even in Edge ;)) 👍
height: 16px; | ||
} | ||
.app-icon { | ||
-ms-filter: 'progid:DXImageTransform.Microsoft.Alpha(Opacity=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.
Do we still need those ugly/strange opacity filters for ie?
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.
Incredible work! :)
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.
Works great now! 👍
Awesome work @juliushaertl! This is so cool :) @MorrisJobke @LukasReschke @rullzer what do we do about the codecov patch diff? |
Ideally in a world where everything is awesome: Add tests for this 😉 /me hides 😉 |
… as that isn't the case => merged 😉 |
As discussed at the hackweek with @jancborchardt
ToDo: