In-page links are broken due to <base href> added during Mojo migration

RESOLVED FIXED

Status

()

RESOLVED FIXED
6 months ago
15 days ago

People

(Reporter: mats, Assigned: kohei)

Tracking

({regression})

Production
regression

Firefox Tracking Flags

(Not tracked)

Details

Attachments

(1 attachment)

(Reporter)

Description

6 months ago
STR
1. load https://bugzilla.mozilla.org/attachment.cgi?id=618051&action=diff
2. click on of the "Link Here" links 

ACTUAL RESULTS
https://bugzilla.mozilla.org/#a/layout/generic/nsFrame.cpp_sec3
is loaded, which is not at all a link to the relevant section
of the patch.
This is due to `<base href="https://bugzilla.mozilla.org/">` in the page `<head>` added during the Mojo migration (Bug 1455495). :dylan, do we need this now?

https://github.com/mozilla-bteam/bmo/blob/master/template/en/default/global/header.html.tmpl#L101
Flags: needinfo?(dylan)
Keywords: regression
Links on this pages are also broken: https://bugzilla.mozilla.org/page.cgi?id=quicksearch.html
Summary: "Link Here" links in patches are broken → In-page links are broken due to <base href> added during Mojo migration
Posted file GitHub Pull Request
I think it was originally added for Bug 1468848, which was backed out shortly. Should be safe to remove it.
Assignee: nobody → kohei.yoshino
Status: NEW → ASSIGNED
Flags: needinfo?(dylan)
Merged to master.
Status: ASSIGNED → RESOLVED
Last Resolved: 6 months ago
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.