px/rem order reversed in Twenty Twelve blocks.css

[superpoincare]

In the new blocks.css file for Twenty Twelve, px overrides rem in 2 styles, when it should be reversed:

​https://github.com/WordPress/WordPress/blob/master/wp-content/themes/twentytwelve/css/blocks.css#L100-L109

.wp-block-quote:not(.is-large):not(.is-style-large) {
	border: 0;
	padding: 1.714285714rem;
	padding: 24px;
}

.wp-block-quote {
	padding: 1.714285714rem;
	padding: 24px;
}

[SergeyBiryukov]

Introduced in [43795], other themes appear to be unaffected.

Since we're updating the theme for 5.0.3 in [44381] and [44382] anyway, this seems simple enough to make it in as well.

[SergeyBiryukov]

In 44432:

Twenty Twelve: Correct padding rule precedence for Quote block.

Props superpoincare.
Fixes #45794.

[SergeyBiryukov]

Reopening for merging to the 5.0 branch.

[SergeyBiryukov]

In 44433:

Twenty Twelve: Correct padding rule precedence for Quote block.

Props superpoincare.
Merges [44432] to the 5.0 branch.
Fixes #45794.

[superpoincare]

Maybe the version needs a change here---the date, or is it not important?

​https://github.com/WordPress/WordPress/blob/master/wp-content/themes/twentytwelve/functions.php#L202-L203

// Theme block stylesheet.
	wp_enqueue_style( 'twentytwelve-block-style', get_template_directory_uri() . '/css/blocks.css', array( 'twentytwelve-style' ), '20181230' );

[SergeyBiryukov]

Replying to superpoincare:

Maybe the version needs a change here---the date, or is it not important?

[44434] updated it from 20181018 to 20181230 for 5.0.3.

It could have been updated to 20190107 to reflect the latest change here, but that doesn't seem really necessary, as the value is only used for cache busting.