In IRC, greggles pointed out that the drop-down to select which view to embed for various things (user list, admin user list, etc) gets pretty unwieldy if the view descriptions are long. It's a theme function that controls this, anyway, but we could make the default theme_signup_settings_view_label() a little more UI-friendly.

CommentFileSizeAuthor
#2 363337_truncate_view_descriptions.2.patch663 bytesdww
#2 363337_truncate_view_descriptions.2.png57.29 KBdww
#1 363337_truncate_view_descriptions.1.patch656 bytesdww

Comments

dww’s picture

dww
we/he/they
commented
Status: Active » Needs review
StatusFileSize
new 363337_truncate_view_descriptions.1.patch656 bytes
dww’s picture

dww
we/he/they
commented
StatusFileSize
new 363337_truncate_view_descriptions.2.png57.29 KB
new 363337_truncate_view_descriptions.2.patch663 bytes

Now with drupal_strlen() instead of just strlen()...

yoroy’s picture

yoroy commented

Now we'll never get people to write short and sweet descriptions! ;-)

But good idea, using just the names of the views is not enough, so trimming the description is a nice gesture.

dww’s picture

dww
we/he/they
commented
Status: Needs review » Fixed

Committed to HEAD and D6--1. Backported to D5--2 and committed there, too.

Status: Fixed » Closed (fixed)

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