Skip to content

Improve privileges panel UI for record privilege groups#9240

Open
tylerjmchugh wants to merge 3 commits intogeonetwork:mainfrom
tylerjmchugh:yellow-record-privilege-group-rows
Open

Improve privileges panel UI for record privilege groups#9240
tylerjmchugh wants to merge 3 commits intogeonetwork:mainfrom
tylerjmchugh:yellow-record-privilege-group-rows

Conversation

@tylerjmchugh
Copy link
Copy Markdown
Contributor

@tylerjmchugh tylerjmchugh commented Apr 16, 2026

#8741 introduced "Record Privilege Groups" which cannot own metadata but privileges for a record can be assigned to them.

Currently it is hard to identify the "Record Privilege Groups" in the privileges panel. System groups are blue and groups that the user cannot set privileges for are yellow and the checkboxes are disabled.

This PR aims to fix this issue by changing "Record Privilege Groups" to the yellow style:
image

The disabled checkboxes should be enough to show the user they cannot set privileges for a group

Additionally I added a legend for the colors to help the users understand what they mean:
image

Checklist

  • I have read the contribution guidelines
  • Pull request provided for main branch, backports managed with label
  • Good housekeeping of code, cleaning up comments, tests, and documentation
  • Clean commit history broken into understandable chucks, avoiding big commits with hundreds of files, cautious of reformatting and whitespace changes
  • Clean commit messages, longer verbose messages are encouraged
  • API Changes are identified in commit messages
  • Testing provided for features or enhancements using automatic tests
  • User documentation provided for new features or enhancements in manual
  • Build documentation provided for development instructions in README.md files
  • Library management using pom.xml dependency management. Update build documentation with intended library use and library tutorials or documentation

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

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant