-
Notifications
You must be signed in to change notification settings - Fork 47
Support permissions in CMS Starter #211
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
Conversation
|
Looking at the screenshot you shared I would expect all inputs to be disabled. And I think the Insufficient Permissions text is really ugly. Just disabling the button should be clear enough. Maybe we could add that text in the title tooltip of the button if you think that is helpful |
a37b0a7 to
72bc08b
Compare
72bc08b to
2dc8376
Compare
2dc8376 to
132bdb1
Compare
132bdb1 to
532756e
Compare
532756e to
1102065
Compare
1102065 to
a0c2aa1
Compare
a0c2aa1 to
7614f82
Compare
jonastreub
left a comment
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.
Great work 👏
7614f82 to
6d1607d
Compare
9472e58 to
56bb118
Compare
6d1607d to
e0ddff6
Compare
Description
Warning: To be merged when permissions are out of beta!
titleon the import button.Fixes: https://github.com/framer/company/issues/32805
Testing