-
Notifications
You must be signed in to change notification settings - Fork 4.2k
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
Section styles: improve (or remove) preview #63035
Comments
+1 for improving (or removing) previews for section styles. |
Just flagging that there were a couple of earlier explorations into updating the UI for Block Styles:
While these were focused on the control itself, e.g. dropdown vs small cards etc., it did come up that the preview isn't really required as it was so easy to see it in action or switch back to the default. I can put together a quick PR to remove the preview and see what people think. |
TBH, I think this is better (and likely easier to ship with RC2?) to remove than to style a preview. |
I think this might be seen more as an enhancement rather than a bug fix, so I don't think it will make it into any of the 6.6 RCs. |
Heh, I could make the argument that it is a bug, but that would only be wishful thinking. In short, I understand and agree with what you said. |
A draft PR is up for removing the block style previews from the inspector panel only: #63039. I am having some problems with my local native env at the moment so I've left the PR in a draft state until I can sort that out and see whether it still makes sense to remove the previews there too. |
The block styles still use an inline preview for the native UI. Given the next iteration of section styles/block style variations will likely involve a major update to this panel, I wonder if it makes sense to hold off on updating the native UI. I've suggested over on #63039 that this could be a separate follow-up but happy to hear other's thoughts on this one. |
Thanks for the quick work here. |
The removal of previews appears to be overkill. An alternate approach, simply improving the Group block's example, is available in #63114. |
Currently, the previews for block style variations don't offer much value when applied to styling sections of blocks with the new feature in 6.6. If anything, the preview looks broken and doesn't match all of the changes one can expect to see:
Here's a video:
Section.Styles.previews.mov
As a result, I think we should either remove these previews for Groups & Columns for now or consider a way to improve the preview offered cc @aaronrobertshaw
The text was updated successfully, but these errors were encountered: