Congrats on being the first contrib maintainer to WIN a FREE usability review during the inaugural UX open hour. Notes:

- Not sure on the vertical tabs. Consider alternative groupings using fieldsets,
- Which settings could live as contextual links? On page-level contextual links maybe?
…

A more concrete suggestion I had was to show the theme's thumbnail on individual settings pages.

CommentFileSizeAuthor
#3 themesettingsextras2.jpg59.89 KByoroy
#1 themesettingsextras2.jpg37.36 KByoroy
#1 themesettingsextras.jpg54.48 KByoroy

Comments

yoroy’s picture

yoroy commented
StatusFileSize
new themesettingsextras.jpg54.48 KB
new themesettingsextras2.jpg37.36 KB
yoroy’s picture

yoroy commented
Issue tags: +Usability

yoroy’s picture

yoroy commented
StatusFileSize
new themesettingsextras2.jpg59.89 KB

andrewmacpherson’s picture

andrewmacpherson commented

Thanks Yoroy!

The field thumb + description is something that should be easy to code.
I had a quick look at the code for the main themes list page, it would just be a re-use of some of that.

Alternative groupings: There was a suggestion (from Bojhan?) that the menu display toggles could be moved to their own fieldset. At the moment that would only be two menus, but themes might also be able to declare support for Navigation and User menus (deserves separate issue).

One of the things I like about Drupal's vertical tabs is the summary text they can offer. So while they hide the settings widgets, they don't have to hide the actual settings. (Assuming the summary text is well-written of course.)

One of the reasons I wanted to use vertical tabs is to reduce the length of the form, so less scrolling is needed. Collapsed fieldsets would achieve that too, but since these don't provide summary text the information would be completely hidden.

andrewmacpherson’s picture

andrewmacpherson commented

What if collapsed fieldsets could also have a summary? Either a static legend or a something dynamic based on the content.
That's beyond themesettings though.

andrewmacpherson’s picture

andrewmacpherson commented

Bonus crazy idea: themes which support color.module could have a re-colourable screenshot!

yoroy’s picture

yoroy commented

Yep, vertical tabs having summaries is an advantage. Collapsed fieldsets with summaries would be #190946: Replace collapsible-fieldsets with Summaries and indeed something core itself should offer.

andrewmacpherson’s picture

andrewmacpherson commented
Status: Active » Closed (fixed)

marking closed, tidy ui issue queue