Closed Bug 652362 Opened 9 years ago Closed 9 years ago

Add <link rel="prev|next|..."> elements to the show_bug.cgi page from query

Categories

(bugzilla.mozilla.org :: User Interface, defect)

Production
defect
Not set
normal

Tracking

()

VERIFIED FIXED

People

(Reporter: mcepl, Assigned: dkl)

References

Details

Attachments

(1 file)

Seems like resolution from bug 87818 got lost somewhere, there are no <link> elements allowing browsing through the query list.
<link> is gone since Bugzilla 3.6, see bug 509108.
(In reply to comment #1)
> <link> is gone since Bugzilla 3.6, see bug 509108.

I think it is sad, that this is not provided by Bugzilla anymore. Sure, <link> elements didn't take the Internet by the storm (to say it mildly), but extensions supporting it are trivial to write (e.g., mine https://addons.mozilla.org/cs/firefox/addon/rodentkiller/) and it is very useful for heavy bugzilla users.
(In reply to comment #2)
> (In reply to comment #1)
> > <link> is gone since Bugzilla 3.6, see bug 509108.
> 
> I think it is sad, [...]

Well, so do I, but what is gone is gone I suppose. <shrug /> With the GoToTop extension I can still scroll to the equivalent <a href=...> links at top and bottom of a bug's content.

FTR, and contrary to what is said in bug 509108 comment #1, Mozilla SeaMonkey has (even in the latest trunk builds) a "Website Navigation Bar" in chrome, with "Top Up First Previous Next Last Document More Subscribe" which does indeed "do something useful" with <link> tags that may be present on a page.
Assignee: nobody → dkl
Status: NEW → ASSIGNED
Component: Bugzilla: Other b.m.o Issues → User Interface
OS: Linux → All
Product: mozilla.org → bugzilla.mozilla.org
QA Contact: general → ui
Hardware: x86_64 → All
Version: other → Current
I admit that these should not have been removed as they are useful for those who need them. This patch adds them back using the new 4.0 method for retrieving last search results.

dkl
Attachment #528937 - Flags: review?(glob)
If it works and doesn't hurt performance, are there reasons not to take this upstream?
IMO, global/site-navigation.html.tmpl is not the right place for this code as it's only relevant when viewing a bug. So bug/show-header.html.tmpl would be a much better place, I think. @Marc: for now, we don't plan to take it back.
In my understanding <link> elements are best in <head> and I am already checking for the bug object before rendering so it should not display for every page.

I have moved it to the BMO extension anyway so I am going to close this out.

Committing to: bzr+ssh://dlawrence%40mozilla.com@bzr.mozilla.org/bmo/4.0
modified extensions/BMO/template/en/default/hook/global/header-additional_header.html.tmpl
Committed revision 7646.
Status: ASSIGNED → RESOLVED
Closed: 9 years ago
Resolution: --- → FIXED
Attachment #528937 - Flags: review?(glob)
Site navigation by means of the SeaMonkey toolbar (cf. comment #3) now works on BMO bug pages once a search has been run. => VERIFIED.
Status: RESOLVED → VERIFIED
You need to log in before you can comment on or make changes to this bug.