Last modified: 2014-08-19 00:25:31 UTC

Wikimedia Bugzilla is closed!

Wikimedia migrated from Bugzilla to Phabricator. Bug reports are handled in Wikimedia Phabricator.
This static website is read-only and for historical purposes. It is not possible to log in and except for displaying bug reports and their history, links might be broken. See T69005, the corresponding Phabricator task for complete and up-to-date bug report information.
Bug 67005 - Image caption text color is too light
Image caption text color is too light
Status: RESOLVED FIXED
Product: MobileFrontend
Classification: Unclassified
General/Unknown (Other open bugs)
unspecified
All All
: Unprioritized normal
: ---
Assigned To: Nobody - You can work on this!
:
Depends on:
Blocks:
  Show dependency treegraph
 
Reported: 2014-06-23 22:29 UTC by Quiddity
Modified: 2014-08-19 00:25 UTC (History)
9 users (show)

See Also:
Web browser: ---
Mobile Platform: ---
Assignee Huggle Beta Tester: ---


Attachments
Quickcolor text (3.06 MB, image/png)
2014-06-24 20:51 UTC, Jared Zimmerman (WMF)
Details
000/444 (184.19 KB, image/png)
2014-06-24 23:42 UTC, Jared Zimmerman (WMF)
Details
screenshot1 - 000/333 (564.58 KB, image/png)
2014-07-03 00:32 UTC, Quiddity
Details
screenshot2 - 000/444 (565.34 KB, image/png)
2014-07-03 00:33 UTC, Quiddity
Details
screenshot3 - 222/555 (565.08 KB, image/png)
2014-07-03 00:33 UTC, Quiddity
Details

Description Quiddity 2014-06-23 22:29:36 UTC
The thumbnail caption-text in Mobile is currently colored #777. This is quite hard to read, and will be inconsistent with planned Desktop changes. Please change it to #333.

Context: The Design Team is planning a change to Desktop thumbnail-styling, that includes various border changes (which will make it consistent with Mobile), but also changing the text styling for captions. The gerrit patch includes this file, which is where the requested #333 originates.
https://gerrit.wikimedia.org/r/#/c/133301/5/skins/vector/components/thumbnails.less

Thanks.
Comment 1 Bingle 2014-06-23 22:30:17 UTC
Prioritization and scheduling of this bug is tracked on Trello card https://trello.com/c/4h2hdeNr
Comment 2 Jared Zimmerman (WMF) 2014-06-23 22:55:07 UTC
consistency! wooooo!
Comment 3 Jon 2014-06-23 23:04:26 UTC
Greys/grays have always been a mess in mobile. We created 5 LESS variables to keep the shades of grey constrained to 5. Would be great to upstream these to core, get core using them and use those.
Comment 4 Jon 2014-06-23 23:04:39 UTC
Even better if we can use the thumbnail styling from core...
Comment 5 Jared Zimmerman (WMF) 2014-06-23 23:06:26 UTC
If body copy is Grey 3 perhaps we should use Grey 5 for captions.
Comment 6 Ryan Kaldari 2014-06-23 23:20:11 UTC
See https://gerrit.wikimedia.org/r/#/c/117105 which is related.
Comment 7 Jared Zimmerman (WMF) 2014-06-23 23:21:41 UTC
@ryan does this include all the colors from https://trello.com/c/IRqbu8p4/15-color-swatches or only the grey values?
Comment 8 Jared Zimmerman (WMF) 2014-06-23 23:49:11 UTC
We're targeting a rerelease of the image styling consistency patch (linked above by ryan) for July 10th, it would be great to get these both using LESS Grey 5 at the same time.
Comment 9 Ryan Kaldari 2014-06-23 23:54:17 UTC
@jared: I'm pretty sure it includes all the new color values, except for a few of the grey mappings like "shadow", "neutral", etc. Should I add those as well?
Comment 10 Jared Zimmerman (WMF) 2014-06-23 23:55:57 UTC
@Ryan,

I think we should include all of the values in the trello card. If we have access to all of them we're much less (ha!) likely to need to be calling custom colors in the future.
Comment 11 Quiddity 2014-06-24 02:59:45 UTC
(In reply to Ryan Kaldari from comment #6)
> See https://gerrit.wikimedia.org/r/#/c/117105 which is related.

Please do Not use #555 for the caption text.
On Mobile or Desktop. 

And please do Not change the color of the Desktop body text, from #252525 to #333, (which line #54 at
https://gerrit.wikimedia.org/r/#/c/117105/7/resources/src/mediawiki.ui/settings/colors.less
seems to do) as the grey introduced with the Typography Refresh has been objected to, and has very little supporting evidence.
https://www.mediawiki.org/wiki/Typography_refresh#Why_did_we_change_the_body_text_color.3F
https://www.mediawiki.org/wiki/Talk:Typography_refresh/Archive_3#Gray_text_is_a_strain_to_the_eye
https://www.mediawiki.org/wiki/Talk:Typography_refresh/Archive_2#You_are_killing_me_eyes.2C_kind_sirs

Facebook uses #141823 for content.
Google's Knowledge Graph uses #000 for title text and #222 for blurb text.
Google docs uses #000 for body text.

Dark grey text looks Fuzzy to many people.

