Skip to content

Commit

Permalink
Update InputControl and SelectControl default heights (WordPress#55490)
Browse files Browse the repository at this point in the history
* Update InputControl and SelectControl default heights

* Update changelog

* Changelog

* Update snapshots
  • Loading branch information
jameskoster authored Oct 25, 2023

Verified

This commit was created on GitHub.com and signed with GitHub’s verified signature.
1 parent 4bc5850 commit 39ffb68
Showing 4 changed files with 16 additions and 12 deletions.
4 changes: 4 additions & 0 deletions packages/components/CHANGELOG.md
Original file line number Diff line number Diff line change
@@ -2,6 +2,10 @@

## Unreleased

### Enhancements

- `InputControl`/`SelectControl`: update `height`/`min-height` to `32px` instead of `30px` to align with modern sizing scale ([#55490](https://github.com/WordPress/gutenberg/pull/55490)).

### Bug Fix

- `Autocomplete`: Add `aria-live` announcements for Mac and IOS Voiceover to fix lack of support for `aria-owns` ([#54902](https://github.com/WordPress/gutenberg/pull/54902)).
Original file line number Diff line number Diff line change
@@ -119,8 +119,8 @@ exports[`DimensionControl rendering renders with custom sizes 1`] = `
white-space: nowrap;
text-overflow: ellipsis;
font-size: 16px;
height: 30px;
min-height: 30px;
height: 32px;
min-height: 32px;
padding-top: 0;
padding-bottom: 0;
padding-left: 8px;
@@ -390,8 +390,8 @@ exports[`DimensionControl rendering renders with defaults 1`] = `
white-space: nowrap;
text-overflow: ellipsis;
font-size: 16px;
height: 30px;
min-height: 30px;
height: 32px;
min-height: 32px;
padding-top: 0;
padding-bottom: 0;
padding-left: 8px;
@@ -671,8 +671,8 @@ exports[`DimensionControl rendering renders with icon and custom icon label 1`]
white-space: nowrap;
text-overflow: ellipsis;
font-size: 16px;
height: 30px;
min-height: 30px;
height: 32px;
min-height: 32px;
padding-top: 0;
padding-bottom: 0;
padding-left: 8px;
@@ -964,8 +964,8 @@ exports[`DimensionControl rendering renders with icon and default icon label 1`]
white-space: nowrap;
text-overflow: ellipsis;
font-size: 16px;
height: 30px;
min-height: 30px;
height: 32px;
min-height: 32px;
padding-top: 0;
padding-bottom: 0;
padding-left: 8px;
Original file line number Diff line number Diff line change
@@ -149,9 +149,9 @@ export const getSizeConfig = ( {

if ( ! __next40pxDefaultSize ) {
sizes.default = {
height: 30,
height: 32,
lineHeight: 1,
minHeight: 30,
minHeight: 32,
paddingLeft: space( 2 ),
paddingRight: space( 2 ),
};
Original file line number Diff line number Diff line change
@@ -85,8 +85,8 @@ const sizeStyles = ( {

if ( ! __next40pxDefaultSize ) {
sizes.default = {
height: 30,
minHeight: 30,
height: 32,
minHeight: 32,
paddingTop: 0,
paddingBottom: 0,
};

0 comments on commit 39ffb68

Please sign in to comment.