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

Make the interface more compact #45657

Closed
7 tasks done
Tracked by #45751
marcoambrosini opened this issue Jun 4, 2024 · 8 comments
Closed
7 tasks done
Tracked by #45751

Make the interface more compact #45657

marcoambrosini opened this issue Jun 4, 2024 · 8 comments
Assignees
Labels
4. to release Ready to be released and/or waiting for tests to finish enhancement
Milestone

Comments

@marcoambrosini
Copy link
Member

marcoambrosini commented Jun 4, 2024

Part of #45751

There's a lot of whitespace between elements, some elements are too big and the usability of Nextcloud would improve if the UI was overall more compact.

  • We've used so far a margin of 8 pixels between list items, buttons and boundaries, we could cut that down to 4 (--default-grid-baseline).
  • We are using a minimum target size of 44 pixels for all clickable elements, while the actual accessibility requirement is much lower than that (24px). Ref: https://www.w3.org/WAI/WCAG22/Understanding/target-size-minimum.html
    I know that touchscreens are a concern for these elements, but we can have fallbacks in place for those, or preserve a bigger size for main action targets.

As an example, this is the range of target sizes in dropbox and google drive. They range from 17px (a link) to 74px (main actions) and a hierarchy is established. Conversely almost all our elements have a 44px minimum target size, even the table header.

Screenshot 2024-06-10 at 12 46 34 Screenshot 2024-06-10 at 13 43 58

TODO

📐 Main work

@marcoambrosini marcoambrosini added enhancement 0. Needs triage Pending check for reproducibility or if it fits our roadmap labels Jun 4, 2024
@marcoambrosini marcoambrosini self-assigned this Jun 4, 2024
@susnux

This comment was marked as outdated.

@marcoambrosini

This comment was marked as outdated.

@skjnldsv

This comment was marked as resolved.

@marcoambrosini

This comment was marked as resolved.

@skjnldsv

This comment was marked as resolved.

@susnux

This comment was marked as resolved.

@ChristophWurst ChristophWurst added 2. developing Work in progress and removed 0. Needs triage Pending check for reproducibility or if it fits our roadmap labels Jul 2, 2024
@jancborchardt

This comment was marked as resolved.

ChristophWurst added a commit to nextcloud/mail that referenced this issue Jul 2, 2024
nextcloud/server#45657 has changes that are
"breaking". If we adjust the app for 30 it will look bad on 29.

Signed-off-by: Christoph Wurst <christoph@winzerhof-wurst.at>
@marcoambrosini
Copy link
Member Author

everything done :)

@jancborchardt jancborchardt added this to the Nextcloud 30 milestone Aug 13, 2024
@skjnldsv skjnldsv modified the milestones: Nextcloud 30, Nextcloud 31 Aug 14, 2024
@jancborchardt jancborchardt added 4. to release Ready to be released and/or waiting for tests to finish and removed 2. developing Work in progress labels Aug 14, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
4. to release Ready to be released and/or waiting for tests to finish enhancement
Projects
Archived in project
Development

No branches or pull requests

5 participants