add-ons OK message when no networking availabe, yields never ending ok loops

VERIFIED FIXED in Firefox 9

Status

defect
VERIFIED FIXED
10 years ago
8 years ago

People

(Reporter: jmaher, Assigned: murshyam)

Tracking

({mobile})

Firefox Tracking Flags

(Not tracked)

Details

Attachments

(1 attachment, 1 obsolete attachment)

on my n810, I turn off my internet connection and go to the add-ons manager.  I pan down to "Get Add-ons" and there is a message "Fennec couldn't retrieve add-ons" with and "ok" button.  I click ok, it does a search (<1 second time) and gives me back my same OK button.  Not a very good UX.

This is tested on my n810 with 20090924 1.9.2 nightly build.
What UX were you expecting? What's so bad about the current behavior?
an OK button indicates you understand the error and are dismissing it.  It is just that this comes back an haunts you, it never really gets dismissed.
Should we use "Retry" instead of "Ok"?
I see the same issue with the "Cancel" button. When my internet is slow, it keeps trying to search for add-ons. So when I get tired of that, I click the "Cancel" button, but I can click as I want on that button, but the search for add-ons just goes on, which makes me think that "Cancel" button doesn't do anything.
Assignee: nobody → mbrubeck
Status: NEW → ASSIGNED
Component: Linux/Maemo → General
OS: Linux (embedded) → All
QA Contact: maemo-linux → general
Hardware: ARM → All
Status: ASSIGNED → NEW
Whiteboard: [mentor=mbrubeck@mozilla.com][good first bug]
Madhava - thoughts?
Assignee: mbrubeck → nobody
Keywords: helpwanted, mobile
Hi Matt,

I would like to work on this bug but I would require total guidance as this would be my first bug. :)

Also, I did not know how to send you this request (whether by mail or some other portal), that is why I am commenting it directly on this bug.

Thanks.
(In reply to Shyam Prasad Murarka from comment #6)
> I would like to work on this bug but I would require total guidance as this
> would be my first bug. :)

Okay.  The first steps are to build Fennec from source code and make sure you can reproduce this bug.  Then you can start looking at and changing code.  Instructions at:
https://wiki.mozilla.org/Mobile/Fennec

Here's some of the relevant code:
https://mxr.mozilla.org/mozilla-central/source/mobile/chrome/content/extensions.js#859

See Mark's suggestion in comment 3 for a simple change to make.

> Also, I did not know how to send you this request (whether by mail or some
> other portal), that is why I am commenting it directly on this bug.

You can comment in this bug for any questions or comments directly related to it.  If you have other questions (e.g. general questions about how Fennec works or getting it to build), please email me directly, find me on #mobile at irc.mozilla.org, or post in the newsgroup: https://groups.google.com/forum/#!forum/mozilla.dev.platforms.mobile

Good luck, and please don't hesitate to ask questions!
Assignee: nobody → murshyam
Keywords: helpwanted
Whiteboard: [mentor=mbrubeck@mozilla.com][good first bug]
Attachment #559758 - Flags: review+ → review?(mbrubeck)
Comment on attachment 559758 [details] [diff] [review]
When no network is available in Addons, display "Retry" message instead of "OK"

Hi, thanks for the patch!  Could you change just one thing, please?  We need to change the string ID so localizers will update their translations.  (You can just change it to "addonsSearchFail2.button" or something.)
Attachment #559758 - Flags: review?(mbrubeck) → review-
Comment on attachment 559764 [details] [diff] [review]
When no network is available in Addons, display "Retry" message instead of "OK"

Looks good!

> addonsSearchFail.button=OK
>+addonsSearchFail.retryButton=Retry

You can delete the old "OK" string here.  I changed this myself and then pushed your patch to inbound.  It will be merged to mozilla-central in included in nightly builds within a day or two:

http://hg.mozilla.org/integration/mozilla-inbound/rev/7b9267933bb4
Attachment #559764 - Flags: review?(mbrubeck) → review+
Status: NEW → ASSIGNED
Whiteboard: [inbound]
http://hg.mozilla.org/mozilla-central/rev/7b9267933bb4
Status: ASSIGNED → RESOLVED
Closed: 8 years ago
Resolution: --- → FIXED
Whiteboard: [inbound]
Target Milestone: --- → Firefox 9
hey guys, this patch will be invisible for our localizers (they will not be able to see that the entity value changed).

Could you please assing a new entity ID for the entity that receives a value change?
(In reply to Zbigniew Braniecki [:gandalf] from comment #13)
> Could you please assing a new entity ID for the entity that receives a value
> change?

Unless I'm missing something, the patch *did* assign a new entity ID.
Note that the final patch as checked in is slightly different from either version attached to the bug (it merges both attachments and also fixes the nit from comment 11):
http://hg.mozilla.org/mozilla-central/rev/7b9267933bb4
Verified fixed on:
Mozilla/5.0 (Android;Linux armv7l;rv:9.0a1)Gecko/20110926
Firefox/9.0a1 Fennec/9.0a1
Status: RESOLVED → VERIFIED
You need to log in before you can comment on or make changes to this bug.