Commit da12f94
Add error handling to localStorage operations (sugarlabs#5329)
Wrap localStorage.setItem() calls in try-catch blocks to prevent
crashes when storage is unavailable (e.g., private browsing mode,
quota exceeded, or security restrictions).
Modified files:
- js/themebox.js: Theme preference storage
- js/languagebox.js: Language preference storage
- js/widgets/reflection.js: Analysis report storage
All localStorage operations now gracefully handle errors with
console.warn logging, ensuring the application remains functional
even when storage fails.
Co-authored-by: Vanshika <vanshika@198625@gmail.com>1 parent 25a84ba commit da12f94
3 files changed
+15
-3
lines changed| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
234 | 234 | | |
235 | 235 | | |
236 | 236 | | |
237 | | - | |
| 237 | + | |
| 238 | + | |
| 239 | + | |
| 240 | + | |
| 241 | + | |
238 | 242 | | |
239 | 243 | | |
240 | 244 | | |
| |||
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
263 | 263 | | |
264 | 264 | | |
265 | 265 | | |
266 | | - | |
| 266 | + | |
| 267 | + | |
| 268 | + | |
| 269 | + | |
| 270 | + | |
267 | 271 | | |
268 | 272 | | |
269 | 273 | | |
| |||
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
575 | 575 | | |
576 | 576 | | |
577 | 577 | | |
578 | | - | |
| 578 | + | |
| 579 | + | |
| 580 | + | |
| 581 | + | |
| 582 | + | |
579 | 583 | | |
580 | 584 | | |
581 | 585 | | |
| |||
0 commit comments