The list of operations in an admin table has a big gap to its left.

Seen at, for example, admin/content.

This is a regression from 6.

CommentFileSizeAuthor
#2 687554_before_patch.png2.74 KBsivaji_ganesh_jojodae
#2 687554_after_patch.png2.74 KBsivaji_ganesh_jojodae
#2 687554_admin_table_padding_2.patch702 bytessivaji_ganesh_jojodae
garland admin ops gap.png10.1 KBjoachim

Comments

joachim’s picture

joachim commented

Only in Garland, btw.

sivaji_ganesh_jojodae’s picture

sivaji_ganesh_jojodae commented
StatusFileSize
new 687554_admin_table_padding_2.patch702 bytes
new 687554_after_patch.png2.74 KB
new 687554_before_patch.png2.74 KB

I can reproduce the same with recent build. Attached patch will fix this bug.

sivaji_ganesh_jojodae’s picture

sivaji_ganesh_jojodae commented
Status: Active » Needs review
joachim’s picture

joachim commented

Patch works great for me.

Might be an idea to get a themer familiar with Garland to check that special-casing node-admin-content is the best way to do this.

thedavidmeister’s picture

thedavidmeister commented
Issue summary: View changes
Status: Needs review » Needs work

It's pretty much guaranteed not to be the right approach to use an ID to special case this.

There's only two possibilities here:

1. This padding exists in multiple places erroneously and therefore we should use a looser CSS selector for the fix so that it applies globally

2. This padding only exists on the #node-admin-content ul and therefore the rule introducing that padding should be removed, not overridden with a new rule

Status: Needs work » Closed (outdated)

Automatically closed because Drupal 7 security and bugfix support has ended as of 5 January 2025. If the issue verifiably applies to later versions, please reopen with details and update the version.