Closed Bug 566194 Opened 14 years ago Closed 14 years ago

safe mode / security & compatibility check status are not exposed in new addon manager UI

Categories

(Toolkit :: Add-ons Manager, defect)

defect
Not set
normal

Tracking

()

VERIFIED FIXED
mozilla2.0b7
Tracking Status
blocking2.0 --- beta7+

People

(Reporter: xtc4uall, Assigned: Unfocused)

References

Details

(Whiteboard: [rewrite][strings][AOMTestday][in-litmus-bug-week])

Attachments

(4 files, 1 obsolete file)

* start Firefox in safe mode and/or
* set extensions.checkCompatibility.xxx to false and start Firefox and/or
* set extensions.checkUpdateSecurity to false and start Firefox.

in old addons manager UI there's a notification bar shown stating that safe mode is enabled and if you disabled the security or compatibility checks.

in the new addons manager this kind of information missing.
Flags: in-testsuite?
Flags: in-litmus?
blocking2.0: --- → final+
Keywords: uiwanted
Let's treat this in exactly the same way as before for now - a top level in-content notification on each pane of the add-ons manager that says "All add-ons have been disabled by safe mode."
Assignee: nobody → bparr
I think Boriss has covered this.
Needed by string freeze
blocking2.0: final+ → beta5+
Assignee: bparr → bmcbride
The attached mock shows a warning over Extensions in list view.  This is the organization bar for current list view - a similar bar would need to be added in Appearance view.

It may seem unusual to repurpose this area for a warning.  I tried a few alternatives, such as placing the warning over the entire section between forward/back and search (shown in mock).  However, this is awkward to color correctly and would only be a temporary solution, as breadcrumb navigation will be placed in this area in Firefox 4.n.  I also tried adding a bar under and over the current bar, but this looked visually redundant.  Though it's a bit unusual, I'm thus recommending we use this area to show a warning and otherwise leave the Name/Last Updated sorting widgets untouched.

We may need a similar warning here if we can add an option under the Gear menu to disable add-ons while already within Firefox.
blocking2.0: beta5+ → beta6+
Whiteboard: [rewrite] → [rewrite][strings]
Attached patch Patch v1 (obsolete) — Splinter Review
Known issues: Doesn't crop or wrap. Not sure why, but unless there's something obvious I'm missing that can be fixed very quickly, I'd like to fix that in a followup.

I'm purposefully not adding any warnings to the discovery pane. My reasoning is that you're not managing your local addons there, so none of the warnings are relevant. But it *is* displaying such warnings in remote search, because its a surprise to have them appear/disappear based on how you're filtering the search results (as opposed to navigating somewhere).

And as far as I know, there's no way to test safe mode, so that will need to be a litmus test.
Attachment #471356 - Flags: review?(dtownsend)
Attachment #471356 - Attachment is patch: true
Attachment #471356 - Attachment mime type: application/octet-stream → text/plain
Attached patch Patch v1.1Splinter Review
Oops - previous patch was missing a file in pinstripe, and had an additional file in winstripe. Fixed.
Attachment #471356 - Attachment is obsolete: true
Attachment #471440 - Flags: review?(dtownsend)
Attachment #471356 - Flags: review?(dtownsend)
Looks good but can you give me a diff ignoring whitespace changes. Also is there any reason we can't just include the warning once above the view deck?
(In reply to comment #9)
> Also is
> there any reason we can't just include the warning once above the view deck?

Only the details pane would be able to use it (other panes have the warnings in existing containers). But also, if it were outside the deck, then we'd need to hide/show it based on what the current view was - which seems a bit messy. The way the patch does it, the warnings can be fitted in based on where it makes sense for each view type. This will also make it easy to add them to the revamped appearance pane (when its eventually done), which will be different again.
Attachment #471440 - Flags: review?(dtownsend) → review+
Attachment #471754 - Flags: review?(dtownsend)
https://hg.mozilla.org/mozilla-central/rev/bb55a00afa1d
Status: NEW → RESOLVED
Closed: 14 years ago
Keywords: uiwanted
Resolution: --- → FIXED
Flags: in-testsuite? → in-testsuite+
Target Milestone: --- → mozilla2.0b6
Depends on: 593796
As it has been implemented now, the Notification Bar Text is not dismiss-able as it has been in the old AOM.

Hence in the "worst Case" of Safe-Mode: on + Compatibility Check: off + Update Security Check: off, only the Safe-Mode State is exposed, there other two are not.
With only Compatibility Check: off + Update Security Check: off, you only get the Security Check State exposed, the other one isn't.

Despite the Fact that this should be okay/sufficient with Fx4, I'm asking beforehand if it's useful/worthwhile to file a Follow-Up to get this done properly, i.e. are there yet Post-Fx4-Plans on this Issue?
(In reply to comment #13)
> Hence in the "worst Case" of Safe-Mode: on + Compatibility Check: off + Update
> Security Check: off, only the Safe-Mode State is exposed, there other two are
> not.
>
> With only Compatibility Check: off + Update Security Check: off, you only get
> the Security Check State exposed, the other one isn't.

I would say file a new bug on both items. Bugs are cheap and discussion is better handled on non-fixed bugs.

Marking as verified fixed with Mozilla/5.0 (Macintosh; Intel Mac OS X 10.6; rv:2.0b7pre) Gecko/20100917 Firefox/4.0b7pre

Blair, what should be tested by a manual test?
Status: RESOLVED → VERIFIED
Whiteboard: [rewrite][strings] → [rewrite][strings][AOMTestday][in-litmus-bug-week]
(In reply to comment #13)
> Despite the Fact that this should be okay/sufficient with Fx4, I'm asking
> beforehand if it's useful/worthwhile to file a Follow-Up to get this done
> properly, i.e. are there yet Post-Fx4-Plans on this Issue?

Yea, as Henrik suggests, file a new bug. We'll figure out what, if anything, we can do to improve it post-4.0.

(In reply to comment #14)
> Blair, what should be tested by a manual test?

Only need the safe-mode notification needs manually tested. The other notifications are tested with browser-chrome tests.
Flags: in-litmus? → in-litmus?(vlad.maniac)
Manual test covered by:
https://litmus.mozilla.org/show_test.cgi?id=15315
Flags: in-litmus?(vlad.maniac) → in-litmus+
You need to log in before you can comment on or make changes to this bug.