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
base: master
Are you sure you want to change the base?
Conversation
863cabc
to
b0d9bb4
Compare
Does anyone know if that change in the license header is the right procedure when modifying the material icon? @alperozturk96 @tobiasKaminsky @ZetaTom |
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.
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 :)
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>
6686aa9
to
4dd35a3
Compare
blue-Light-Screenshot test failed: https://www.kaminsky.me/nc-dev/android-integrationTests/12847-Screenshot-blue-Light-08-33 |
APK file: https://www.kaminsky.me/nc-dev/android-artifacts/12847.apk |
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 :)
Current design
I just quickly added an icon for it
@nextcloud/designers what do you think? Is that icon ok?