Closed Bug 494118 Opened 16 years ago Closed 16 years ago

Breadcrumbs in the Collections view self-link; should go to the homepage

Categories

(addons.mozilla.org Graveyard :: Public Pages, defect)

Product:
addons.mozilla.org Graveyard
Component:
Public Pages
Platform:
x86
All
Type:
defect
Priority:
Not set
Severity:
normal

Tracking

(Not tracked)

Status:
VERIFIED FIXED

People

(Reporter: stephend, Assigned: jbalogh)

References

(
URL
)

Details

Attachments

(1 file, 1 obsolete file)

backing off on my overzealous deletion
1.74 KB, patch
rdoherty
: review+
Details | Diff | Splinter Review
STR: 1) Load https://preview.addons.mozilla.org/en-US/firefox/collections/popular 2) Look at the link to the breadcrumb of "Add-ons for Firefox", which it thinks is itself: Actual: https://preview.addons.mozilla.org/en-US/firefox/collections/popular I loathe self-linking breadcrumbs. Expected: Expecting this to take me to https://preview.addons.mozilla.org/en-US/firefox
This should also be using amo2009/breadcrumbs.
Assignee: nobody → jbalogh
Attachment #378902 - Flags: review?(rdoherty)
Comment on attachment 378902 [details] [diff] [review] making stephend happy with better breadcrumbs The header for the page was removed. Otherwise patch works.
Attachment #378902 - Flags: review?(rdoherty) → review-
Attachment #378902 - Attachment is obsolete: true
Attachment #379241 - Flags: review?(rdoherty)
Comment on attachment 379241 [details] [diff] [review] backing off on my overzealous deletion w00t!
Attachment #379241 - Flags: review?(rdoherty) → review+
r26127
Status: NEW → RESOLVED
Closed: 16 years ago
Keywords: push-needed
Resolution: --- → FIXED
Verified FIXED; thanks, Jeff!
Status: RESOLVED → VERIFIED
removing "push-needed" from 105 AMO 5.0.6 bugs; filter on "I hate stephend!"
Keywords: push-needed
Product: addons.mozilla.org → addons.mozilla.org Graveyard
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Creator:
Created:
Updated:
Size: