Show "install" dialog for single extension updates as well as multiple

RESOLVED FIXED

Status

()

Toolkit
Add-ons Manager
RESOLVED FIXED
13 years ago
10 years ago

People

(Reporter: Ben Basson, Assigned: rstrong)

Tracking

Trunk
Points:
---
Bug Flags:
blocking1.8b5 -

Firefox Tracking Flags

(Not tracked)

Details

(Whiteboard: [unknown l10n impact], URL)

Attachments

(1 attachment, 1 obsolete attachment)

4.31 KB, patch
Benjamin Smedberg
: review+
Benjamin Smedberg
: approval1.8b4+
Details | Diff | Splinter Review
(Reporter)

Description

13 years ago
If multiple extension updates are found by the EM (when manually checking with
"Find Updates") a dialog is presented which lists them prompts the user to install. 

Currently, a single update is displayed only with the red arrow and the "Update
Now" button. The problem with this situation is that the single extension could
be out of the current view, requiring the user to scroll before they realise
that an update is available. 

To work around this, the same dialog that is launched for multiple updates
should be used for single updates, when manually checking with the "Find
Updates" button.

Updated

13 years ago
Flags: blocking1.8b4?

Updated

13 years ago
(Reporter)

Comment 1

13 years ago
Asa, this might affect l10n if Rob is adding a progress indicator with new
strings as part of this patch (if at all).
(Reporter)

Updated

13 years ago
Whiteboard: [unknown l10n impact]
Created attachment 191142 [details] [diff] [review]
patch (fixes more than this bug)

The recent update changes have caused a couple of issues which this patch
addresses by restoring the previous wizard when checking for updates to more
than one item. The issues are:
1) This bug. By using the wizard when checking updates for multiple items the
wizard will show there is an update and give the user the option to install it
whereas the current behavior is if only one update is available and the item is
not in view the user will not see any indication that there is an update
available.
2) bug 302273. This restores the old behavior when checking for updates from
options.
3) Thunderbird extension update from options is broken. This restores the old
behavior when checking for updates from options.
4) On upgrade _upgradeThemeChrome can be skipped in _finishOperations.
5) What I believe caused the upgradeExtensionChrome failure in bug 302527. If
an item no longer exists in the file system during an upgrade then the metadata
needs to be removed.
6) Clicking finish on the update wizard during an upgrade fails due to a
constant name change (e.g. PREF_EXTENSIONS_UPDATE_ENABLED was changed to
PREF_UPDATE_EXTENSIONS_ENABLED).
7) When performing an update the update commands are enabled.
Attachment #191142 - Flags: review?(benjamin)
Comment on attachment 191142 [details] [diff] [review]
patch (fixes more than this bug)

Since there is no additional review I'm asking Ben for sr due to this restoring
some of the recent changes he made.
Attachment #191142 - Flags: superreview?(bugs)

Updated

13 years ago
Attachment #191142 - Flags: review?(benjamin) → review+
No, don't restore the old wizard. If there are bugs with the current impl we
need to fix those, rather than backing away.

I think the extensions window is probably missing a call to
ensureElementIsVisible or something. 
Attachment #191142 - Flags: superreview?(bugs) → superreview-
(Reporter)

Comment 5

13 years ago
Ben, if it helps, I listed the problems I have with the new system at the thread
in the URL field above.
Created attachment 191434 [details] [diff] [review]
patch

Thanks Ben.

This addresses this bug by using scrollToElement... this will bring the item
that has an update into view and make it the first viewable item in the list
which is more noticeable than using ensureElementIsVisible.

I removed the calls to LOG due to errors these cause when closing the EM during
a find updates. Also, the commands are now disabled during an update.
Attachment #191142 - Attachment is obsolete: true
Attachment #191434 - Flags: review?(benjamin)

Updated

13 years ago
Attachment #191434 - Flags: review?(benjamin)
Attachment #191434 - Flags: review+
Attachment #191434 - Flags: approval1.8b4+
Fixed on trunk

Checking in mozilla/toolkit/mozapps/extensions/content/extensions.js;
/cvsroot/mozilla/toolkit/mozapps/extensions/content/extensions.js,v  <-- 
extensions.js
new revision: 1.71; previous revision: 1.70

Note: there is also bug 303191 regarding displaying a progress indicator
Status: NEW → RESOLVED
Last Resolved: 13 years ago
Resolution: --- → FIXED
Product: Firefox → Toolkit
You need to log in before you can comment on or make changes to this bug.