Twenty Twenty-One: Consider removing `.woocommerce` class from theme styles

[sabernhardt]

The selectors for Twenty Twenty-One's default max-width have used the .woocommerce class since the theme's ​initial commit.

Current:

.entry-content > *:not(.alignwide):not(.alignfull):not(.alignleft):not(.alignright):not(.wp-block-separator):not(.woocommerce),
*[class*="inner-container"] > *:not(.entry-content):not(.alignwide):not(.alignfull):not(.alignleft):not(.alignright):not(.wp-block-separator):not(.woocommerce) {

@davidbaumwald stated in ticket:56337#comment:11:

I don't think Core should have that code as it would need to be supported forever. This gets worse as WC evolves and changes its code which then requires more adjustments by Core.

[audrasjb]

Moving for 6.1 consideration.

[tahmidulkarim]

This CSS style is reflected in the Cart and Checkout page. There is already a separate style for the twenty-twenty-one theme set from Woocommerce which overrides the current .entry-content style, so removing :not(.woocommerce) from the selectors doesn't have any effects on the existing styles. Here is a screenshot - ​https://prnt.sc/6IvB9C-zBanA

[mukesh27]

Hi there!

56366.diff​ works fine in WC pages. Thanks for the patch.

cc. @audrasjb for final review.

[audrasjb]

Looks good to me, thanks for the ping.
Self assigning for final testing and commit.

[audrasjb]

I spotted a few more occurrences.

Trac ticket: https://core.trac.wordpress.org/ticket/56366

[audrasjb]

Ready to go.

[audrasjb]

In 54170:

Twenty Twenty-One: Remove WooCommerce related CSS selectors.

The selectors for Twenty Twenty-One's default max-width have used the .woocommerce class since the theme's ​initial commit. This changeset removes all WooCommerce CSS selector from the theme, as they would need to be supported forever. The change was tested with WooCommerce installed and no issues were raised.

Props sabernhardt, audrasjb, davidbaumwald, tahmidulkarim, mukesh27.
Fixes #56366.

Committed in https://core.trac.wordpress.org/changeset/54170