Closed
Bug 1017850
Opened 11 years ago
Closed 10 years ago
Don't break the site (js) if localStorage isn't available
Categories
(support.mozilla.org :: General, defect, P3)
support.mozilla.org
General
Tracking
(Not tracked)
RESOLVED
FIXED
2014Q4
People
(Reporter: gingerbread_man, Assigned: rehandalal+mozilla)
References
Details
(Whiteboard: u=user c=questions p=1 s=2014.20)
Attachments
(1 file)
32.49 KB,
image/png
|
Details |
Also see bug 1001284 and bug 1001283.
The correct functioning of the site is dependent on DOM storage being enabled. Among some of the weird things that happen with it disabled, the buttons at the top of the reply box are all missing, and the Sign Out link does nothing but scroll the page (it acts like a regular anchor).
It would be nice if there was a warning when DOM storage is disabled. This would benefit those who've had it disabled by an extension, disabled it at the urging of advice on the web, or simply turned it off themselves not anticipating consequences like this.
If needed, some ideas for checking if DOM storage is enabled:
http://stackoverflow.com/questions/16427636/check-if-localstorage-is-available
Comment 1•10 years ago
|
||
This just came up again. See:
https://support.mozilla.org/forums/contributors/710809
Unable to post a new question in the support forum
Since it is possible to answer questions on the support forum without DOM storage enabled, why make it impossible to ask them without it? It is only on https://support.mozilla.org/en-US/questions where DOM storage is needed to ask questions, but not on https://support.mozilla.org/en-US/forums/contributors. Why?
Comment 3•10 years ago
|
||
We should look into this. :mythmon mentioned the real problem is that we are using localStorage without checking if it's available.
Priority: -- → P3
Whiteboard: u=user c=questions p= s=2014.20
Target Milestone: --- → 2014Q4
Comment 4•10 years ago
|
||
I think it's at least 2pts to go through and fix localStorage usage.
Summary: Display a warning when DOM storage is disabled (dom.storage.enabled = false) → Don't break the site (js) if localStorage isn't available
Whiteboard: u=user c=questions p= s=2014.20 → u=user c=questions p=2 s=2014.20
Assignee | ||
Updated•10 years ago
|
Assignee: nobody → rdalal
Comment 5•10 years ago
|
||
Rehan says 1pt!
Whiteboard: u=user c=questions p=2 s=2014.20 → u=user c=questions p=1 s=2014.20
Assignee | ||
Comment 6•10 years ago
|
||
Landed:
https://github.com/mozilla/kitsune/commit/b03b099e57a43717796fe0890af44ba96a7b51c8
Pushed to prod by me, right now.
Status: NEW → RESOLVED
Closed: 10 years ago
Resolution: --- → FIXED
Comment 7•10 years ago
|
||
This came up again today.
https://support.mozilla.org/en-US/forums/contributors/710986 Support site not working. Unable to search for problem solution; "Ask this" button not working on FF 20.0. Question post not available
"about:config dom.storage.enabled;true solved the problem"
Comment 8•10 years ago
|
||
Problems again see discussion
> SuMo articles: Images are not shown if "dom.storage.enabled" = false
https://support.mozilla.org/en-US/forums/contributors/711083
Further bug filed Bug 1131771
You need to log in
before you can comment on or make changes to this bug.
Description
•