passport / www.passportjs.org

The official website of Passport.js.
https://www.passportjs.org
45 stars 68 forks source link

New site breaks non-JS browsers #29

Open joepie91 opened 9 years ago

joepie91 commented 9 years ago

Re-filing this as there is apparently a separate repository for the website (original at jaredhanson/passport#353).

Browser: Chrome 42.0.2311.152, openSUSE 13.1 x86_64.


Upon finding that the site for Passport had changed significantly, and that it used a lot of fancy JS "enhancements", I decided to have a look at how well it functioned without JS. As it turns out, it doesn't, at all.

The "view strategies" button is completely broken. "Section jumping" in the documentation is completely broken. Neither of these are functionality that absolutely requires JS. Whatever happened to the practice of progressive enhancement?

Documentation should really be available to anybody with a web browser (and, ideally, even to people without a web browser). There are many people who either cannot enable JS in their browser, do not want to enable JS in their browser (eg. for security reasons), or who recursively download a site because their internet connection is unreliable/limited.

The current site breaks all of these usecases, and that should really be fixed.