Skip to content
This repository has been archived by the owner on Feb 23, 2024. It is now read-only.

Disable font-size for the Featured Category block #5681 #5682

Merged
merged 1 commit into from
Jan 31, 2022

Conversation

gigitux
Copy link
Contributor

@gigitux gigitux commented Jan 31, 2022

The implementation of the font size with the Global Styles API has changed the default design of the Featured Category block.

This PR disables the possibility to change the font size with the Global Styles API and restore the old and correct default design.

Fixes #5681

Screenshots

image
Featured Category Block using the TT1 Block Theme with the correct default design

Testing

Manual Testing

How to test the changes in this Pull Request:

Check out this branch

  1. Be sure to activate a block theme and have WordPress 5.9 or WordPress 5.8 with Gutenberg enabled.
  2. Open the editor and add the Feature Category Block.
  3. Check that description and title have different font size and look equal to the attached image.

Disable font-size for the Featured Category block
@gigitux gigitux requested a review from Aljullu January 31, 2022 16:26
@gigitux gigitux self-assigned this Jan 31, 2022
@gigitux gigitux added type: bug The issue/PR concerns a confirmed bug. focus: global styles Issues that involve styles/css/layout structure. status: blocker Used on issues or pulls that block work from being released. labels Jan 31, 2022
@rubikuserbot rubikuserbot requested a review from a team January 31, 2022 16:26
@github-actions
Copy link
Contributor

Size Change: -22 B (0%)

Total Size: 745 kB

