The usage of bullets in pages like /admin/structure and /admin/configuration cause significant performance issues in terms of reading, they where introduced at first as preceding to any list item. However I am more and more starting to dislike the effect this has on scanning pages like this, because the flow of reading is constantly interrupted by an indent that doesn't allow the eye to look straight down the category heading, also it distracts as its constantly in the periferal vision.

Mark Boulton has a very interesting article on this topic, http://www.markboulton.co.uk/journal/comments/five-simple-steps-to-bette... - how indented bullets tend to disrupt the flow of reading. In our case its even more serious as we basically have 19 bullets on /admin/configuration that are not used for orientation but rather distract.

I have discussed this with Mark and he agreed we needed to remove this.

Support from Acquia helps fund testing for Drupal Acquia logo

Comments

Bojhan’s picture

Issue tags: -Usability
FileSize
120.16 KB
Bojhan’s picture

Issue tags: +Usability, +markup
swentel’s picture

Status: Active » Needs review
FileSize
917 bytes
mfer’s picture

Status: Needs review » Reviewed & tested by the community

While I cannot speak to the UX element the code works as expected all the way back through ie6.

jbrown’s picture

Bojhan’s picture

Not really related at all. We are not dropping the feature.

sun’s picture

Status: Reviewed & tested by the community » Needs work
+++ themes/seven/style.css	28 Sep 2010 09:29:04 -0000
@@ -404,9 +404,9 @@ ul.admin-list.compact {
+  margin: 0.25em 0 0.25em 0em;

Should be shortened into

0.25em 0

Powered by Dreditor.

swentel’s picture

Status: Needs work » Needs review
FileSize
906 bytes

Comment in #7 applied.

Bojhan’s picture

Status: Needs review » Reviewed & tested by the community

Back to RTOTHEBC

Dries’s picture

Using bullets reduces reading performance is a weird statement. Weren't bullets invented to help improve reading?

sun’s picture

Version: 7.x-dev » 8.x-dev

Sounds like D8 material to me.

Bojhan’s picture

Version: 8.x-dev » 7.x-dev

This is definitely not D8 material.

@Dries Yes, they where - however in this case they instead are distracting. They are primarily to be used in paragraphs of text, whenever they are used for menu's there performance is very limited (especially at this mass). I will ask mark boulton to comment, as this is a fairly difficult optimization to understand.

markboulton’s picture

Using bullets in the flow of text increasing reading performance as they are designed to act as visual representations of pauses in speech, or gesticulation. But, stand-alone, especially in this context, they don't add anything to the legibility of the type and should be removed.

One thing to be mindful of, however, is if any of the items in the list drop onto a second line. You need to make sure the line-height is such that the second line does not appear like another item in the list.

Jeff Burnz’s picture

Status: Reviewed & tested by the community » Needs work

I agree the bullets are distracting in this context, however we need to mindful that we need to deliver D7 - therefore its difficult to endorse UI changes unless they are at least major bugs. This seems more of a design tweak.

Also we need to account for #13 "if any of the items in the list drop onto a second line...", we can achieve this by setting line-height to 1 (Unitless), change the LI margin to 0 0 0.75em; and add some margin-top to the UL (maybe 8px).

While it might appear items dropping to a second line is unlikely, they certainly can, in the case where the user may increase the font-size to a very large size and use a low resolution screen (800x600 for example) in which case Case and hover style are the only indicators.

Bojhan’s picture

@jeff Can you supply a patch?

Jeff Burnz’s picture

Status: Needs work » Needs review
FileSize
708 bytes

Updated patch, to test set the browser to 800x600 and push the text size up as far as you can (you might want to hide or disable toolbar when doing this).

Note that this patch will also remove bullets from admin/structure list, I am not sure if that is wanted also?

Cliff’s picture

With all due respect for Mark Boulton — and he is right, in this case it's easier to skim the titles without bullets — this is fine tuning. If it's easy, do it. But if it's taking any resources at all away from getting D7 out the door, save it for the next release. Aren't there critical issues to fix?

Cliff’s picture

In fact, now that I've read @webchick's comment here, I wonder: Will this affect screenshots already produced for books in the works? Maybe we do need to hold off on this issue for a bit. Think it over.

Jeff Burnz’s picture

Well, if it doesnt make it to 7.x-1.x then it could slip in later on. Either way I am not fussed, the bullets I am used to, but that's only me, I really do think its better without them, however it does render the admin/structure page even more naked that it was before, which was pretty naked and IMO weirdly inconsistent with other admin lists.

moshe weitzman’s picture

jeez guys. this is an improvement. lets commit it, instead of wringing hands over a screenshot somewhere that might have bullets in it. folks can re-take screenshots. we have a live audience of thousand and thousands that merit consideration too.

Bojhan’s picture

Status: Needs review » Reviewed & tested by the community

From structure is good too, I think - especially if you start having more items, the bullets don't work as well.

@Cliff Lets not get to over worried on that, the people in this issue spend enough time on criticals :) There is no reason to delay it on books, probably only a very small amount show the compact version anyway. Also lets not demote all UX changes to polish - unless we are adding a gloss effect :). Because this issue is similar to performance bugs that slow down Drupal, just in a different area.

Dries’s picture

Status: Reviewed & tested by the community » Fixed

Alright, committed to CVS HEAD.

Status: Fixed » Closed (fixed)
Issue tags: -Usability, -markup

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