-
Notifications
You must be signed in to change notification settings - Fork 58
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
Add advanced options to block settings #1385
Comments
One big thing to note. We need to figure out how to more gracefully handle text input. Note: This is related to the other issue regarding scaling the inspector sheets. Right now, we have very limited space, considering the input is only allowed space in part of a single row. This is a problem we're already experiencing with text inputs on settings sheets for images, links, etc. I propose that we consider doing either of the following:
|
Maybe we should also relay on number of chars that is allowed/defined for particular text input. E.g. We define bunch of textinputs and we can set up max-chars for each of them. If we know that the amount of chars fits nicely to current space we have then we do not enter the extended editor. On the other scenario we expect large amount of text here and then use one of the solution above. We can achieve the same by e.g passing IMO the BottomSheet solution that @iamthomasbishop proposed sounds the most interesting. |
I might be misunderstanding your point here, but I don't think we'd want to set a max-chars limitation for text inputs unless absolutely necessary. This would be especially troublesome for long URLs, for example.
Are you referring to the first example I shared above (the dialog/alert input)? |
Yes but my point was that I like below solution most
I never liked single-field input dialogs. I find them looks ugly :( |
I would agree w/ that – I would much prefer one of the following (listed in order of preference):
If we can't do one of those things, then we should defer to a dialog as mentioned above. |
We should add the advanced options field to the block settings:
Since every block has this option we'll always be showing the BlockSettings button. So we can also simplify the code that decides to render the BlockSettings button according to
InspectorControls
having fills.The text was updated successfully, but these errors were encountered: