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
align subadmins css class name #38793
align subadmins css class name #38793
Conversation
73adc86
to
eb4bddb
Compare
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.
What about
class="subadmins"> |
class="subadmins"> |
?
Don't .subadmin
target other elements?
Why don't the component declare its own css?
You're perfectly right, I seem to have missed the spots where the subadmins class is actually used. I've duplicated the css now so both class names work, thanks! |
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.
Looks good
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.
looks good now
…sGroups to fix display issue when users are in many groups Signed-off-by: Silke Suck <silke.suck@adacor.com>
Signed-off-by: Silke Suck <silke.suck@adacor.com>
e7ddbd6
to
1442cfb
Compare
Thanks for your first pull request and welcome to the community! Feel free to keep them coming! If you are looking for issues to tackle then have a look at this selection: https://github.com/nextcloud/server/issues?q=is%3Aopen+is%3Aissue+label%3A%22good+first+issue%22 |
Thanks @Jenandra ! |
Summary
The actually used class name in apps/settings/src/components/UserList/UserRowSimple.vue is subAdminsGroups. Because of this, css for the subadmins groups column in the user list is missing, causing formatting to break when users are admins of many (10+) groups.
See also: https://help.nextcloud.com/t/ansicht-von-user-mit-vielen-gruppen-ist-unubersichtlich/72149/3