Filename Size Change
build/featured-category.js 8.63 kB -17 B (0%)
build/wc-blocks-style-rtl.css 21.8 kB -3 B (0%)
build/wc-blocks-style.css 21.8 kB -2 B (0%)
ℹ️ View Unchanged
Filename Size
build/active-filters-frontend.js 6.21 kB
build/active-filters.js 7.08 kB
build/all-products-frontend.js 18.6 kB
build/all-products.js 35.1 kB
build/all-reviews.js 8.39 kB
build/atomic-block-components/add-to-cart--atomic-block-components/button--atomic-block-components/image---8f355022.js 238 B
build/atomic-block-components/add-to-cart--atomic-block-components/button--atomic-block-components/image---a7e2bb9b.js 2.67 kB
build/atomic-block-components/add-to-cart--atomic-block-components/button.js 1.48 kB
build/atomic-block-components/add-to-cart-frontend.js 7.04 kB
build/atomic-block-components/add-to-cart.js 6.6 kB
build/atomic-block-components/button-frontend.js 1.48 kB
build/atomic-block-components/button.js 854 B
build/atomic-block-components/category-list-frontend.js 458 B
build/atomic-block-components/category-list.js 458 B
build/atomic-block-components/image-frontend.js 1.37 kB
build/atomic-block-components/image.js 1.05 kB
build/atomic-block-components/price-frontend.js 1.74 kB
build/atomic-block-components/price.js 1.69 kB
build/atomic-block-components/rating-frontend.js 703 B
build/atomic-block-components/rating.js 700 B
build/atomic-block-components/sale-badge-frontend.js 624 B
build/atomic-block-components/sale-badge.js 624 B
build/atomic-block-components/sku-frontend.js 386 B
build/atomic-block-components/sku.js 384 B
build/atomic-block-components/stock-indicator--atomic-block-components/summary--atomic-block-components/title.js 452 B
build/atomic-block-components/stock-indicator-frontend.js 926 B
build/atomic-block-components/stock-indicator.js 624 B
build/atomic-block-components/summary-frontend.js 1.23 kB
build/atomic-block-components/summary.js 925 B
build/atomic-block-components/tag-list-frontend.js 459 B
build/atomic-block-components/tag-list.js 458 B
build/atomic-block-components/title-frontend.js 1.19 kB
build/atomic-block-components/title.js 932 B
build/attribute-filter-frontend.js 16.3 kB
build/attribute-filter.js 12.6 kB
build/blocks-checkout.js 17.6 kB
build/cart-blocks/accepted-payment-methods-frontend.js 1.14 kB
build/cart-blocks/checkout-button-frontend.js 1.14 kB
build/cart-blocks/empty-cart-frontend.js 345 B
build/cart-blocks/express-payment-frontend.js 4.73 kB
build/cart-blocks/filled-cart-frontend.js 713 B
build/cart-blocks/items-frontend.js 277 B
build/cart-blocks/line-items-frontend.js 5.26 kB
build/cart-blocks/order-summary-frontend.js 8.62 kB
build/cart-blocks/totals-frontend.js 294 B
build/cart-frontend.js 45.4 kB
build/cart.js 43.8 kB
build/checkout-frontend.js 45.3 kB
build/checkout.js 22.7 kB
build/featured-product.js 9.91 kB
build/handpicked-products.js 7.31 kB
build/legacy-template.js 2.16 kB
build/mini-cart-component-frontend.js 14.2 kB
build/mini-cart-contents.js 3.57 kB
build/mini-cart-frontend.js 1.76 kB
build/mini-cart.js 6.43 kB
build/price-filter-frontend.js 12.6 kB
build/price-filter.js 8.89 kB
build/price-format.js 1.18 kB
build/product-best-sellers.js 7.52 kB
build/product-categories.js 3.43 kB
build/product-category.js 8.67 kB
build/product-new.js 7.98 kB
build/product-on-sale.js 8.35 kB
build/product-search.js 2.47 kB
build/product-tag.js 8.06 kB
build/product-top-rated.js 7.94 kB
build/products-by-attribute.js 8.78 kB
build/reviews-by-category.js 11.9 kB
build/reviews-by-product.js 12.9 kB
build/reviews-frontend.js 7.34 kB
build/single-product-frontend.js 22.2 kB
build/single-product.js 10.5 kB
build/stock-filter-frontend.js 6.8 kB
build/stock-filter.js 6.82 kB
build/vendors--atomic-block-components/add-to-cart--cart-blocks/order-summary-frontend.js 19 kB
build/vendors--atomic-block-components/add-to-cart-frontend.js 7.42 kB
build/vendors--atomic-block-components/price--cart-blocks/line-items--cart-blocks/order-summary-frontend.js 5.71 kB
build/vendors--cart-blocks/line-items-frontend.js 3.33 kB
build/vendors--cart-blocks/order-summary-frontend.js 4.75 kB
build/wc-blocks-data.js 8.84 kB
build/wc-blocks-editor-style-rtl.css 4.7 kB
build/wc-blocks-editor-style.css 4.7 kB
build/wc-blocks-google-analytics.js 1.56 kB
build/wc-blocks-middleware.js 949 B
build/wc-blocks-registry.js 2.7 kB
build/wc-blocks-shared-context.js 1.52 kB
build/wc-blocks-shared-hocs.js 1.14 kB
build/wc-blocks-vendors-style-rtl.css 1.28 kB
build/wc-blocks-vendors-style.css 1.28 kB
build/wc-blocks-vendors.js 65.6 kB
build/wc-blocks.js 2.96 kB
build/wc-payment-method-bacs.js 816 B
build/wc-payment-method-cheque.js 811 B
build/wc-payment-method-cod.js 909 B
build/wc-payment-method-paypal.js 837 B
build/wc-settings.js 2.61 kB

compressed-size-action

@gigitux gigitux added the skip-changelog PRs that you don't want to appear in the changelog. label Jan 31, 2022
@gigitux gigitux added this to the 6.9.0 milestone Jan 31, 2022
Copy link
Contributor

@Aljullu Aljullu left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This is testing well and code looks good. Just left one question below. 👇

@@ -1,5 +1,5 @@
.wp-block-woocommerce-featured-category {
background-color: #000;
background-color: $gray-900;
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Just to confirm, this change is unrelated to this fix, right? You are setting the background color to how it was before #5542, correct?

LGTM but just wanted to make sure I'm understanding it. 😄

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Yes, I noticed the difference and I fixed it! 👍

@gigitux gigitux merged commit cc95e2b into trunk Jan 31, 2022
@gigitux gigitux deleted the bug/5681-disable-font-size-featured-category branch January 31, 2022 17:03
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
focus: global styles Issues that involve styles/css/layout structure. skip-changelog PRs that you don't want to appear in the changelog. status: blocker Used on issues or pulls that block work from being released. type: bug The issue/PR concerns a confirmed bug.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Featured Category: Font-Size Global Styles implementation has changed the default design
2 participants