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)

x86_64
Linux
defect
Not set
normal

Tracking

(fennec-)

VERIFIED FIXED
Tracking Status
fennec - ---

People

(Reporter: blassey, Assigned: wesj)

Details

(Whiteboard: [fennec-6])

Attachments

(1 file)

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.
tracking-fennec: ? → 2.0-
Whiteboard: [fennec-6]
This used to work
Assignee: nobody → wjohnston
Attached patch Patch v1Splinter Review
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)
Attachment #527801 - Attachment is patch: true
Attachment #527801 - Attachment mime type: application/octet-stream → text/plain
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+
Flags: in-testsuite?
http://hg.mozilla.org/mozilla-central/rev/e8de79122e19
Status: NEW → RESOLVED
Closed: 13 years ago
Resolution: --- → FIXED
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
Flags: in-testsuite?
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: