432033 - Accessibility fixes in mail/components/preferences XUL files

Status: RESOLVED FIXED

(Thunderbird :: Preferences, defect)

Description

[Marco Zehe (:MarcoZ)]

Add missing label/control relations, aria-labelledby (for those expressions made up of several controls), and other accessibility-related markup that may be missing.

Comment 1

[Marco Zehe (:MarcoZ)]

Attachment: Patch

Comment 2

[Marco Zehe (:MarcoZ)]

Attachment: Updated patch

Got rid of xul:description/aria-labelledby constructs and replaced with xul:label markup. Affected files: display.xul, receipts.xul, sendoptions.xul, viewPasswords.xul. In sendOptions.xul, also added the control attribute to the xul:description element that says "Note: ...". That was, the combobox gets this added information in its accessibleDescription so users can ask for it.

Comment 3

[Phil Ringnalda (:philor)]

Comment on attachment 320668 [details] [diff] [review]
Updated patch

After all the time I took to review it, you'd think I'd have more than just a handful of nits, wouldn't you? Though most of that time was talking myself out of other nits that aren't your fault or your responsibility to fix (or, often enough, even in lines you were touching). r=philringnalda, with:

>Index: mail/components/preferences/advanced.xul
>+            <label value="&markAsReadEnd.label;" id="markAsReadEnd"/>

id as the first attribute, please: we're not quite perfect about doing that, but "more often than not" counts as intended style, given the inconsistency in our prefs xul.

>+              <label id="connectionTimeoutLabel" value="&connectionTimeout.label;" accesskey="&connectionTimeout.accesskey;" control="connectionTimeoutBox" />

>+              <label id="connectionSeconds" value="&connectionSeconds.label;" />

Please remove the space before the "/>" - this isn't pretend-XHTML served as HTML, which is the only place that comes close to making sense.

>Index: mail/components/preferences/display.xul
>+            <hbox class="indent" align="center" role="group" id="displayText">

id first here, too.

And thanks for being patient.

Comment 4

[Marco Zehe (:MarcoZ)]

Checking in mail/components/preferences/advanced.xul;
/cvsroot/mozilla/mail/components/preferences/advanced.xul,v  <--  advanced.xul
new revision: 1.26; previous revision: 1.25
done
Checking in mail/components/preferences/changeaction.xul;
/cvsroot/mozilla/mail/components/preferences/changeaction.xul,v  <--  changeaction.xul
new revision: 1.8; previous revision: 1.7
done
Checking in mail/components/preferences/compose.xul;
/cvsroot/mozilla/mail/components/preferences/compose.xul,v  <--  compose.xul
new revision: 1.16; previous revision: 1.15
done
Checking in mail/components/preferences/display.xul;
/cvsroot/mozilla/mail/components/preferences/display.xul,v  <--  display.xul
new revision: 1.8; previous revision: 1.7
done
Checking in mail/components/preferences/dsn.xul;
/cvsroot/mozilla/mail/components/preferences/dsn.xul,v  <--  dsn.xul
new revision: 1.2; previous revision: 1.1
done
Checking in mail/components/preferences/general.xul;
/cvsroot/mozilla/mail/components/preferences/general.xul,v  <--  general.xul
new revision: 1.15; previous revision: 1.14
done
Checking in mail/components/preferences/offline.xul;
/cvsroot/mozilla/mail/components/preferences/offline.xul,v  <--  offline.xul
new revision: 1.6; previous revision: 1.5
done
Checking in mail/components/preferences/privacy.xul;
/cvsroot/mozilla/mail/components/preferences/privacy.xul,v  <--  privacy.xul
new revision: 1.17; previous revision: 1.16
done
Checking in mail/components/preferences/receipts.xul;
/cvsroot/mozilla/mail/components/preferences/receipts.xul,v  <--  receipts.xul
new revision: 1.4; previous revision: 1.3
done
Checking in mail/components/preferences/sendoptions.xul;
/cvsroot/mozilla/mail/components/preferences/sendoptions.xul,v  <--  sendoptions.xul
new revision: 1.6; previous revision: 1.5
done
Checking in mail/components/preferences/viewpasswords.xul;
/cvsroot/mozilla/mail/components/preferences/viewpasswords.xul,v  <--  viewpasswords.xul
new revision: 1.3; previous revision: 1.2
done

Checked in with review comments addressed.