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

Design regressions of "Recommended files" widget #660

Closed
5 tasks done
jancborchardt opened this issue Oct 17, 2023 · 8 comments
Closed
5 tasks done

Design regressions of "Recommended files" widget #660

jancborchardt opened this issue Oct 17, 2023 · 8 comments

Comments

@jancborchardt
Copy link
Member

jancborchardt commented Oct 17, 2023

There are some regressions to the recommended Files widget as seen below:

  • Filetypes without preview are shown as if they are avatar placeholders, not with their proper mimetype icon
  • Folders are put in a circle, not just shown as icon
  • Similar for previews, they are circled, but should be a square with rounded corners like in the Files app
  • File names are aligned to the top, but should be vertically centered with the filetype icon / preview as before
  • File extensions are shown same color as filename, but should be color-text-maxcontrast as we do in Files
Intended Current Nextcloud
image image
@jancborchardt
Copy link
Member Author

@szaimen @marcoambrosini could one of you pick this up to fix this, maybe even for 29? It’s a regression which makes the Dashboard look a bit unpolished.

@jancborchardt jancborchardt moved this from 🧭 Planning evaluation / ideas to 📐 Design phase in 🖍 Design team Mar 8, 2024
@susnux
Copy link
Contributor

susnux commented Mar 9, 2024

@jancborchardt the dashboard widget looks exactly like all others - if you prefer the less rounded one then we need to change it also for all others (e.g. the "statuses" dashboard).

@susnux
Copy link
Contributor

susnux commented Mar 9, 2024

Like the general dashboard widget styles are that rounded - same as the login box.
Screenshot 2024-03-09 at 15-47-19 Nextcloud
Screenshot 2024-03-09 at 15-47-04 Dashboard - Nextcloud

Unrelated but we should probably either use the drop shadow for all of these components or for none for consistency (there is shadow for the login box but not for dashboard widgets), no?

@jancborchardt jancborchardt moved this from 📐 At design to 🕶️ Design review in 🖍 Design team Mar 27, 2024
@jancborchardt
Copy link
Member Author

@jancborchardt the dashboard widget looks exactly like all others - if you prefer the less rounded one then we need to change it also for all others (e.g. the "statuses" dashboard).

@susnux sorry the screenshot is a bit confusing – I am not talking about the bigger border-radius of the outside container, that is of course fine. :) I mean that the filetype icons themselves are in a rounded container, being cut off.

@susnux
Copy link
Contributor

susnux commented Mar 27, 2024

Ah yes I see, the avatar component should probably not used here, it makes not much sense.

@emoral435 emoral435 self-assigned this Apr 1, 2024
@emoral435
Copy link

This seems interesting. I have time right now to pick this up, so will be working on this 👍

@emoral435
Copy link

emoral435 commented Apr 16, 2024

Really weird finding --- on my mac, the recommendations widget seems to be pulling directly from the recommendations app itself, and shows that there is no regression to be found - everything is styled correctly:

image

However, on my windows, it seems like the dashboard widget is using our default NcWidget component, and other default dashboard components. Will try to debug this, but it is a bit weird 😕

@marcoambrosini marcoambrosini moved this to 👓 Design review in 🖍 Design team Apr 22, 2024
@jancborchardt jancborchardt moved this from 👓 Design review to 🏗️ At engineering in 🖍 Design team Apr 22, 2024
@emoral435 emoral435 removed their assignment May 25, 2024
@jancborchardt
Copy link
Member Author

This was all fixed by now.

@github-project-automation github-project-automation bot moved this from 🏗️ At engineering to 🎉 Done in 🖍 Design team Aug 19, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
Archived in project
Development

Successfully merging a pull request may close this issue.

3 participants