readium / readium-js-viewer

👁 ReadiumJS viewer: default web app for Readium.js library
BSD 3-Clause "New" or "Revised" License
553 stars 186 forks source link

Fixes #533 Focus trapping in Dialogs #553

Closed becka11y closed 8 years ago

becka11y commented 8 years ago

This pull request is Finalized

Bootstrap-accessibility-plugin was updated for proper focus trapping. Readium build process updated to pull bootstrap-accessibility-plugin master branch. Remove existing readium code to trap focus in dialogs since it is no longer needed.

rkwright commented 8 years ago

@danielweck I am fine with this change. If you agree, please merge.

danielweck commented 8 years ago

Don't forget to invoke npm run prepare ! ;) (because PayPal's Bootstrap Accessibility Plugin package.json dependency URL changed)