Closed Bug 562925 Opened 14 years ago Closed 6 years ago

Check for Updates link in details view missed visual feedback

Categories

(Toolkit :: Add-ons Manager, defect, P2)

defect

Tracking

()

RESOLVED INACTIVE
Tracking Status
blocking2.0 --- .x+

People

(Reporter: tchung, Unassigned)

References

Details

(Keywords: ux-userfeedback, Whiteboard: [AddonsRewriteTestday][rewrite][softblocker][wanted fx5])

Attachments

(1 file, 2 obsolete files)

The check for updates button within the extension details needs to return some sort of alert or notification when clicked on.   If there is no updates returned, at least show the user a popup or something.   Currently, clicking it does nothing, and does not intuitively tell the user if this functionality is working or not.  

1) install nightly Mozilla/5.0 (Macintosh; U; Intel Mac OS X 10.6; en-US;
rv:1.9.3a5pre) Gecko/20100430 Minefield/3.7a5pre
2) install any up to date extension (eg. weave or quick fox notes)
3) restart, and double click your installed extension
4) uncheck "Update automatically" checkbox
5) click the "Check for Updates" button below it
6) Verify there is no user feedback or notification that something has happened.   If there is no updates for real, then at least show something to the user.

Expected:
- some kind of notification if no updates are returned when clicking the "check for updates" button

Actual:
- clicking it shows nothing, and can be misled to be a broken functionality.
Blocks: 550048
Looks like this is actually busted behavior.   I had an installed addon that i know has an update (installed weave 1.2), and clicking the button should have given me an update (weave 1.2.3).   instead, no response on click.

Changing title to reflect the new broken behavior.
Summary: Check for Updates button needs to provide some alert or notification → Check for Updates button in Addon Details does not work
The update button is a placeholder at the moment. There is no action behind it. This bug should be fixed for bug 562902.
Blocks: 562902
Priority: -- → P2
blocking2.0: --- → beta1+
blocking2.0: beta1+ → final+
Assignee: nobody → bmcbride
(In reply to comment #2)
> The update button is a placeholder at the moment. There is no action behind it.

To clarify: Clicking the button should indeed check for (and install) updates. But there is currently no feedback that anything is happening.
(In reply to comment #4)
> To clarify: Clicking the button should indeed check for (and install) updates.
> But there is currently no feedback that anything is happening.

This is still a problem which needs to be solved. What should happen when an user clicks onto the link? Shall we show the "Check for updates" text in the header?
Keywords: uiwanted
Summary: Check for Updates button in Addon Details does not work → Check for Updates link in details view missed visual feedback
(In reply to comment #5)
> This is still a problem which needs to be solved. What should happen when an
> user clicks onto the link? Shall we show the "Check for updates" text in the
> header?

It should do what the list view does, which is:
* Show "Checking for updates"
* If there's an update found, and its set to update automatically, show the download status
* If there's an update found, and its set to update manually, show an "Update now" button

Its just a matter of hooking all this up. We already have all the strings, etc.
Keywords: uiwanted
Assignee: bmcbride → nobody
Assignee: nobody → dietrich
Attached patch wip (obsolete) — Splinter Review
basic stuff in place. the no-update scenario works. need to handle the complete update scenario still. the test is just a copy that i've started to modify, nothing to look at yet.

Dave: i'm new to this code - can you take a peek to see if i'm on the right path here?
Attachment #501654 - Flags: feedback?
Attachment #501654 - Flags: feedback? → feedback?(dtownsend)
Comment on attachment 501654 [details] [diff] [review]
wip

Deferring to Blair as he'll probably be reviewing this one anyway
Attachment #501654 - Flags: feedback?(dtownsend) → feedback?(bmcbride)
Comment on attachment 501654 [details] [diff] [review]
wip

Yep, looks like that'll work nicely.
Attachment #501654 - Flags: feedback?(bmcbride) → feedback+
Whiteboard: [AddonsRewriteTestday][rewrite] → [AddonsRewriteTestday][rewrite][soft blocker]
Whiteboard: [AddonsRewriteTestday][rewrite][soft blocker] → [AddonsRewriteTestday][rewrite][softblocker]
Attached patch more (obsolete) — Splinter Review
Attachment #501654 - Attachment is obsolete: true
Attached patch moreSplinter Review
Attachment #505028 - Attachment is obsolete: true
** PRODUCT DRIVERS PLEASE NOTE **

This bug is one of 7 automatically changed from blocking2.0:final+ to blocking2.0:.x during the endgame of Firefox 4 for the following reasons:

 - it was marked as a soft blocking issue without a requirement for beta coverage
blocking2.0: final+ → .x+
Not working on this. Someone with more experience in this could get it all hooked up the rest of the way far faster than I.

(Also, there are a LOT of moving pieces in the AOM UI code. Would be great to kill all the XBL extra dead, and just use JS. Would really simplify things.)
Assignee: dietrich → nobody
Whiteboard: [AddonsRewriteTestday][rewrite][softblocker] → [AddonsRewriteTestday][rewrite][softblocker][wanted fx5]
Per policy at https://wiki.mozilla.org/Bug_Triage/Projects/Bug_Handling/Bug_Husbandry#Inactive_Bugs. If this bug is not an enhancement request or a bug not present in a supported release of Firefox, then it may be reopened.
Status: NEW → RESOLVED
Closed: 6 years ago
Resolution: --- → INACTIVE
You need to log in before you can comment on or make changes to this bug.