Closed Bug 472803 Opened 12 years ago Closed 12 years ago

aria-channel is being deprecated, use the "assertive" politeness level instead

Categories

(Other Applications :: ChatZilla, defect)

Product:
Other Applications
Component:
ChatZilla
Type:
defect
Priority:
Not set
Severity:
normal

Tracking

(Not tracked)

Status:
RESOLVED FIXED

People

(Reporter: Gijs, Assigned: Gijs)

References

(Blocks 1 open bug,
URL
)

Details

(Keywords: access, Whiteboard: [cz-0.9.85])

Attachments

(1 file)

Patch
982 bytes, patch
aaronlev
: review+
bugzilla-mozilla-20000923
: review+
Details | Diff | Splinter Review 
The line in the URL needs to be changed to:

msgRow.setAttribute("aria-live", "assertive");

as the current thing is being deprecated (and will not be implemented by any AT vendors, sadly).
Attached patch Patch — — Splinter Review 
Trivial patch
Attachment #356153 - Flags: review?(aaronleventhal)
Attachment #356153 - Flags: review?(silver)
Attachment #356153 - Flags: review?(aaronleventhal) → review+
Comment on attachment 356153 [details] [diff] [review]
Patch

Perfect! Thanks Gijs.
Yes, and as of this morning, FF trunk removed support for aria-channel, recommend pushing this fix sooner than later.
Comment on attachment 356153 [details] [diff] [review]
Patch

r=silver on the condition that aria-channel was either never in a released product or that nobody will try to use it.
Attachment #356153 - Flags: review?(silver) → review+
Status: NEW → ASSIGNED
Checking in mozilla/extensions/irc/xul/content/static.js;
/cvsroot/mozilla/extensions/irc/xul/content/static.js,v  <--  static.js
new revision: 1.290; previous revision: 1.289
done
Status: ASSIGNED → RESOLVED
Closed: 12 years ago
Resolution: --- → FIXED
Whiteboard: [cz-0.9.85]
You need to log in before you can comment on or make changes to this bug.