-------
If we want more contrast between body and caption, then darken the body text.
Comment 12 Jared Zimmerman (WMF) 2014-06-24 20:51:21 UTC
Created attachment 15719 [details]
Quickcolor text
Comment 13 Jared Zimmerman (WMF) 2014-06-24 20:51:53 UTC
@Quiddity, I uploaded a quick color test, easier to visualize this way.
Comment 14 Ryan Kaldari 2014-06-24 21:26:12 UTC
@Quiddity: I changed the text color to #222222 since that is closer to the original Typography Refresh value (#252525). Let me know if that is acceptable.
Comment 15 Jared Zimmerman (WMF) 2014-06-24 21:50:37 UTC
#222222 isn't a LESS value… but it could be one #222 body "Grey 2" and #555 caption "Grey 5" seems to have high contrast and be reasonably differentiated
Comment 16 Ryan Kaldari 2014-06-24 21:51:37 UTC
Sorry, I meant to say I changed it to @colorGray2 (which is #222222).
Comment 17 Quiddity 2014-06-24 22:44:03 UTC
(In reply to Ryan Kaldari from comment #14)
> @Quiddity: I changed the text color to #222222 since that is closer to the
> original Typography Refresh value (#252525). Let me know if that is
> acceptable.

That is great. Much thanks.


(In reply to Jared Zimmerman (WMF) from comment #13)
> @Quiddity, I uploaded a quick color test, easier to visualize this way.

Thank you! Mockups are much easier to understand and comment on, than pure code. :)

Much like the "type-size selector" that NYTimes uses (and I believe we're planning on implementing eventually) - https://i.imgur.com/7U3yAoF.png - it would be lovely to have 2 or 3 options for color-changes. This would enable us to use many more of the intriguing ideas in https://en.wikipedia.org/wiki/Wikipedia:Unsolicited_redesigns#Unsolicited_redesigns_of_Wikipedia and past community discussions, without upsetting anyone by altering the global defaults in ways that will please some but dismay others. But that's something for the future.  

For now:

* #777 is not even WCAG AA compliant, and should not be used anywhere, unless the text is huge. (cf. http://snook.ca/technical/colour_contrast/colour.html )

* #222 (body) with #555 (caption) is passable, but will result in negative reactions from large numbers of editors and readers (and positive reactions from a few).

* #000 or #111 (body) with #333 (caption), would be my recommendation, for a happy release of https://gerrit.wikimedia.org/r/#/c/133301/ - That would achieve the subtle contrast that is wanted, but without making the text harder to read for people with accessibility issues (bad screens, bright backlighting, old eyes, etc). It would also improve the look of long captions with links, without having to create custom styles for links-within-captions. eg. https://en.m.wikipedia.org/wiki/Kronan_%28ship%29#Sinking
Comment 18 Jared Zimmerman (WMF) 2014-06-24 22:48:05 UTC
In my opinion the 000/333 combo is almost indistinguishable and doesn't provide enough differentiation between body and caption but 222/555 seems to.
Comment 19 Juliusz Gonera 2014-06-24 23:28:17 UTC
Maybe #444? ;)
Comment 20 Quiddity 2014-06-24 23:34:27 UTC
(In reply to Juliusz Gonera from comment #19)
> Maybe #444? ;)

#000/#444 would be a good thing to test. :)
Comment 21 Jared Zimmerman (WMF) 2014-06-24 23:41:53 UTC
000/444 r3vrod451puj67w/Screenshot%202014-06-24%2016.41.18.png">https://www.dropbox.com/s/r3vrod451puj67w/Screenshot%202014-06-24%2016.41.18.png 

it looks ok in swatches, but in an actual text sample, it hardly looks different
Comment 22 Jared Zimmerman (WMF) 2014-06-24 23:42:52 UTC
Created attachment 15720 [details]
000/444

pasted url became garbled, uploading instead
Comment 23 Elitre 2014-06-25 13:05:44 UTC
I can see the differentiation in this last mockup; I would /not/ recommend a lighter grey for the caption.
Comment 24 Quiddity 2014-07-03 00:32:54 UTC
Created attachment 15822 [details]
screenshot1 - 000/333
Comment 25 Quiddity 2014-07-03 00:33:24 UTC
Created attachment 15823 [details]
screenshot2 - 000/444
Comment 26 Quiddity 2014-07-03 00:33:53 UTC
Created attachment 15824 [details]
screenshot3 - 222/555
Comment 27 Quiddity 2014-07-03 00:39:40 UTC
I've attached 3 screenshots, of what I see.
I would prefer 000/444, and I think it looks good/reasonable. 
Especially so, for people who have thin fonts by default (eg me on Linux, with DejaVu Sans), or find the #222 to be 'fuzzy'.
However, I gather (from discussion during meetings) that is not an option at this time. 
I look hopefully forward to the future option-widgets, that will let each user tweak their own experience more easily than user.css  
In the meantime, I guess we're going forward with 222/555.
Comment 28 Jared Zimmerman (WMF) 2014-07-03 01:40:29 UTC
Grey 2 body  / Grey 5 caption seems to be acceptable, Quiddity, I'm looking forward to the user controllable text styling as well. but in the mean time, i think "high contrast mode" falls into the bucket of, let's not spend our time and energy recreating OS and browser functions on the site. 

The ability to boost contrast is available in the OS to users with extreme visual impairments, an the proposed colors are well within WCAG AAA
Comment 29 Gerrit Notification Bot 2014-08-12 23:12:02 UTC
Change 153726 had a related patch set uploaded by JGonera:
Make thumbnail caption color darker

https://gerrit.wikimedia.org/r/153726
Comment 30 Gerrit Notification Bot 2014-08-19 00:24:17 UTC
Change 153726 merged by jenkins-bot:
Make thumbnail caption color darker

https://gerrit.wikimedia.org/r/153726

Note You need to log in before you can comment on or make changes to this bug.


Navigation
Links