[4.x]: New naming convention for user permissions is very unclear #13375
-
I'm setting up more user groups for a Craft 4 site, and I've just seen that the labels for permissions have been changed. In Craft 3, you could set up a user group which allowed a user to create entries and save changes, but they couldn't make those changes live and required a user with the "Publish live changes" permission to actually save the entry. I find the new naming convention for the ability to "Save entries" very unclear. Please consider changing it back to how it used to be, "Publish live changes".
Craft CMS version4.4.14 |
Beta Was this translation helpful? Give feedback.
Replies: 9 comments
-
The permission changes were made via #10467. Maybe this helps clear it up?
|
Beta Was this translation helpful? Give feedback.
-
I guess closing it means it won't be changed, but no - I still think the wording is confusing. When you're creating new user groups, I think it's now very unclear how you can set up a group that is able to edit, but not publish their changes. |
Beta Was this translation helpful? Give feedback.
-
@brandonkelly For what it's worth, I agree with @darylknight – the new terminology is really confusing. Whenever I work on a site with granular permissions, I have to sift through the documentation and old issues, as well as do some trial and error. Most confusing to me is the following:
I think 2 & 3 are fine, only the fact that drafts for existing and new entries are treated differently makes it a bit confusing what the Create entries permission does. But the real issue is that Save really means Publish – the previous wording in 3.x was much clearer in that sense. I would suggest the following naming changes (no BC-break, the underlying permissions can stay the same, only the wording would be improved):
And the View permission could be made more explicit as well (a bit verbose, maybe that's a case for making this into two separate permissions):
|
Beta Was this translation helpful? Give feedback.
-
Beta Was this translation helpful? Give feedback.
-
@brandonkelly That sounds great!
Good to know the reasoning! But is it important to have that distinction in the wording of the permission? After all, users with that permission can always enable a disabled entry. And in some cases, disabled entries could still be visible in the frontend. What really matters here, I think, is that users can update/create the live/published/canonical version of a published entry (technically not canonical since unpublished drafts are also canonical). In the end, the permission allows users to update/create a version of an entry that may be visible in the frontend – in that sense, I still think publish is a better wording than save.
I think that makes perfect sense! That stood out to me as well when I wrote my comment above, though I didn't want to suggest this change since I wasn't sure if it would be a BC break 😅 It might be, in case some frontend code relies on users with the View permission to be able to create drafts for existing entries. In the meantime, I think the wording changes you mentioned are a solid improvement 👍 Though I would still suggest a change to the Save permission, that one is the least clear in my opinion. |
Beta Was this translation helpful? Give feedback.
-
This whole conversation is about confusing permission labels, so yes, I would say avoiding a label that implies something more specific than it’s actually for is important :) |
Beta Was this translation helpful? Give feedback.
-
@brandonkelly Absolutely! Though I would still argue that Save is more confusing than Publish, even given the nuances of what Publish might mean, since Save can also mean a bunch of things (like saving drafts). Maybe there's another wording that better captures the difference between creating drafts and creating something that the public might see? |
Beta Was this translation helpful? Give feedback.
-
This would be a very good call IMO, especially considering the editing UI uses "Create entry" for when a draft is first made into a real entry (an action that actually requires the "Save entries" permission).
"Save entries" is a bit fuzzy because it's technically required for (what is often perceived to be) a few different actions, like applying drafts, saving existing entries, and creating new entries from drafts. It's probably going to be difficult to find a name that accurately describes everything it does, though (at least without getting a bit verbose). Maybe instead of renaming the permission, it could be given an info toggle (similar to the one a few of the user permissions have), explaining in a bit more detail what it does? |
Beta Was this translation helpful? Give feedback.
-
This is the direction we decided to take for Craft 4, and will consider adding the explicit “Create drafts” permission in Craft 5. Craft 4.4.16 was just released, which adds the following info messages:
|
Beta Was this translation helpful? Give feedback.
This is the direction we decided to take for Craft 4, and will consider adding the explicit “Create drafts” permission in Craft 5.
Craft 4.4.16 was just released, which adds the following info messages: