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

Merged
merged 7 commits into from
May 29, 2024

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 :)

@alperozturk96 alperozturk96 force-pushed the Feature/Add-collabsable-icon-in-uploads-tab branch from 6686aa9 to 4dd35a3 Compare May 24, 2024 08:22
@JonasMayerDev JonasMayerDev force-pushed the Feature/Add-collabsable-icon-in-uploads-tab branch from a4755ed to 65e3b79 Compare May 28, 2024 13:57
@JonasMayerDev
Copy link
Collaborator Author

Screenshot_20240528_164634

Thats what it looks like now.

JonasMayerDev and others added 7 commits May 29, 2024 16:38
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>
Signed-off-by: Jonas Mayer <jonas.a.mayer@gmx.net>
Signed-off-by: Jonas Mayer <43114340+JonasMayerDev@users.noreply.github.com>
Signed-off-by: github-actions <github-actions@github.com>
@alperozturk96 alperozturk96 force-pushed the Feature/Add-collabsable-icon-in-uploads-tab branch from f36ab6e to 5ba6784 Compare May 29, 2024 14:38
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

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.

@JonasMayerDev JonasMayerDev merged commit 5922cd7 into master May 29, 2024
21 checks passed
@delete-merged-branch delete-merged-branch bot deleted the Feature/Add-collabsable-icon-in-uploads-tab branch May 29, 2024 17:56
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

5 participants