Active
Project:
Flex Slider
Version:
7.x-2.x-dev
Component:
Documentation
Priority:
Normal
Category:
Support request
Assigned:
Unassigned
Issue tags:
Reporter:
Created:
16 Apr 2013 at 15:08 UTC
Updated:
15 Feb 2017 at 15:34 UTC
Jump to comment: Most recent
Comments
Comment #1
rcaracaus commentedYou can add an href=# to the a tag in the library, really, the maintainer needs to update flexslider to make that pause button accessibile. https://github.com/woothemes/FlexSlider/issues/15
Comment #2
rcaracaus commentedhttps://github.com/woothemes/FlexSlider/pull/681 I submitted a PR to do it.
Comment #3
mgiffordThanks @rcaracaus for adding this to the core library.
Pausing the text is certainly important as is the previous/next links. This library also allows you to click on a range of sliders to go forward/back. That functionality should be brought ahead for keyboard only users.
Hopefully we can get some momentum behind your pull request and of a better review of the flex slider code.
For screen readers it would be useful to actually fill that link with some hidden text. Something like:
var pausePlayScaffold = $('<div class="' + namespace + 'pauseplay"><span>Pause/Play</span><a href="#"></a></div>');However, there are issues with both i18n & making that invisible (when you can't rely on Drupal's element-invisible).
Comment #4
minoroffense commentedAny docs or fixes will be against 2.x
Comment #5
amaria commentedComment #6
rooby commented@amaria, could you give a brief description of why it is outdated? Has this already been fixed since the last comment?
Comment #7
amaria commentedOh, wait. I thought this was already fixed in the library itself but it seems they're targeting the 2.7 version. Although the question was answered in the comments. So maybe I'll add that in the documentation as well as the target for completion, to close this.