-
-
Notifications
You must be signed in to change notification settings - Fork 188
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
fix role map for Show Toolbar Permission
#2108
Conversation
see end of discussion of #1570
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
idea was to not change the default behavior, but if others are fine with it, i'm too
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@fgrcon I'm totally against this change: the toolbar must be fixed as a whole and not just to solve your specific use case.
as mentioned before, I agree with @jensens: all authenticated users must see at least a toolbar with the personal bar or we'll be creating new basic usability problems like user doesn't know she is logged-in and user doesn't know how to log out.
if you want to hide the toolbar on your projects, do that as part of your policy.
yes, please add Authenticated to the defaults - if you then remove it in your policy thats fine. |
absolutely sure about that. ;-) |
Bye |
1 similar comment
Bye |
after a while (but still a little bit annoyed ...) @jensens: adding authenticated renders all of @agitator s changes useless. But if you prefer to force plone users to create policies for something which really should be default behavior - well ok for me .. |
@fgrcon you are right about the "regression" uiwise there is the problem that the personal actions would switch between top-right and bottom left depending on the users permission, which would be inconsistent. in my usecase, an intranet, i'm adding additional edit-tools within the content area and never show the toolbar for that role, so that the user doesn't have that inconsistency. |
@fgrcon of course we have a regression and many other problems related to the toolbar: previously we had different elements: an edit bar and a personal bar among them; today we have only one big toolbar with huge UI/UX problems and I've being pointing to them even before the release of Plone 5 as you can see in #957. you are messing around the so start by being careful yourself and be open to criticism: @jensens had already rose the concern before me and you just ignored both of us. |
The problem of a more consistent editing UI is bigger than a bunch of small fixes introducing new problems. I hope Pastanaga will help in future. We may also improve the current toolbar, but this needs to be done carefully with discussions, probably best at a sprint or conference with a wide variety of Plonistas attending. |
see end of discussion of #1570