DROP Task: Improve accessibility of forms and messages

kika - May 27, 2004 - 15:15
Project:Drupal
Version:7.x-dev
Component:theme system
Category:task
Priority:normal
Assigned:kika
Status:postponed
Issue tags:accessibility, CSS, Needs color accessibility review
Description

Here's a first sub-task of the greater "Improve the user experience of Drupal forms" metatask.

I took standard Drupal log-in dialog and similar Basecamp (http://www.basecamphq.com) dialog, run them through the VisCheck (a color blindness analyzer)
and added couple of defict simulations of my own.

The results with comments are attached.

For reference, VisCheck simulation URLs are here (but I am not sure how long they gonna stay in cache):

Drupal:

http://www.vischeck.com/vischeck/vischeckImage.php?&img1=http%3A%2F%2Fvi...

Basecamp:

http://www.vischeck.com/vischeck/vischeckImage.php?&img1=http%3A%2F%2Fvi...

AttachmentSizeStatusTest resultOperations
drupal_forms_accessibility_simulation.png179.32 KBIgnoredNoneNone

#1

Dries - May 31, 2004 - 10:11

Committed some first changes to HEAD to facilitate these suggestions. See my mail sent to the mailing list for details.

#2

kika - June 12, 2004 - 19:24

Here's the next task: styling the required form fields

I tend to prefer the combination of #2.1 and #3.1

AttachmentSizeStatusTest resultOperations
drupal_forms_styling_required.png74.47 KBIgnoredNoneNone

#3

kika - July 8, 2004 - 09:17

Improved "styling required" mockup

AttachmentSizeStatusTest resultOperations
drupal_forms_styling_required.jpg112.66 KBIgnoredNoneNone

#4

kika - July 8, 2004 - 09:18

styling "error" form fields mockup

AttachmentSizeStatusTest resultOperations
drupal_forms_error_required.jpg76.18 KBIgnoredNoneNone

#6

Dries - July 8, 2004 - 10:28

Any pointers on how we can preserve the "sunkeness" of these form fields? I tried fiddling with paddings and margins of 'form-item input.error' in misc/drupal.css, yet to no avail.

#7

kika - July 15, 2004 - 22:13

An item to the linkdump:

http://www-106.ibm.com/developerworks/usability/library/us-widget/
Jodi Bollaert defines several basic HTML Web widgets, shows graphic examples, and discusses common usability problems and solutions

#8

coreb - December 5, 2006 - 00:30
Version:x.y.z» 6.x-dev

Moving from x.y.z queue to 6.x-dev.

#9

kkaefer - February 7, 2008 - 16:57
Version:6.x-dev» 7.x-dev

This won't be tackled in 6.x anymore (also, it's merely a theming issue).

#10

Pancho - February 8, 2008 - 06:16
Title:Form accessibility» Improve accessibility of forms and messages
Component:base system» theme system
Priority:normal» critical

True, but let's at least tackle it in D7! This is another member of first generation issues that still haven't been solved.
I mark this critical as we are striving to be the most accessible CMS, and the reality is that we are very much behind the 'state of the art'.

#11

webchick - February 17, 2008 - 04:50

Wow. Neat issue. Subscribe.

#12

webchick - February 17, 2008 - 04:51
Title:Improve accessibility of forms and messages» DROP Task: Improve accessibility of forms and messages

And in fact, let's DROP-ify this.

Completing this task will first require re-running the tests in kika's original post, since the images are over 3 years old now, and reporting on the results. Next, we'll need a patch that solves any accessibility issues within the default theme functions.

#13

Stefan Nagtegaal - February 17, 2008 - 14:13

Hmmm... Indeed it is webchick.. :-)
This is on my todo list, if none of you guys/girls beat me to it...

#14

Rob Loach - February 28, 2008 - 02:46

Another thing that might be helpful is the jQuery Form Input Example Plugin: Demonstration.

AttachmentSizeStatusTest resultOperations
yahoo-registration-form[1].png21.79 KBIgnoredNoneNone

#15

catch - March 1, 2008 - 22:31
Title:DROP Task: Improve accessibility of forms and messages» DROP Task: Improve accessibility of forms and messages

We noticed that the password strength checker has red text on a red background during usability testing at UMN, I can't imagine any way this would pass accessibility requirements so changing title.

#16

aclight - August 21, 2008 - 11:22

subscribing

#17

gaele - September 30, 2008 - 16:54

Re #15: this is about .admin-missing in admin.css.

Here is an online color contrast checker: http://snook.ca/technical/colour_contrast/colour.html

Red text is #FF0000, background is #FFCCCC. The color difference is too low. A text color of #A30000 would do. (Too me it's easier on the eye too.)

AttachmentSizeStatusTest resultOperations
admin.css_.patch259 bytesIdlePassed on all environments.View details | Re-test

#18

lilou - October 17, 2008 - 00:03
Status:active» needs review

#19

Anonymous (not verified) - November 12, 2008 - 07:40
Status:needs review» needs work

The last submitted patch failed testing.

#20

gaele - November 12, 2008 - 14:44

The results are detailed indeed, but totally incomprehensible.

#21

catch - November 12, 2008 - 17:31
Status:needs work» needs review

re-setting to see if it was a broken HEAD.

#22

Anonymous (not verified) - November 12, 2008 - 17:35
Status:needs review» needs work

The last submitted patch failed testing.

#23

catch - November 12, 2008 - 18:03
Status:needs work» needs review

and again.

#24

Anonymous (not verified) - November 12, 2008 - 18:05
Status:needs review» needs work

The last submitted patch failed testing.

#25

catch - November 12, 2008 - 18:07
Status:needs work» needs review

daa dee da

#26

Anonymous (not verified) - November 12, 2008 - 18:10
Status:needs review» needs work

The last submitted patch failed testing.

#27

catch - November 12, 2008 - 18:20
Status:needs work» needs review
AttachmentSizeStatusTest resultOperations
admin.css_.patch259 bytesIdlePassed on all environments.View details | Re-test

#28

webchick - November 13, 2008 - 00:33

Can we please get a screenshot with this patch applied to more easily evaluate it?

#29

gaele - November 13, 2008 - 14:16
Status:needs review» active

Forget about the patch.

The old password strength checker was abusing the .admin-missing style. But the whole password strength checker was replaced with something better #199870: Usability: Even better password strength experience. Now .admin-missing is only used on a grey background (for missing modules on the modules page), so #FF0000 is appropriate.

Not sure what to do with the issue. Can we close it? Or should we turn it into "run all of D7 through a color blindness analyzer" after code freeze?

#30

catch - November 13, 2008 - 15:27
Priority:critical» normal
Status:active» postponed

It'd be good to have a 'run D7 through a color blindness analyzer' -but it might be better discussed in the usability/theming groups (if there's no specific accessibility group). Let's mark this postponed until code freeze.

#31

mgifford - February 4, 2009 - 17:07

I just ran the user page through the Vischeck Analyzer.

I had applied the patch, but that seems to no longer be required. The results are up on Flickr.

Mouse-overs seem to be the biggest problem from these tests.

Mike

#32

mgifford - February 17, 2009 - 13:21

#33

mgifford - March 17, 2009 - 16:09
Issue tags:+CSS

Just added this related post - http://groups.drupal.org/node/20189

#34

mgifford - September 8, 2009 - 21:10

Adding a tag for color accessibility review. Needs to be done for both red/green as well as contrast. Seven in particular uses a lot of greys and the vertical tabs have some issues with the summary text too.

 
 

Drupal is a registered trademark of Dries Buytaert.