Bug 46192 – Add link to bug to review page

• Component: WebKit Website
• Status: RESOLVED FIXED

[Darin Adler]

I’d really like to have a link to the bug on the review page!

[Adam Barth]

There are three (small) problems that need to be solved to make that happen:

1) Adding the link at a time when we know the bug_id.  We know the bug_id in the context of this function:
http://trac.webkit.org/browser/trunk/BugsSite/code-review.js#L225

2) Deciding where to display the link.  I think putting it on the toolbar makes sense, but the toolbar is getting a bit crowded.  Maybe we should move the help text elsewhere?

3) The CSS for hyperlinks in PrettyPatch is overbroad, which means it would apply to the bug link too.  We probably need to tighten down the CSS for hyperlinks so the bug link isn't the wrong font/size.

[Adam Barth]

Created attachment 68267 [details]
dev extension

Here's the extension I use to develop the tool.  Basically, it substitutes the existing code-review.js on the live site with the one in the extension.  That way you can hack on the script and see the effects on the live site at the same time.

[Darin Adler]

(In reply to comment #1)
> 1) Adding the link at a time when we know the bug_id.  We know the bug_id in the context of this function:
> http://trac.webkit.org/browser/trunk/BugsSite/code-review.js#L225

Now that I know about this it won’t be too hard.

> 2) Deciding where to display the link.  I think putting it on the toolbar makes sense, but the toolbar is getting a bit crowded.  Maybe we should move the help text elsewhere?
> 
> 3) The CSS for hyperlinks in PrettyPatch is overbroad, which means it would apply to the bug link too.  We probably need to tighten down the CSS for hyperlinks so the bug link isn't the wrong font/size.

When doing the work myself, I used the web inspector to insert the link and it looked fine. For both (2) and (3) above, it was OK the way I had it. I figured others might later want to refine further, but I think just putting the bug before the instructions is fine. And the style for links looked OK to me.

Assuming it’s a Chromium extension, I’ll have to make it work in Safari, but thanks very much for showing me that extension.

[Adam Barth]

> Assuming it’s a Chromium extension, I’ll have to make it work in Safari, but thanks very much for showing me that extension.

Yeah, it's a Chrome extension, but should give you the general idea.

[Adam Barth]

*** Bug 46282 has been marked as a duplicate of this bug. ***

[Adam Barth]

Created attachment 68495 [details]
Patch

[Eric Seidel (no email)]

Comment on attachment 68495 [details]
Patch

OK.

[Adam Barth]

Committed r68118: <http://trac.webkit.org/changeset/68118>

[Alexey Proskuryakov]

*** Bug 46498 has been marked as a duplicate of this bug. ***