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

Global Styles: Re-add styles that were removed, for classic themes #45570

Conversation

shimotmk
Copy link
Contributor

@shimotmk shimotmk commented Nov 6, 2022

What?

fix #44659.
Add the styles removed in 6.1 to the CSS for the classic theme

Styles were removed in this pull request.
#43553

This pull request restored some of the styles, but left some colors removed, such as hover.
#44334

Why?

How?

Testing Instructions

  1. Please place a button block with Twenty Ten in WP6.0.
    The hover color should be white.
  2. Please place the button block in WP6.1.
    The color when hovered should be red

The classic.scss is managed in the gutenberg repository, but I could not tell if the styles are reflected without overwriting the core stylesheet in my environment.

So perhaps the styles will be reflected when they are merged into the core.
I'm putting it out there in the sense of adding styles to classic.scss in this pull request.

Screenshots or screencast

@shimotmk shimotmk requested a review from ajitbohra as a code owner November 6, 2022 05:44
@codesandbox
Copy link

codesandbox bot commented Nov 6, 2022

CodeSandbox logoCodeSandbox logo  Open in CodeSandbox Web Editor | VS Code | VS Code Insiders

@shimotmk shimotmk closed this Dec 8, 2022
@shimotmk shimotmk deleted the fix/button/hover-color-in-classic-themes branch December 8, 2022 01:11
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

WP6.1 Button block Themes based on block-library css will have a different style
1 participant