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

Search Doesn't Work on Safari in Private Browsing #53

Open
sebthedev opened this issue Apr 20, 2017 · 0 comments
Open

Search Doesn't Work on Safari in Private Browsing #53

sebthedev opened this issue Apr 20, 2017 · 0 comments
Assignees
Labels

Comments

@sebthedev
Copy link
Owner

On both desktop and mobile Safari, search doesn't work when the user is browsing with Private Browsing. This is because Safari throws a JavaScript error on any attempt to read/write to localStorage if the user is in Private Browsing. Specifically, when searching, search.js:70 throws an error QuotaExceededError (DOM Exception 22): The quota has been exceeded. This JavaScript error causes the searching operation to abort. Googling confirms this is a thing that Safari does – it's not just us!

To fix this, we could make use of try/catch statements around all localStorage calls to ensure that even if the call to localStorage fails, the rest of the action continues.

This is probably not an imminently urgent bug because of the relatively few people who will be using Princeton Courses on with Private Browsing enabled.

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

No branches or pull requests

3 participants