Could the permissions be grouped per workflow type? It is now alphabetically sorted per state and this will be very hard to use with more workflow types. (See image.)
| Comment | File | Size | Author |
|---|---|---|---|
| #6 | CM_permissions_after.PNG | 19.64 KB | neograph734 |
| #4 | group_content-2899306-4.patch | 2.33 KB | neograph734 |
| #2 | group_content-2899306-2.patch | 1.03 KB | neograph734 |
Comments
Comment #2
neograph734Inverted the pattern so permissions now look like this (to match other core modules):
Editorial workflow: Use Publish transition.
Comment #4
neograph734Updated the test
Comment #5
timmillwoodAwesome work @Neograph734, looking nice.
Great the permission ID hasn't changed so should be fully backwards compatible.
Might be worth showing a before/after screenshot for UX review.
Comment #6
neograph734Thanks @timmillwood :)
Here is a screenshot after the patch, grouped by workflow name (In my test setup Profiles was renamed to Article):
Comment #7
timmillwoodThanks, lets see what the usability team think, then move to RTBC.
Comment #8
Bojhan commentedMooooore overview on permissions, lets do it :)
Labeling wise this seems fine, and it makes sense to group this.
Comment #12
larowlanCommitted as 517619e and pushed to 8.5.x.
Cherry-picked as da667d3 and pushed to 8.4.x.