Ticket #12902 (closed Bug: fixed)

Opened 2 years ago

Last modified 2 years ago

Presentation view needs language attribute and color specs for accessibility

Reported by: rmattb Owned by: rmattb
Priority: minor Milestone: 4.x
Component: Unknown Version: 4.2
Keywords: accessibility Cc:

Description

Presentation view needs html language attribute and specified background color & font size unit to meet WCAG 2.0 AA.

Change History

comment:2 Changed 2 years ago by bslash

Some of the offending css (a pixel-based font-size) is inserted by javascript at Products.CMFPlone/Products/CMFPlone/skins/plone_3rdParty/s5_slides.js

Dividing the calculated pixel-size by 16 and inserting that as "em" seems to work just fine, and is WCAG 2.0 compliant. More browsertests to follow.

change line 395 from

fontSize(newSize + 'px');

to:

fontSize(newSize/16 + 'em');

Last edited 2 years ago by bslash (previous) (diff)

comment:3 Changed 2 years ago by bslash

The background color is actually defined (as "white") on the <body> tag. Which is as it should be, it should not be defined on the <html> tag. So that was probably just a glitch in the testing software.

comment:4 Changed 2 years ago by bslash

Browsertests were successful, including various mobile browsers. The presentation view is broken on various Opera flavours, but they were already so before this fix.

comment:5 Changed 2 years ago by bslash

  • Status changed from new to closed
  • Resolution set to fixed
Note: See TracTickets for help on using tickets.