Closed
Bug 892125
Opened 11 years ago
Closed 10 years ago
Create "Add-ons" page in settings
Categories
(Firefox for Android Graveyard :: General, defect, P5)
Tracking
(fennec+)
RESOLVED
WONTFIX
Tracking | Status | |
---|---|---|
fennec | + | --- |
People
(Reporter: Margaret, Unassigned)
References
Details
(Keywords: productwanted)
Attachments
(3 files)
13.45 KB,
patch
|
Details | Diff | Splinter Review | |
13.80 KB,
patch
|
Details | Diff | Splinter Review | |
59.58 KB,
patch
|
Details | Diff | Splinter Review |
This bug is about making a first implementation of the "Add-ons" page in settings. To start, let's have "Browse Firefox Add-ons" open a link to AMO, and just show a list of installed add-ons. You'll need to communicate with JS to get the add-on data, similarly to how the "SearchEngines:Get"/"SearchEngines:Data" messages work. I think it would be good to implement this in a lazy-loaded script (Addons.js or something like that), since we won't need this data on startup. You'll need to use toolkit's AddonManager.jsm module to get the addons. Here's an example of how we currently do it: http://mxr.mozilla.org/mozilla-central/source/mobile/android/chrome/content/aboutAddons.js#232
Comment 1•11 years ago
|
||
To start - the uninteresting resource-modification-and-such patch.
Attachment #777284 -
Flags: review?(liuche)
Comment 2•11 years ago
|
||
It appears that not all add-ons are equal - we categorise them into extensions, themes, and locales. Would it be worthwhile to have these on separate menus? I can certainly imagine a "Customise themes" menu might be worth having, and managing extra locales through a deeply hidden menu seems sort of annoying for the user (Who presumably isn't so great at english, since they're installing an addon to change it) For now I'll proceed with the "Jam everything onto one list" approach - just mentioning this thought as a possible enhancement.
Comment 3•11 years ago
|
||
Yeah, I'd like to keep them in one list for now. Most of our users don't really understand the difference between these types of add-ons, so there isn't much benefit to chunking them out.
Updated•11 years ago
|
tracking-fennec: ? → 26+
Comment 5•11 years ago
|
||
Comment on attachment 777284 [details] [diff] [review] Part 1/3: Adding the new menu page (Resource file changes, strings, etc.) Clearing this review flag - let's do review when we have more of the patches in a finished state.
Attachment #777284 -
Flags: review?(liuche)
Comment 7•11 years ago
|
||
Alas, there is no more time. Uploading my WIP patches - may or may not be of use.
Updated•11 years ago
|
Assignee: chriskitching → nobody
Reporter | ||
Comment 8•11 years ago
|
||
Re-noming, since this isn't a priority anymore.
tracking-fennec: 27+ → ?
Updated•11 years ago
|
tracking-fennec: ? → 29+
Reporter | ||
Comment 9•10 years ago
|
||
This isn't going to make 29. If we want to fix this, we should find an assignee.
tracking-fennec: 29+ → ?
Comment 10•10 years ago
|
||
Deb, what do you want to do here?
Flags: needinfo?(deb)
Keywords: productwanted
Comment 11•10 years ago
|
||
Is it realistic to target this for Fx31?
Reporter | ||
Comment 12•10 years ago
|
||
(In reply to Deb Richardson [:dria] from comment #11) > Is it realistic to target this for Fx31? I think liuche was going to start looking at this. She would probably be able to give an estimate.
Flags: needinfo?(liuche)
Comment 13•10 years ago
|
||
No, this won't make 31, because this is dependent on another settings rework, and we need to support some JS for add-on options. I'm not sure what this is supposed to look like, so I'll talk with ibarlow and see what ideas he has.
Flags: needinfo?(liuche)
Updated•10 years ago
|
tracking-fennec: ? → 32+
Updated•10 years ago
|
Flags: needinfo?(deb)
Comment 14•10 years ago
|
||
I've been playing a bit with reflecting the DOM into native UI, so that we could continue to use options.xul here, and allow pages to dynamically edit it. It doesn't do any work to actually reflect real add-ons into the Native UI yet. This uses MutationObservers to try and reflect dom changes into the native ui. The updates are slow sometimes. I'm not sure if that's the MutationObserver being very async, or Android being dumb about preference view invalidation (I assumed the former originally, but now I've definitely seen the later happen).
Updated•10 years ago
|
Status: NEW → ASSIGNED
Updated•10 years ago
|
tracking-fennec: 32+ → +
Updated•10 years ago
|
QA Contact: mihai.g.pop
Reporter | ||
Comment 16•10 years ago
|
||
We decided that it's not worth the effort to try to make a whole new native add-ons UI. Instead, we're using bug 1053397 to track improvements to the current about:addons page.
Status: ASSIGNED → RESOLVED
Closed: 10 years ago
Resolution: --- → WONTFIX
Assignee | ||
Updated•3 years ago
|
Product: Firefox for Android → Firefox for Android Graveyard
You need to log in
before you can comment on or make changes to this bug.
Description
•