Skip to content

Admin reskin: Remove line-height from input elements#11383

Closed
t-hamano wants to merge 5 commits intoWordPress:trunkfrom
t-hamano:64763-remove-line-height
Closed

Admin reskin: Remove line-height from input elements#11383
t-hamano wants to merge 5 commits intoWordPress:trunkfrom
t-hamano:64763-remove-line-height

Conversation

@t-hamano
Copy link
Copy Markdown
Contributor

@t-hamano t-hamano commented Mar 28, 2026

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

Use of AI Tools

AI assistance: Yes
Tool(s): Claude

@github-actions
Copy link
Copy Markdown

github-actions bot commented Mar 28, 2026

The following accounts have interacted with this PR and/or linked issues. I will continue to update these lists as activity occurs. You can also manually ask me to refresh this list by adding the props-bot label.

Core Committers: Use this line as a base for the props when committing in SVN:

Props wildworks, audrasjb.

To understand the WordPress project's expectations around crediting contributors, please review the Contributor Attribution page in the Core Handbook.

@t-hamano t-hamano marked this pull request as draft March 28, 2026 07:18
@github-actions
Copy link
Copy Markdown

Test using WordPress Playground

The changes in this pull request can previewed and tested using a WordPress Playground instance.

WordPress Playground is an experimental project that creates a full WordPress instance entirely within the browser.

Some things to be aware of

  • All changes will be lost when closing a tab with a Playground instance.
  • All changes will be lost when refreshing the page.
  • A fresh instance is created each time the link below is clicked.
  • Every time this pull request is updated, a new ZIP file containing all changes is created. If changes are not reflected in the Playground instance,
    it's possible that the most recent build failed, or has not completed. Check the list of workflow runs to be sure.

For more details about these limitations and more, check out the Limitations page in the WordPress Playground documentation.

Test this pull request with WordPress Playground.

@t-hamano t-hamano force-pushed the 64763-remove-line-height branch from a7f2c63 to 24c630f Compare March 28, 2026 07:29
Copy link
Copy Markdown
Contributor Author

@t-hamano t-hamano left a comment

Choose a reason for hiding this comment

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

I will explain which UI each change corresponds to. Also, note that select elements are out of scope in this PR, unless input and select are included in the same selector.


.wp-filter .search-form input[type="search"] {
min-height: 32px;
line-height: 2.14285714; /* 30px for 32px height with 14px font */
Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

Plugins > Add Plugin > "Search Plugins" field

Image

}

.themes-filter-bar .wp-filter-search {
line-height: 2.14285714; /* 30px for 32px compact input */
Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

Activate classic theme > Appearance > Customize > Click "Change" button > "Search Plugins" field

Image

width: 40%;
margin: 0;
min-height: 32px;
line-height: 2.14285714;
Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

Dashboard > WordPress Events and News > Click "Select location"

Image

input[type="week"] {
padding: 0 12px;
/* inherits font size 14px */
line-height: 2.71428571; /* 38px for 40px min-height */
Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

This is the basic style for the input element.

-webkit-appearance: none;
padding: 0 12px;
min-height: 40px;
line-height: 2.5; /* 40px for 16px font */
Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

This is the basic style for mobile viewports.

p.search-box input[type="search"],
p.search-box input[type="text"] {
min-height: 32px;
line-height: 2.14285714; /* 30px for 32px height with 14px font */
Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

Search posts

Image

font-size: 13px;
text-align: center;
min-height: 32px;
line-height: 2.30769231; /* 30px for 32px height with 13px font */
Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

Pagination input field

Image


.inline-edit-row select,
.inline-edit-row input:where(:not([type=checkbox],[type=radio],[type=submit],[type=button])) {
line-height: 2.14285714;
Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

Quick edit form

Image


.media-frame.mode-grid .media-toolbar input[type="search"] {
min-height: 32px;
line-height: 2.14285714; /* 30px for 32px height with 14px font */
Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

Media > Switch to Grid View > Search media field

Image

@t-hamano t-hamano changed the title 64763 remove line height Admin reskin: Remove line-height from input elements Mar 28, 2026
@t-hamano t-hamano marked this pull request as ready for review March 28, 2026 07:35
@audrasjb
Copy link
Copy Markdown
Contributor

Thanks for the PR! I spotted two more occurrences.

General settings:
Capture d’écran 2026-03-28 à 09 49 42

Colorpicker:
Capture d’écran 2026-03-28 à 09 50 00

Otherwise the rest looks good to me!

@t-hamano
Copy link
Copy Markdown
Contributor Author

Nice catch!

Colorpicker:

Addressed in ea7f3d0

General settings:

Addressed in 935dd5e

Simply removing the line-height changes the input field's height from 26px to 24px. This value is considered small in our design system, but the field feels too tiny:

remove-line-height

Therefore, I removed the line-height and increased the height from 26px to 32px. What do you think?

remove-line-height-32px

@audrasjb
Copy link
Copy Markdown
Contributor

Looking great to me, thanks!

@github-actions
Copy link
Copy Markdown

A commit was made that fixes the Trac ticket referenced in the description of this pull request.

SVN changeset: 62171
GitHub commit: 19d4be9

This PR will be closed, but please confirm the accuracy of this and reopen if there is more work to be done.

@github-actions github-actions bot closed this Mar 28, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants