Skip to content
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

Styles Panel: Don't force it to be closed by default. #20617

Merged
merged 1 commit into from
Mar 17, 2020

Conversation

pento
Copy link
Member

@pento pento commented Mar 4, 2020

Description

#6038 originally hid the Advanced panel by default, which I agree is the correct behaviour. In #10543, which turned the Advanced panel into the Styles panel, this behaviour was retained.

Whilst working on Automattic/jetpack#14852, I found the inability to set the initialOpen value of the Styles panel to be a little frustrating, but on further investigation, I'm inclined to think that #10543 incorrectly retained this behaviour from the Advanced panel, and it should be changed.

How has this been tested?

Tested against Automattic/jetpack#14852.

It can also be tested against Core blocks that provide styles: for example, the Quote block.

Checklist:

  • My code is tested.
  • My code follows the WordPress code style.
  • My code follows the accessibility standards.
  • My code has proper inline documentation.
  • I've included developer documentation if appropriate.
  • I've updated all React Native files affected by any refactorings/renamings in this PR.

@pento pento added Needs Design Feedback Needs general design feedback. [Feature] Theme Style Variations Related to style variations provided by block themes labels Mar 4, 2020
@pento pento self-assigned this Mar 4, 2020
@github-actions
Copy link

github-actions bot commented Mar 4, 2020

Size Change: -1 B

Total Size: 865 kB

Filename Size Change
build/block-editor/index.js 105 kB -1 B
ℹ️ View Unchanged
Filename Size Change
build/a11y/index.js 1.01 kB 0 B
build/annotations/index.js 3.43 kB 0 B
build/api-fetch/index.js 3.39 kB 0 B
build/autop/index.js 2.58 kB 0 B
build/blob/index.js 620 B 0 B
build/block-directory/index.js 6.02 kB 0 B
build/block-directory/style-rtl.css 760 B 0 B
build/block-directory/style.css 760 B 0 B
build/block-editor/style-rtl.css 10.5 kB 0 B
build/block-editor/style.css 10.5 kB 0 B
build/block-library/editor-rtl.css 7.36 kB 0 B
build/block-library/editor.css 7.36 kB 0 B
build/block-library/index.js 116 kB 0 B
build/block-library/style-rtl.css 7.5 kB 0 B
build/block-library/style.css 7.51 kB 0 B
build/block-library/theme-rtl.css 669 B 0 B
build/block-library/theme.css 671 B 0 B
build/block-serialization-default-parser/index.js 1.65 kB 0 B
build/block-serialization-spec-parser/index.js 3.1 kB 0 B
build/blocks/index.js 57.6 kB 0 B
build/components/index.js 191 kB 0 B
build/components/style-rtl.css 15.6 kB 0 B
build/components/style.css 15.5 kB 0 B
build/compose/index.js 5.76 kB 0 B
build/core-data/index.js 10.5 kB 0 B
build/data-controls/index.js 1.03 kB 0 B
build/data/index.js 8.22 kB 0 B
build/date/index.js 5.37 kB 0 B
build/deprecated/index.js 772 B 0 B
build/dom-ready/index.js 568 B 0 B
build/dom/index.js 3.06 kB 0 B
build/edit-post/index.js 90.9 kB 0 B
build/edit-post/style-rtl.css 8.53 kB 0 B
build/edit-post/style.css 8.53 kB 0 B
build/edit-site/index.js 4.63 kB 0 B
build/edit-site/style-rtl.css 2.51 kB 0 B
build/edit-site/style.css 2.51 kB 0 B
build/edit-widgets/index.js 4.42 kB 0 B
build/edit-widgets/style-rtl.css 2.59 kB 0 B
build/edit-widgets/style.css 2.58 kB 0 B
build/editor/editor-styles-rtl.css 325 B 0 B
build/editor/editor-styles.css 327 B 0 B
build/editor/index.js 44.6 kB 0 B
build/editor/style-rtl.css 3.98 kB 0 B
build/editor/style.css 3.98 kB 0 B
build/element/index.js 4.45 kB 0 B
build/escape-html/index.js 733 B 0 B
build/format-library/index.js 7.6 kB 0 B
build/format-library/style-rtl.css 502 B 0 B
build/format-library/style.css 502 B 0 B
build/hooks/index.js 1.92 kB 0 B
build/html-entities/index.js 622 B 0 B
build/i18n/index.js 3.48 kB 0 B
build/is-shallow-equal/index.js 710 B 0 B
build/keyboard-shortcuts/index.js 2.3 kB 0 B
build/keycodes/index.js 1.68 kB 0 B
build/list-reusable-blocks/index.js 2.99 kB 0 B
build/list-reusable-blocks/style-rtl.css 226 B 0 B
build/list-reusable-blocks/style.css 226 B 0 B
build/media-utils/index.js 4.85 kB 0 B
build/notices/index.js 1.57 kB 0 B
build/nux/index.js 3.02 kB 0 B
build/nux/style-rtl.css 616 B 0 B
build/nux/style.css 613 B 0 B
build/plugins/index.js 2.54 kB 0 B
build/primitives/index.js 1.49 kB 0 B
build/priority-queue/index.js 780 B 0 B
build/redux-routine/index.js 2.84 kB 0 B
build/rich-text/index.js 14.3 kB 0 B
build/server-side-render/index.js 2.54 kB 0 B
build/shortcode/index.js 1.7 kB 0 B
build/token-list/index.js 1.27 kB 0 B
build/url/index.js 4 kB 0 B
build/viewport/index.js 1.61 kB 0 B
build/warning/index.js 1.14 kB 0 B
build/wordcount/index.js 1.18 kB 0 B

compressed-size-action

@scruffian
Copy link
Contributor

This looks good. I do wonder if we should make it an option. We could combine with #20620, and provide a few different options for styles...

@Copons
Copy link
Contributor

Copons commented Mar 12, 2020

I agree with @scruffian in that it would be nice to have a set of options for the styles panel.

Also, changing the current behaviour for existing blocks might be confusing.
What about having an initialOpen option and defaulting it to false?

@pento
Copy link
Member Author

pento commented Mar 13, 2020

I'm wary of making a configurable option for something this small, particularly given that I don't think the current behaviour was intended.

@pento pento added this to the Gutenberg 7.8 milestone Mar 13, 2020
@youknowriad
Copy link
Contributor

@jasmussen @mapk thoughts

@jasmussen
Copy link
Contributor

Just to clarify:

  • This is for the style picker that is shown in the sidebar when selecting, for example, a blockquote or separator?
  • This does not change the behavior of the "Advanced" section, which still defaults to being closed?

If yes to both of these, I'd say ship it as it seems an obvious improvement.

@pento pento merged commit eda85f2 into master Mar 17, 2020
@pento pento deleted the change/show-styles-panel-by-default branch March 17, 2020 00:52
@pento pento removed the Needs Design Feedback Needs general design feedback. label Mar 17, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
[Feature] Theme Style Variations Related to style variations provided by block themes
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants