Closed
Bug 1250967
Opened 10 years ago
Closed 10 years ago
Extension inline options do not display
Categories
(Firefox for Android Graveyard :: Add-on Manager, defect)
Tracking
(firefox44 unaffected, firefox45 unaffected, firefox46+ fixed, firefox47+ fixed, fennec46+)
RESOLVED
FIXED
Firefox 47
| Tracking | Status | |
|---|---|---|
| firefox44 | --- | unaffected |
| firefox45 | --- | unaffected |
| firefox46 | + | fixed |
| firefox47 | + | fixed |
| fennec | 46+ | --- |
People
(Reporter: tech4pwd, Assigned: mfinkle)
References
Details
(Keywords: regression)
Attachments
(1 file)
|
3.77 KB,
patch
|
mossop
:
review+
lizzard
:
approval-mozilla-aurora+
|
Details | Diff | Splinter Review |
User Agent: Mozilla/5.0 (Android 6.0; Mobile; rv:47.0) Gecko/47.0 Firefox/47.0
Build ID: 20160224030246
Steps to reproduce:
The section within which online options should display simply displays the word "options".
| Reporter | ||
Updated•10 years ago
|
Severity: normal → major
OS: Unspecified → Android
Hardware: Unspecified → All
Comment 1•10 years ago
|
||
Hi,i would like to work on this.Could you provide more details on the issue?I'm unable to understand.
Thanks
Updated•10 years ago
|
Flags: needinfo?(pwd.mozilla)
| Reporter | ||
Comment 2•10 years ago
|
||
Assuming you have ABP installed:
STR:
Open Fennec
Open overflow menu
Open tools
Open add-ons
Open ABP
Expected Behaviour:
In-line add-on options are displayed e.g. "Disable everywhere, Filter subscriptions, etc"
Actual Results
The word OPTIONS is written in bold with nothing displayed below.
Notes:
This works as expected in release, but the bug exists on Nightly. I'm unsure about beta or aurora.
Flags: needinfo?(pwd.mozilla)
Updated•10 years ago
|
Comment 3•10 years ago
|
||
[Tracking Requested - why for this release]: regression
Status: UNCONFIRMED → NEW
status-firefox44:
--- → unaffected
status-firefox45:
--- → unaffected
status-firefox46:
--- → affected
status-firefox47:
--- → affected
tracking-firefox46:
--- → ?
tracking-firefox47:
--- → ?
Ever confirmed: true
Updated•10 years ago
|
Summary: Inline options do not display → Extension inline options do not display
Comment 4•10 years ago
|
||
You done gone and broke some stuff.
Mozregression - https://hg.mozilla.org/mozilla-central/pushloghtml?fromchange=9d6ffc7a08b6b47056eefe1e652710a3849adbf7&tochange=e0bcd16e1d4b99ba3e542149d0d41e0f60c54b5c
Blocks: 590127
Flags: needinfo?(mark.finkle)
Comment 5•10 years ago
|
||
We should be able to write a simple mochitest for this. I have an add-on that creates the different types of options we support:
https://github.com/leibovic/options-kitchen-sink/
See also bug 1079466, where I want us to get rid of this options.xul support.
| Assignee | ||
Comment 6•10 years ago
|
||
Looks like we need to add the setting.xml file back, maybe the css too.
Flags: needinfo?(mark.finkle)
| Assignee | ||
Comment 7•10 years ago
|
||
Dave - I had to add setting.xml back in since we use that binding. The binding uses a DTD file, so I also added it back.
Tested with Pocket Hits and the options now show up.
Assignee: nobody → mark.finkle
Attachment #8726076 -
Flags: review?(dtownsend)
Updated•10 years ago
|
Attachment #8726076 -
Flags: review?(dtownsend) → review+
Updated•10 years ago
|
tracking-fennec: ? → 46+
| Assignee | ||
Comment 9•10 years ago
|
||
Comment on attachment 8726076 [details] [diff] [review]
fix-addon-options v0.1
Approval Request Comment
[Feature/regressing bug #]: A patch on Fx46
[User impact if declined]: No options in Add-on Manager for add-ons
[Describe test coverage new/current, TreeHerder]:
[Risks and why]: Low. Added back 2 excluded files.
[String/UUID change made/needed]: None
Attachment #8726076 -
Flags: approval-mozilla-aurora?
Comment 10•10 years ago
|
||
| bugherder | ||
Status: NEW → RESOLVED
Closed: 10 years ago
Resolution: --- → FIXED
Target Milestone: --- → Firefox 47
Comment 12•10 years ago
|
||
Comment on attachment 8726076 [details] [diff] [review]
fix-addon-options v0.1
We want the addon manager ui to work. Please uplift!
Let's verify this in beta 1.
Attachment #8726076 -
Flags: approval-mozilla-aurora? → approval-mozilla-aurora+
Comment 13•10 years ago
|
||
| bugherder uplift | ||
Updated•5 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
•