System.css sets form-item classed tags to be nowrap. Long option names on narrow layouts push the table outside of the containing div. We can override the system.css setting quite easily to avoid this. Patch follows.

CommentFileSizeAuthor
#2 wrap_option_names.patch342 bytesIsland Usurper
#1 500884_200906240955-0400.patch448 bytessammys

Comments

sammys’s picture

sammys commented
Status: Active » Needs review
StatusFileSize
new 500884_200906240955-0400.patch448 bytes

Patch attached.

Island Usurper’s picture

Island Usurper commented
Status: Needs review » Fixed
StatusFileSize
new wrap_option_names.patch342 bytes

Since theme_uc_object_options_form() adds the "draggable" class to all of the table rows, we can simplify the selector by using it instead of "even" and "odd". I almost decided to put it in the tr.draggable div.form-item selector that's already there, but I figure this should actually be something that applies only to that table.

Committed.

sammys’s picture

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

Reviewed and works. RTBC! Well... this small part of the community anyway!

Island Usurper’s picture

Island Usurper commented
Status: Reviewed & tested by the community » Fixed

Thanks, Sammy. A little late, but thanks. ;)

sammys’s picture

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

Lol... pay attention Sammy!

Island Usurper’s picture

Island Usurper commented
Status: Reviewed & tested by the community » Fixed

Status: Fixed » Closed (fixed)

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