Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Graceful fallback for blocked JavaScript? #274

Closed
philippbayer opened this issue Mar 22, 2016 · 3 comments · Fixed by #511
Closed

Graceful fallback for blocked JavaScript? #274

philippbayer opened this issue Mar 22, 2016 · 3 comments · Fixed by #511
Labels

Comments

@philippbayer
Copy link
Member

Quite a few people nowadays run their browsers full-time with plugins which block all JavaScript to disable tracking scripts (like NoScript).

Not surprisingly this breaks half of openSNP's functionality if you don't whitelist the page. I believe it would be nice to have a graceful fallback to a clunkier interface if a user doesn't support JavaScript. It's something bigger to fix, you'd have to partially reimplement a few pages (replace a drop-down menu with all options, for example). If you don't agree, feel free to close this issue.

Other "big" places like GitHub or Twitter don't do it, Twitter simply doesn't work without JavaScript. DuckDuckGo has a non-JS html fallback: https://duckduckgo.com/html/?q=bootstrap%20without%20jquery

@gedankenstuecke
Copy link
Member

👍 Good idea, could be part of #264 😃

@tsujigiri
Copy link
Collaborator

It's quite a lot of work, though. Much more than having everyone who blocks JavaScript, collectively whitelist openSNP, I'd assume. ;) On the other hand, it would benefit users of screen readers as well. And it would be a great opportunity to clean up the HTML...

@raivivek
Copy link
Member

raivivek commented May 20, 2016

I think Bootstrap does try to fall back to a HTML interface in case it cannot render a feature. In any case, we can always display a message saying OpenSNP works best with Javascript enabled, currently viewing in reduced functionality mode.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging a pull request may close this issue.

4 participants