Project

General

Profile

Feature #7898

WCAG AAA in 1.10.0

Added by Yannick Warnier almost 4 years ago. Updated almost 4 years ago.

Status:
Feature implemented
Priority:
Low
Category:
Global / Others / Misc
Target version:
Start date:
14/10/2015
Due date:
% Done:

100%

Estimated time:
Complexity:
Normal
SCRUM pts - complexity:
?

Description

Some issues need to be fixed to widely support WCAG AAA (Validator at http://achecker.ca/checker/index.php).

In general terms, using the <i> tag generates an error, so all <i class="..."></i> used for font awesome have to be changed to <em>.
As suggested here: http://stackoverflow.com/questions/23015537/how-do-i-make-font-awesome-wcag-2-0-compatible we could use a span also, but it would be even better to adopt aria tags or use bootstrap's sr-only tag to put a title there
Then all "alt" attributes need a value, etc.


Files

wcag.png (57.1 KB) wcag.png Yannick Warnier, 14/10/2015 07:22

Associated revisions

Revision b6d11076 (diff)
Added by Angel Fernando Quiroz Campos almost 4 years ago

Change of <i> to <em> for WCAG support - refs #7898

Revision 483f502e (diff)
Added by José Loguercio almost 4 years ago

WCAG AAA improvement onChange JS - Refs #7898

Revision acf81d55
Added by Yannick Warnier almost 4 years ago

Merge pull request #882 from jloguercio/7898

WCAG AAA improvement onChange JS - Refs #7898

History

#1

Updated by Yannick Warnier almost 4 years ago

  • Description updated (diff)
#2

Updated by Yannick Warnier almost 4 years ago

  • File wcag.png wcag.png added
  • Assignee changed from Yannick Warnier to Julio Montoya

AAA now validates!

There is an "likely problem" with the fact that the <select> language element contains an "onchange" attribute. Apparently this is not accepted too well for AAA. It would be nice to move this "onchange" to a document.ready() function in the header of the page... Assigning to Julio for now.

#3

Updated by Yannick Warnier almost 4 years ago

  • Assignee changed from Julio Montoya to José Loguercio
#4

Updated by Yannick Warnier almost 4 years ago

  • Priority changed from Normal to Low

This is not required for 1.10.0. Only an improvement.

#5

Updated by José Loguercio almost 4 years ago

  • Status changed from Assigned to Needs testing
  • Assignee changed from José Loguercio to Yannick Warnier

onChange now implemented in jquery, script works fine, need testing for WCAG AAA.

PR : https://github.com/chamilo/chamilo-lms/pull/882

#6

Updated by José Loguercio almost 4 years ago

  • % Done changed from 0 to 80
#7

Updated by Yannick Warnier almost 4 years ago

  • Status changed from Needs testing to Feature implemented
  • % Done changed from 80 to 100

Tested and approved.

Also available in: Atom PDF