Templates have hard-coded paths to stylesheets.

[Ryan J Ollos]

For example, peerReviewBrowser.html contains:

<link rel="stylesheet" href="/trac-mysql/chrome/common/css/code.css" type="text/css" media="screen, projector, print" />
<link rel="stylesheet" href="/trac-mysql/chrome/common/css/browser.css" type="text/css" media="screen, projector, print" />

[Ryan J Ollos]

The templates that should have these hard-coded link elements removed from head are: peerReviewBrowser.html, peerReviewCommentCallback.html, peerReviewMain.html, peerReviewOptions.html, peerReviewSearch.html and peerReviewView.html. If those stylesheets are needed in those templates, they should be added using a call to add_stylesheet. I haven't looked as to whether that is already done, and whether the stylesheets are being added everywhere they are needed.

If someone else would like to pickup this ticket and resolved the issue, feel free to do so.

[Olemis Lang]

Thanks rjollos . I think this is a very good starting point for our work . Considering your previous I'll be assigning this ticket to extorue . Do not hesitate to change anything is there's a good reason for doing so .

[Ryan J Ollos]

#10731 and #10782 may be good starting points for getting familiar with the codebase as well, and I wouldn't mind at all if someone beat me to implementing those ;)

[Ryan J Ollos]

In 13502:

Add stylesheets using add_stylesheet. Fixes #10729.

This fixes some incorrect hard-coded paths in link elements in the templates.

[Ryan J Ollos]

[13502#file0] is an unintended change, but I'm not going to worry about it since it's generally harmless and the branch won't be maintained.