Improvements for personal info view
Open
@nimishavijay is already working on this.
Since Jul 28, 2023.
1. to develop
design
enhancement
feature: profile
- Dominant language
- PHP
- Stars
- 36.9k
- Forks
- 5.2k
- Avg merge
- 2d 3h
- Merged PRs (30d)
- 713
Description
I would suggest to not include the labels for this PR, just remove the opacity on the buttons. My reasoning:
- The labels don't make much sense for people who are new (for eg. "Federated" would be shown by default, which is not super easy to understand)
- A label may also take up a lot of space, esp on mobile and for translated labels
- Using a secondary styling will also draw a lot of attention for those elements, which are not the main focus at all. The result will be something like the screenshot attached (couldn't get the labels to work :/ ). Even if we are using tertiary button styling the button labels will be bold while the field labels will be regular weight, drawing more attention to the button.
Since there is a follow up issue anyway, I'd say for this a11y problem we can just change the opacity :) In the future when we change the layout of the entire page we can make sure that we use the correct components and styling :)
Originally posted by @nimishavijay in https://github.com/nextcloud/server/issues/39476#issuecomment-1643996623
Contributor guide
First steps
- Read the whole issue, then the project's contributing guide.
- Comment on the issue to say you are picking it up — it saves two people doing the same work.
- Fork the repository and make your change on a branch.
- Open a pull request that references the issue number.
Assessment
This issue has not been assessed yet.