Changes between Version 1 and Version 2 of Ticket #58443, comment 13
- Timestamp:
- 10/16/2023 12:11:45 PM (16 months ago)
Legend:
- Unmodified
- Added
- Removed
- Modified
-
Ticket #58443, comment 13
v1 v2 1 ~~Using~~ //Switching a specific value to//`inherit` was not the best way to fix custom `line-height`.2 //Correction : When the theme—or block library styles—specify the desired value on a **parent** element, then using `inherit` could be highly appropriate.//1 ~~Using~~ **Switching a specific value to** `inherit` was not the best way to fix custom `line-height`. 2 //Correction//: When the theme—or block library styles—specify the desired value on a **parent** element, then using `inherit` could be highly appropriate. 3 3 4 4 On the front end, [https://github.com/WordPress/twentynineteen/blob/be850d11ee31434ad700db518ff8570f82e65b9c/sass/forms/_buttons.scss#L16 all button-like elements] have had a `line-height` of 1.2 since [https://github.com/WordPress/twentynineteen/blob/be850d11ee31434ad700db518ff8570f82e65b9c/sass/blocks/_blocks.scss#L130 initial commit], and it needs to stay that value when users do not select their own. This ticket did not change the front-end `style.css` and the RTL was reverted, so the current state is still only wrong with a custom `line-height`. … … 6 6 The editor's value of 1.8 did not match the front, but removing the entire line can be better than changing to `inherit`. 7 7 8 ~~Any~~ //Some//of the customizable properties probably could be moved to the `wp-block-button` or `wp-block-buttons` element instead of the link.9 //Correction : The em-based `font-size` is one property that probably should **not** be moved to another element because that would change the size for anyone who has set a value in custom CSS or who is satisfied with the relative difference in size using the block setting.//8 ~~Any~~ **Some** of the customizable properties probably could be moved to the `wp-block-button` or `wp-block-buttons` element instead of the link. 9 //Correction//: The em-based `font-size` is one property that probably should **not** be moved to another element because that would change the size for anyone who has set a value in custom CSS or who is satisfied with the relative difference in size using the block setting. 10 10 11 11 **If** tests are still giving any errors, that might be related to the missing space after the colon in `style-editor.scss`.