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

[UX] Provide an indication for the default admin/editor roles (if configured) #6099

Open
klonos opened this issue May 2, 2023 · 3 comments · May be fixed by backdrop/backdrop#4427
Open

Comments

@klonos
Copy link
Member

klonos commented May 2, 2023

Since the fieldset that allows to configure these two special role functionalities is collapsed by default, I would like us to be indicating whether a role is set as the default admin/editor in the role management page.

Ideas to consider:

  • take advatange of the descriptions/hints for fieldsets
  • use the same patern that we are currently using to indicate that the anonymous and authendicated roles are special (the (required) suffix to the role name)
  • possibly warn that the role has special functionality when deliting one of these roles (in the role deletion confirmation dialog)
  • possibly indicate if a role is special in the role edit form
  • possibly allow setting a role as the default admin/editor if not already set
@klonos klonos self-assigned this May 2, 2023
@klonos
Copy link
Member Author

klonos commented May 2, 2023

...something like this perhaps?:
image

@klonos
Copy link
Member Author

klonos commented May 2, 2023

...and this:
image

image

@klonos
Copy link
Member Author

klonos commented May 2, 2023

...each time I have to deal with confirmation forms, I'm being reminded of #3422 and the need for #3329 as well. I'll leave this as "needs feedback", and go and work on those other 2 issues instead.

In the meantime, I'm pushing what I currently have on my local in a PR that people can play with (not ready for review - has some cruft in it).

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging a pull request may close this issue.

1 participant