-
Notifications
You must be signed in to change notification settings - Fork 806
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
Align, resize, and layout everything uniformly in the unified search view #4829
Conversation
ba9d3f5
to
d44ad50
Compare
Codecov Report
@@ Coverage Diff @@
## master #4829 +/- ##
=======================================
Coverage 57.17% 57.18%
=======================================
Files 138 138
Lines 17133 17133
=======================================
+ Hits 9796 9797 +1
+ Misses 7337 7336 -1
|
Looks very good! 2 details:
|
d44ad50
to
6714f96
Compare
The tooltip presents the text in the unified search result fully (i.e. without eliding). Would you still want the tooltip removed?
Thanks, good catch! Fixed now: |
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.
Seems good now! :)
6714f96
to
1245586
Compare
…view Signed-off-by: Claudio Cambra <claudio.cambra@gmail.com>
1245586
to
316f398
Compare
AppImage file: nextcloud-PR-4829-316f3981abc6a007750f81b351fd8ecbed41f2ab-x86_64.AppImage |
SonarCloud Quality Gate failed. |
Addresses a point in #4434
Screen.Recording.2022-08-09.at.00.22.59.mov