The edit and delete links under the operation column in admin content page (admin/content) looks closer for me. It will be good if there is enough space between them like in garland theme.

CommentFileSizeAuthor
#2 644208-operation-links-padding-in-seven-D7_2.patch569 bytesmrfelton
#1 d7-operations-padding-in-seven.png4.27 KBmrfelton
#1 644208-operation-links-padding-in-seven-D7_1.patch567 bytesmrfelton
seven_operation.png2.1 KBSivaji_Ganesh_Jojodae
garland_operation.png2.15 KBSivaji_Ganesh_Jojodae
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

mrfelton’s picture

mrfelton CreditAttribution: mrfelton commented
Status: Active » Needs review
FileSize
644208-operation-links-padding-in-seven-D7_1.patch567 bytes
d7-operations-padding-in-seven.png4.27 KB

Attached patch gives 1em padding on the right - same as in Garland.

mrfelton’s picture

mrfelton CreditAttribution: mrfelton commented
FileSize
644208-operation-links-padding-in-seven-D7_2.patch569 bytes

updated patch with proper spacing and a semicolon.

Sivaji_Ganesh_Jojodae’s picture

Sivaji_Ganesh_Jojodae CreditAttribution: Sivaji_Ganesh_Jojodae commented

Patch looks good for me.

xmacinfo’s picture

xmacinfo
he/him
Primary language French
Location Canada
CreditAttribution: xmacinfo commented
Status: Needs review » Reviewed & tested by the community

Looks good to me too.

Dries’s picture

Dries CreditAttribution: Dries commented
Status: Reviewed & tested by the community » Fixed

Committed to CVS HEAD. Thanks.

Status: Fixed » Closed (fixed)

Automatically closed -- issue fixed for 2 weeks with no activity.