Skip to content
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

Add visual hint to collapsible headers in uploads tab #12847

Open
wants to merge 4 commits into
base: master
Choose a base branch
from

Conversation

JonasMayerDev
Copy link
Collaborator

@JonasMayerDev JonasMayerDev commented Apr 9, 2024

The dividers / headers in the upload tab are collapsible, but I think it is missing a visual hint that it is.

The fact that I only found out about that feature after countless hours in that view underlines my point :)

Screenshot_2024-04-09-11-25-17-06_367f977887f4a5f4712ef04f9b973e21
Current design

Screenshot_2024-04-09-11-18-53-22_06ea656a9ba653ff96746f477f32d0f9
I just quickly added an icon for it

@nextcloud/designers what do you think? Is that icon ok?

  • Tests written, or not not needed

@JonasMayerDev JonasMayerDev force-pushed the Feature/Add-collabsable-icon-in-uploads-tab branch from 863cabc to b0d9bb4 Compare May 23, 2024 17:42
@JonasMayerDev
Copy link
Collaborator Author

Does anyone know if that change in the license header is the right procedure when modifying the material icon? @alperozturk96 @tobiasKaminsky @ZetaTom

@JonasMayerDev JonasMayerDev marked this pull request as ready for review May 23, 2024 17:57
Copy link
Member

@nimishavijay nimishavijay left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Definitely an improvement, great catch! One small nitpick about alignment: the arrow could be center-aligned with the file icons, and the text ("Aktuell (5)", "Hochgeladen (5)") could be left aligned with the file names :)

JonasMayerDev and others added 4 commits May 24, 2024 10:22
Signed-off-by: Jonas Mayer <jonas.a.mayer@gmx.net>
Signed-off-by: Jonas Mayer <jonas.a.mayer@gmx.net>
Signed-off-by: Jonas Mayer <jonas.a.mayer@gmx.net>
Signed-off-by: github-actions <github-actions@github.com>
@alperozturk96 alperozturk96 force-pushed the Feature/Add-collabsable-icon-in-uploads-tab branch from 6686aa9 to 4dd35a3 Compare May 24, 2024 08:22
Copy link

Codacy

Lint

TypemasterPR
Warnings7170
Errors33

SpotBugs

CategoryBaseNew
Bad practice6464
Correctness6969
Dodgy code346346
Experimental11
Internationalization77
Malicious code vulnerability22
Multithreaded correctness66
Performance5757
Security1919
Total571571

Copy link

Copy link

APK file: https://www.kaminsky.me/nc-dev/android-artifacts/12847.apk

qrcode

To test this change/fix you can simply download above APK file and install and test it in parallel to your existing Nextcloud app.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants