Closed
Bug 630092
Opened 14 years ago
Closed 14 years ago
Missing notification for allowing offline app cache
Categories
(SeaMonkey :: UI Design, defect)
SeaMonkey
UI Design
Tracking
(Not tracked)
RESOLVED
FIXED
People
(Reporter: kairo, Assigned: neil)
References
()
Details
Attachments
(1 file, 1 obsolete file)
(deleted),
patch
|
iannbugzilla
:
review+
|
Details | Diff | Splinter Review |
When loading http://html5demos.com/offlineapp in Firefox, I get a notification where I can allow offline app caching, but I don't get that in SeaMonkey.
It just sets a permission for "offline-apps" in the end, and if that one is set, the demo works, but the notification is missing in SeaMonkey.
Reporter | ||
Updated•14 years ago
|
blocking-seamonkey2.1: --- → ?
Comment 1•14 years ago
|
||
Possibly relevant Firefox bugs:
[Bug 394392] UI for enabling/disabling offline applications.
[bug 414931] remove cruft from browser.js and make offline storage website list in preferences.
[Bug 399528] Add 'never for this site' option to the offline-app notification bar.
[Bug 419108] Offline exceptions button uses Ci where it isn't available.
[Bug 397417] monitor disk usage of offline apps in the UI.
[Bug 424017] list of offline apps broken on non-mac platforms.
[Bug 424281] Fix up openAdvancedPreferences() for modal prefs dialogs.
[Bug 442806] Use separate, versioned caches for offline apps.
[Bug 442810] update ui for app cache changes.
[Bug 526230] nsIApplicationCacheService::getGroups count outparam should be optional.
Reporter | ||
Comment 2•14 years ago
|
||
Bug 394392 (the main/notification part of it, the prefs part is something I'm integrating into Data Manager, that's how I found this) sounds like it, though we should check to see that we port the current version of this.
That should be relatively easy to do, maybe Edmund wants to help there?
Assignee | ||
Comment 3•14 years ago
|
||
(In reply to comment #1)
> Possibly relevant Firefox bugs:
> [Bug 394392] UI for enabling/disabling offline applications.
Not forgetting the dependent bug 462856.
Assignee | ||
Comment 4•14 years ago
|
||
Rather conveniently, indexedDB uses the same strings, so they're already there!
Assignee | ||
Comment 5•14 years ago
|
||
This just does the prompt; there's another part that monitors usage.
Attachment #510114 -
Flags: feedback?(iann_bugzilla)
Assignee | ||
Comment 6•14 years ago
|
||
Apparently the quota warning is expected to be able to open the offline app preferences, but somebody has to write them first, so in the mean time it's just a plain notification bar.
Attachment #510114 -
Attachment is obsolete: true
Attachment #510167 -
Flags: review?(iann_bugzilla)
Attachment #510114 -
Flags: feedback?(iann_bugzilla)
Comment on attachment 510167 [details] [diff] [review]
Notifications part only
[Checked in: Comment 10]
Is there a bug logged on the offline app preferences you mentioned and for linking in this notification, if so could you add the relevant dependencies, if not could you add the bugs with relevant dependencies.
Would this ever become a doorhanger?
Attachment #510167 -
Flags: review?(iann_bugzilla) → review+
Assignee | ||
Comment 8•14 years ago
|
||
(In reply to comment #7)
> Is there a bug logged on the offline app preferences you mentioned and for
> linking in this notification, if so could you add the relevant dependencies, if
> not could you add the bugs with relevant dependencies.
Well, I'm not sure what prefs there are, but from comment #2 it sounds as if KaiRo was last looking into that side of it.
> Would this ever become a doorhanger?
Possibly (by overriding offlineAppRequested and checkUsage and maybe some refactoring), but I would have to wait for Firefox to convert theirs first.
Reporter | ||
Comment 9•14 years ago
|
||
(In reply to comment #8)
> Well, I'm not sure what prefs there are, but from comment #2 it sounds as if
> KaiRo was last looking into that side of it.
I haven't really looked into much of this, actually, in that comment I just went through the bugs and looked what sounds like it, which a fast glance at its patch.
Also, I don't know much about prefs for this, I just stumbled over the whole issue when investigating the web storage panel for Data Manager.
Assignee | ||
Comment 10•14 years ago
|
||
Comment on attachment 510167 [details] [diff] [review]
Notifications part only
[Checked in: Comment 10]
Pushed changeset 3c6ddf017567 to comm-central.
What other parts are there to finish support for offline apps?
Updated•14 years ago
|
Attachment #510167 -
Attachment description: Notifications part only → Notifications part only
[Checked in: Comment 10]
Comment 11•14 years ago
|
||
No comments to neil's Q since Feb, initial patch landed; this looks done.
Status: ASSIGNED → RESOLVED
blocking-seamonkey2.1: ? → ---
Closed: 14 years ago
Resolution: --- → FIXED
Assignee | ||
Comment 12•14 years ago
|
||
Looks like we're still missing the preferences.
You need to log in
before you can comment on or make changes to this bug.
Description
•