When there is also a "Latest version" the eye does not immediately distinguish that it is not recommended. In this case it might be better to bold "Recommended."

Comments

merlinofchaos’s picture

Status: Active » Needs review
StatusFileSize
new2.31 KB

This simple patch adds an extra class if there is a recommended and a latest, and that class gets the bolding. Recommended version doesn't fit in 13em if bolded, so width expands to 14em to accomodate.

merlinofchaos’s picture

Status: Needs review » Fixed

dww wanted this to also highlight if a 'security update' is listed and not used. That was an easy enough change, and committed.

merlinofchaos’s picture

StatusFileSize
new1.81 KB

Here's a slight addendum based upon dww's comments via IRC. It highlights the security update IF there isn't a recommended version being listed next to it.

nancydru’s picture

Thanks

dww’s picture

Assigned: Unassigned » dww
Status: Fixed » Needs review
StatusFileSize
new3.15 KB

Since #3 introduces an else clause to that funky special case that used to be written with inverted logic, it's actually much clearer to re-write the if. Attached patch does that, and also includes the conditional bold if there's both "Recommended" and "Also available".

dww’s picture

Status: Needs review » Fixed

merlin gave his blessings for #5 in IRC, so I committed to HEAD.

dww’s picture

Status: Fixed » Active
StatusFileSize
new96.11 KB

FYI -- there was a bug where we weren't bold'ing Recommended if we were just displaying that and a dev snapshot. Fixed: http://drupal.org/cvs?commit=71470

Also, in IRC, UnConeD and hunmonk suggested another visual enhancement to make the recommended line stick out and be more visually clear, even if there's only that 1 line. A subtle zebra stripe sort of thing with a slighly lighter shade of the color. Screenshot here from garland.

dww’s picture

StatusFileSize
new106.22 KB

Same, but screenshot with marvin enabled.

dww’s picture

Status: Active » Needs review
StatusFileSize
new753 bytes

And now, the trivial patch (just touches update_status.css).

dww’s picture

Status: Needs review » Fixed

Committed to HEAD (http://drupal.org/cvs?commit=71473). We can always tweak later if we must, but this is an improvement, IMHO.

Anonymous’s picture

Status: Fixed » Closed (fixed)