-
Notifications
You must be signed in to change notification settings - Fork 804
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
Replace unified search text field busy indicator with custom indicator #4753
Conversation
Codecov Report
@@ Coverage Diff @@
## master #4753 +/- ##
==========================================
- Coverage 56.80% 56.67% -0.13%
==========================================
Files 138 138
Lines 17142 17142
==========================================
- Hits 9737 9715 -22
- Misses 7405 7427 +22
|
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.
I would have kept the BusyIndicator component and just style its contentItem
to have it easier to understand that we just have a plain BusyIndicator
but with a custom style
cf1d59d
to
6ea565c
Compare
Sure -- I separated it into its own file and it is now a BusyIndicator |
6ea565c
to
75e16b8
Compare
75e16b8
to
3fa35a3
Compare
Signed-off-by: Claudio Cambra <claudio.cambra@gmail.com>
3fa35a3
to
b1deb6c
Compare
AppImage file: Nextcloud-PR-4753-b1deb6cd1d43c99e554014007c66bc86dbb5eb6a-x86_64.AppImage |
Kudos, SonarCloud Quality Gate passed! |
Addresses a point in #4434
New spinner:
Screen.Recording.2022-07-18.at.17.21.14.mov
Old spinner:
Screen.Recording.2022-07-18.at.17.26.24.mov