-
Notifications
You must be signed in to change notification settings - Fork 805
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
Simplify activity list delegates by making them ItemDelegates, clean up #4786
Conversation
Kudos, SonarCloud Quality Gate passed! |
Codecov Report
@@ Coverage Diff @@
## master #4786 +/- ##
==========================================
+ Coverage 57.15% 57.23% +0.07%
==========================================
Files 138 138
Lines 17135 17146 +11
==========================================
+ Hits 9794 9813 +19
+ Misses 7341 7333 -8
|
@claucambra Nice, but, please check my comment. |
7976538
to
2028b7e
Compare
Kudos, SonarCloud Quality Gate passed! |
2028b7e
to
283188d
Compare
Signed-off-by: Claudio Cambra <claudio.cambra@gmail.com>
283188d
to
85c5002
Compare
AppImage file: nextcloud-PR-4786-85c5002d34f8f41847d9d166c6f3a7c86b8267c9-x86_64.AppImage |
Rather than use a
MouseArea
and some wacky workarounds to get the sizing we want, theItemDelegate
does exactly what we want to the proper waySigned-off-by: Claudio Cambra claudio.cambra@gmail.com