Opened 5 months ago
Closed 5 months ago
#65233 closed defect (bug) (fixed)
Taxonomy: Delete button is misaligned
| Reported by: | wildworks | Owned by: | wildworks |
|---|---|---|---|
| Priority: | normal | Milestone: | 7.0 |
| Component: | General | Version: | |
| Severity: | normal | Keywords: | good-first-bug has-patch dev-reviewed |
| Cc: | Focuses: | ui, administration |
Description
This is likely due to the button height being standardized to 40px in version 7.0. I propose we take this opportunity to refactor the two buttons using a flex layout. The current approach of using line-height for alignment is unstable.
Steps to reproduce
- Access Posts > Tags
- Create a tag.
- Edit the tag.
- Check the Alignment of the delete button.
Attachments (1)
Change History (17)
This ticket was mentioned in PR #11823 on WordPress/wordpress-develop by @tusharbharti.
5 months ago
#1
- Keywords has-patch added
Trac ticket: https://core.trac.wordpress.org/ticket/65233
This PR fixes the alignment of delete button after the standardization of 40px in buttons.
More details are on the issue but a before and after is below:
| Before | After |
|---|---|
## Use of AI Tools
AI assistance: Yes
Tool(s): Github Copilot
Model: Claude Haiku 4.5
Used for: finding where the css for delete link and tags are present, the css addition and removal is done and reviewed by me
@tusharbharti commented on PR #11823:
5 months ago
#3
@tusharbharti commented on PR #11823:
5 months ago
#4
Hi, and yeah, I also think migrating to a full flex layout seems like the right choice since many files are now using display: flex, based on the grep.
I wasn’t sure how much of this was allowed in this case, so I added flex to the .edit-tag-actions.
I will check whether fully migrating to a flex layout breaks anything before I commit this.
@tusharbharti commented on PR #11823:
5 months ago
#5
and just to confirm, I also need to update the -rtl.css files?
cc: @t-hamano
@tusharbharti commented on PR #11823:
5 months ago
#6
@tusharbharti commented on PR #11823:
5 months ago
#8
and just to confirm, I also need to update the -rtl.css files?
Found the grunt cmd for this 👍🏻
#10
@
5 months ago
- Keywords dev-feedback added; commit removed
- Resolution fixed
- Status closed → reopened
#11
@
5 months ago
Hi @wildworks, Just to confirm, should I open a backport PR for 7.0, or will the maintainers handle it?
#12
@
5 months ago
@tusharbharti, you don't need to create a backport PR. Another committer will approve the commit to the trunk branch and then commit it to 7.0.
#14
@
5 months ago
Tested the fix from [62357] on WordPress 7.0 RC3. The Delete link aligns correctly with the Update button after applying the patch. The flexbox approach is clean and low-risk. +1 for backporting to the 7.0 branch.
![(please configure the [header_logo] section in trac.ini)](/chrome/site/your_project_logo.png)

Comparison of the delete button's position between 6.9 and 7.0