Opened 12 months ago
Closed 8 months ago
#63369 closed defect (bug) (fixed)
Unnecessary vertical space appears on Tags screen when no tags exist
| Reported by: |
|
Owned by: |
|
|---|---|---|---|
| Milestone: | 6.9 | Priority: | normal |
| Severity: | normal | Version: | 3.0 |
| Component: | Administration | Keywords: | has-patch has-test-info dev-feedback |
| Focuses: | ui | Cc: |
Description
On the Tags admin page (wp-admin/edit-tags.php?taxonomy=post_tag), when no tags exist, a container element intended for bulk actions and pagination is still rendered. Although its child elements are hidden via display: none, the container itself still occupies vertical space.
This results in a visible but functionally empty gap between the tags table and the “Tags can be selectively converted to categories using the tag to category converter.” message, creating a layout inconsistency and an impression that something is missing in that space.
Steps to Reproduce:
- Navigate to Posts > Tags in the admin dashboard.
- Ensure no tags exist.
- Observe the spacing between the table and the “No tags found.” paragraph.
Environment
- WordPress: 6.9-alpha-60093-src
- PHP: 8.2.28
- Server: nginx/1.27.5
- Database: mysqli (Server: 8.4.5 / Client: mysqlnd 8.2.28)
- Browser: Chrome 135.0.0.0
- OS: macOS
- Theme: Twenty Twenty-Five 1.2
- MU Plugins: None activated
- Plugins: Test Reports 1.2.0
Screenshot
Attachments (2)
Change History (22)
This ticket was mentioned in PR #8757 on WordPress/wordpress-develop by @sainathpoojary.
12 months ago
#1
- Keywords has-patch added
@
12 months ago
We remove the bottom table navbar section directly within the PHP script instead of simply hiding it with CSS.
#2
@
12 months ago
@sainathpoojary I've attached the patch. Please review it and apply it to your PR.
#3
@
12 months ago
Thanks for the patch @dilipbheda, I’ve reviewed it and it looks great. It handles the issue more cleanly by skipping the bottom tablenav at the DOM level, which avoids relying on CSS to hide unused markup. I’ll apply it to my PR and update the description accordingly.
Appreciate the contribution!
@sainathpoojary commented on PR #8757:
12 months ago
#5
Agreed @himanshupathak95, I was able to reproduce the issue as well. I’ll open a separate ticket for it.
#6
@
12 months ago
Test Report
Description
This report validates the patch works as expected.
Patch tested: https://core.trac.wordpress.org/attachment/ticket/63369/63369.patch
Environment
- WordPress: 6.9-alpha-60093-src
- PHP: 8.2.22
- Server: nginx/1.27.0
- Database: mysqli (Server: 8.0.39 / Client: mysqlnd 8.2.22)
- Browser: Chrome 135.0.0.0
- OS: Linux
- Theme: Twenty Ten 4.4
- MU Plugins: None activated
- Plugins:
- Test Reports 1.2.0
Actual Results
- ✅ Issue resolved with patch.
#7
@
12 months ago
- Focuses css removed
- Keywords dev-feedback has-testing-info added
- Version trunk deleted
@SirLouen commented on PR #8757:
10 months ago
#10
Agreed @himanshupathak95, I was able to reproduce the issue as well. I’ll open a separate ticket for it.
Which is the other ticket you have opened?
@sainathpoojary commented on PR #8757:
10 months ago
#11
Hey @SirLouen, The other ticket I’ve opened is: #63547
@SirLouen commented on PR #8757:
10 months ago
#12
Hey @SirLouen, The other ticket I’ve opened is: #63547
@SainathPoojary I'm not sure that this ticket is related with this issue. And you are not participating there. Double check it please.
@sainathpoojary commented on PR #8757:
10 months ago
#14
Sorry for the inconvenience caused, @SirLouen. I’ve updated the comment to point to the correct ticket
This ticket was mentioned in Slack in #core by sirlouen. View the logs.
10 months ago
#16
@
10 months ago
- Keywords dev-feedback added
- Severity changed from normal to trivial
6.9.0 Milestone proposed during today's <bug-scrub>
More information
Hides the bottom tablenav container on taxonomy term list screens when it contains a hidden .tablenav-pages.no-pages element. This prevents unnecessary vertical spacing when no tags exist and improves layout consistency in the admin UI.
Before:
After:
Trac ticket: #63369