-
Notifications
You must be signed in to change notification settings - Fork 2
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
Update the README's FAQ to make the default settings more clear #36
Comments
Would you also consider relabeling in the plugin itself? |
Hey @danieliser, The plugin name works for me. Did you have any other names in mind? BTW, I'm having fun with it on my test site. I'm going to add it on my live site next. Thanks! |
@marklchaves Should have been more specific, the labeling inside the plugin for those fields you are explaining or others. Do you see similar simplifications like we made in PM's interfaces you can suggest them here as well. |
Hey @danieliser, Sorry. Yes. Absolutely. Today, I'll work on a PR for the README too. Thanks! |
Created PR #37 |
Update the README's FAQ How do I setup this plugin? question so the default behavior is more clear.
Proposed changes
Logged in Users option
The Choose which roles can see this link radio button is selected by default.
If no roles are selected, all roles can see the menu item by default.
Once a role is checked, then only checked roles can see the menu item.
The Choose which roles won't see this link is not selected by default. When this radio button is selected, these rules apply:
If no roles are selected, all roles still have visibility to the menu item by default.
Once a role is checked, then only checked roles won't see the menu item.
The text was updated successfully, but these errors were encountered: