I’d really like to have a link to the bug on the review page!
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.
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.
(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.
> 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.
*** Bug 46282 has been marked as a duplicate of this bug. ***
Created attachment 68495 [details] Patch
Comment on attachment 68495 [details] Patch OK.
Committed r68118: <http://trac.webkit.org/changeset/68118>
*** Bug 46498 has been marked as a duplicate of this bug. ***