Closed
Bug 649206
Opened 13 years ago
Closed 13 years ago
clicking on recommended addon from about:home should highlight and ensure that addon is visible
Categories
(Firefox for Android Graveyard :: General, defect)
Tracking
(fennec-)
VERIFIED
FIXED
Tracking | Status | |
---|---|---|
fennec | - | --- |
People
(Reporter: blassey, Assigned: wesj)
Details
(Whiteboard: [fennec-6])
Attachments
(1 file)
5.16 KB,
patch
|
mfinkle
:
review+
|
Details | Diff | Splinter Review |
currently if you click on a recommended addon it takes you to the addons panel, but not to the addon you clicked on. If you have more than ~8 addons the one you clicked on isn't even visible.
Updated•13 years ago
|
tracking-fennec: ? → 2.0-
Whiteboard: [fennec-6]
Assignee | ||
Comment 2•13 years ago
|
||
Apparently this broke when bug 594795 landed, and changed the number of arguments to this function. I'm not sure when the return from appendSearchResults changed, but I changed it again later. I started a test for this. Will try and get it done after we get green on the current ones.
Attachment #527801 -
Flags: review?(mark.finkle)
Updated•13 years ago
|
Attachment #527801 -
Attachment is patch: true
Attachment #527801 -
Attachment mime type: application/octet-stream → text/plain
Comment 3•13 years ago
|
||
Comment on attachment 527801 [details] [diff] [review] Patch v1 Oh boy. I can now see the two different changes that broke this. I think I may have added them both. Yes, I test would be excellent.
Attachment #527801 -
Flags: review?(mark.finkle) → review+
Updated•13 years ago
|
Flags: in-testsuite?
Assignee | ||
Comment 4•13 years ago
|
||
http://hg.mozilla.org/mozilla-central/rev/e8de79122e19
Status: NEW → RESOLVED
Closed: 13 years ago
Resolution: --- → FIXED
Comment 5•13 years ago
|
||
VERIFIED FIXED on: Build Id: Mozilla /5.0 (Android;Linux armv7l;rv:6.0a1) Gecko/20110427 Firefox/6.0a1 Fennec /6.0a1 Device: HTC Desire Z (Android 2.2)
Status: RESOLVED → VERIFIED
Updated•10 years ago
|
Flags: in-testsuite?
You need to log in
before you can comment on or make changes to this bug.
Description
•