Problem/Motivation
Theme screenshot alt-text and theme descriptions need improved text (and perhaps standardization) to provide more detailed explanations of the features supported by the theme (especially for non-sighted admins).
Proposed resolution
Several variations have been proposed - #922696: Follow-up: New methods, standards for describing core themes on themes admin page (was Theme screenshots do not have adequate alt text...) - such as:
The default administration theme for Drupal 8 was designed with clean lines, simple blocks, and san-serif font to emphasizes the tools and tasks at hand.
vs
A single-column, monochromatic theme, used for site administration.
Remaining tasks
The issue itself requests more than just improved descriptions:
two things in the .info file
standardized alt text for the screenshot
tagging for the core themes such as uses the color module, responsive, fixed width, 3 column
In addition see
comment #31
"If we're thinking long term, here's my proposal of an ideal classification for the core themes. I see several issues coming out of this:
description: (existing property) Describes the purpose, stylistic characteristics, and design philosophy of a theme, in 2-3 sentences. I think that Jen's proposed text for Seven is the best example of this.
alt text: (new .info property) Alt text for the theme screenshot - describes what the screenshot looks like, as per Everett's original request.
tags: (new .info property) Tags would match up with a new taxonomy vocabulary for themes on Drupal.org. If you clicked on a particular tag, then a lightbox would show up with Drupal.org download links for themes that matched that tag (i.e., like how things work for installing themes in WordPress). This would mean that you could click single-column, monochromatic, recolorable, multi-column, fluid, fixed, etc. and see what other themes had that characteristic.
Preview button: Each theme would also have a preview button that you could click that would be somewhat similar to "Demonstrate block regions" by switching your site into the given theme, with a back link that would take you back to themes administration. This would, I think address Bojhan's concern that we want people to actually try out themes.
Obviously, this is an ambitious plan, but we have a whole release cycle to make it happen. If only 2 parts end up getting implemented though, I would have it be points 1 & 2, since that would be a relatively easy fix and would address the accessibility issue, while at the same time improving the theme descriptions.
User interface changes
(see above)
API changes
n/a
Related Issues
n/a
Original report by Everett Zufelt
On /admin/appearance there is a list of all Core themes. Each theme provides a screen-shot, a name, and a description.
1. Every screen-shot has the alt text "Screen-shot for [theme-name] theme".
2. The theme descriptions are inconsistent in how they describe the themes.
3. None of the theme descriptions are close to being sufficient to inform a blind administrator about how the theme looks or how its regions are laid out.
Comment | File | Size | Author |
---|---|---|---|
#24 | seven-theme-descriptors-2193691-24.patch | 660 bytes | Bojhan |
Comments
Comment #1
LewisNymanComment #2
mgiffordThanks @LewisNyman - that patch changed it to 'A clean and friendly administration theme.'
I'm not sure I'd describe it as friendly. Maybe if it were yellow rather than grey.
Anyways, appreciate it.
Comment #3
LewisNymanI think most of the inspiration for the wording came from the style guide and the interpretation of Drupal's basic principles into Seven's own principles:
The objective of the style guide in summary:
Maybe we aren't fully there in implementation, but they are the objectives.
Comment #4
mgiffordThe wording doesn't need to be perfect for the release of D8. But it should be better.
I do think that too much time has gone into this. It really shouldn't be this difficult to craft a paragraph of text....
Comment #5
holly.ross.drupal CreditAttribution: holly.ross.drupal commentedComment #6
holly.ross.drupal CreditAttribution: holly.ross.drupal commentedUpdated the description per Jen's suggestion in comment 27 as suggested. Added Alt text and tags as well. Changes are as follows:
Comment #8
martin107 CreditAttribution: martin107 commented@holly.ross
by way of support ... here is the relevant extract from the log file ... it is quite well hidden
Hope this helps... I am in IRC if you need a nudge in the right direction.
Comment #9
holly.ross.drupal CreditAttribution: holly.ross.drupal commentedUpdated "alt text" indentifier to read "alttext"
Comment #11
holly.ross.drupal CreditAttribution: holly.ross.drupal commentedUpdated patch to remove line endings for descriptions and change arrange key spaces to underscores.
Comment #13
holly.ross.drupal CreditAttribution: holly.ross.drupal commentedReverted array key spaces, kept line endings as changed above.
Comment #15
holly.ross.drupal CreditAttribution: holly.ross.drupal commentedFinally learned to test locally first and not spam the issue queue. Sorry!
Comment #16
martin107 CreditAttribution: martin107 commentedComment #17
BLadwin CreditAttribution: BLadwin commentedre-rolled Holly's patch to correct the spelling of the 'sans-serif' tag. When this is committed to core, please pass attribution to holly.ross.drupal, bladwin, and tedbow [core sprint mentors at DCATX14]
Comment #19
BLadwin CreditAttribution: BLadwin commentedre-rolled Holly's patch to correct the spelling of the 'sans-serif' tag. When this is committed to core, please pass attribution to holly.ross.drupal, bladwin, and tedbow [core sprint mentors at DCATX14]
EDIT: new line missing from end of file. applied cleanly to local 8.x branch.
Comment #20
BLadwin CreditAttribution: BLadwin commentedneeds review
Comment #21
LewisNymanSorry y'all I spotted a typo.
This could either say " to emphasize the tools" or "that emphasizes the tools"
Nothing else apart from that.
Comment #22
Bojhan CreditAttribution: Bojhan commentedFixed that to "to emphasize the tools".
Thanks Holly! It nicely captures the core focus of the Seven theme :)
Comment #23
dixon_The description field seems to have a weird leading quote mark. Otherwise it looks good.
Comment #24
Bojhan CreditAttribution: Bojhan commentedNice catch.
Comment #25
dixon_Looks good. Should come back green :)
Comment #26
alexpottCommitted 2858a83 and pushed to 8.x. Thanks!
Comment #29
star-szrJust a heads up to anyone interested, a newer issue is proposing to remove the 'tags' data because it's not currently being used: #2471619: Remove tags from all core theme.info.yml files