Closed Bug 1170843 Opened 9 years ago Closed 9 years ago

Show a warning when a user attempts to install an unsigned add-on (if the pref allows)

Categories

(Firefox for Android Graveyard :: Add-on Manager, defect)

35 Branch
defect
Not set
normal

Tracking

(firefox41 verified)

VERIFIED FIXED
Firefox 41
Tracking Status
firefox41 --- verified

People

(Reporter: Margaret, Assigned: Margaret)

References

Details

Attachments

(3 files)

Android version of bug 1148021.
Bug 1170843 - Show a warning when a user attempts to install an unsigned add-on. r=mfinkle
Attachment #8616804 - Flags: review?(mark.finkle)
Attached image screenshot
This isn't really pretty, but right now we only support a single text message in these prompts, so this is the best I can do without a lot more effort.

I don't think this is so bad, and this wouldn't been seen by release users, so I don't care to make it better.
Actually, Anthony has a mock-up for this in bug 1170043, so I should probably update this patch to follow that more closely.

Anthony, the current install notification includes the name of the add-on, but the name isn't included in your mock-up. Should I include it?

I do see you included the name of the add-on in the "Verified add-on" install dialog (although I couldn't implement the blue color for the add-on name without more extensive changes). We don't currently have a bug filed for changing the regular install dialog, so I can file another bug for this.
Flags: needinfo?(alam)
I don't think we can add the little images in the top/left corner of the dialog, can we? Your dialog looks OK as a first draft.
Comment on attachment 8616804 [details]
MozReview Request: Bug 1170843 - Show a warning when a user attempts to install an unsigned add-on. r=mfinkle

https://reviewboard.mozilla.org/r/10477/#review9239

Ship It!
Attachment #8616804 - Flags: review?(mark.finkle) → review+
(In reply to Mark Finkle (:mfinkle) from comment #4)
> I don't think we can add the little images in the top/left corner of the
> dialog, can we? Your dialog looks OK as a first draft.

Yeah, I talked with Anthony about that, and it would just be a "nice to have". I think we would need to change our prompt implementation if we want to support that, so definitely not a requirement here.
Works for me! Can we have the name of the add-on below, like this? and the title change too?
Flags: needinfo?(alam) → needinfo?(margaret.leibovic)
(In reply to Anthony Lam (:antlam) from comment #7)
> Created attachment 8617568 [details]
> prev_installwarned_temp.png
> 
> Works for me! Can we have the name of the add-on below, like this? and the
> title change too?

Yep, updated the patch and landed it.
Flags: needinfo?(margaret.leibovic)
https://hg.mozilla.org/mozilla-central/rev/3543d6a877ea
Status: NEW → RESOLVED
Closed: 9 years ago
Resolution: --- → FIXED
Target Milestone: --- → Firefox 41
Verified as fixed on Firefox 41 Beta 1
Status: RESOLVED → VERIFIED
Product: Firefox for Android → Firefox for Android Graveyard
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: