See screnshots.
Missing row 'add-new' class, trivial patch.

(+ remove the 'menu-disabled' class, it was used in CCK D6 to get a 'dimmed' effect on some rows, but this class disappeared in Seven theme.

CommentFileSizeAuthor
#1 field_ui_visual_glitch.patch663 bytesyched
Field_ui_add_existing-AFTER.png8.2 KByched
Field_ui_add_existing-BEFORE.png13.03 KByched

Comments

yched’s picture

yched commented
StatusFileSize
new field_ui_visual_glitch.patch663 bytes

Where's ma patch ?

Bojhan’s picture

Bojhan commented
Issue tags: +Accessibility

Lets get an accessibility annotation on this, sure there is a reason for doing this like this.

mgifford’s picture

mgifford
he/him
Primary language English
commented

Isn't this just a CSS issue? The patch just seems to add a class that is relevant for D7 & drop one that was relevant for D6. Might be worth looking for other instances like this.

I don't think the patch has any accessibility advantages. It's just styled consistently now. Right @yched?

Bojhan’s picture

Bojhan commented
Status: Needs review » Reviewed & tested by the community

Alright :) awesome.

yched’s picture

yched commented

yes, 'add-new' is just the class field_ui uses to take care of precisely those two glitches. 'Add new field' row has it, but for some reason it's missing for 'Add existing field'. Nothing new, just a trivial bugfix.

dries’s picture

dries commented
Status: Reviewed & tested by the community » Fixed

Committed to CVS HEAD. Thanks.

Status: Fixed » Closed (fixed)
Issue tags: -Accessibility

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

Issue tags: +Accessibility

Restoring issue tags, see #2125755: System messages removed all issue tags during D7 upgrade.