Closed
Bug 265499
Opened 20 years ago
Closed 20 years ago
include custom stylesheets in header.html.tmpl
Categories
(Bugzilla :: User Interface, defect)
Tracking
()
RESOLVED
FIXED
Bugzilla 2.20
People
(Reporter: myk, Assigned: myk)
Details
Attachments
(1 file)
1.11 KB,
patch
|
gerv
:
review+
|
Details | Diff | Splinter Review |
Bug 256708 makes skins/custom/ for installation-specific custom stylesheets. To make use of this directory, header.html.tmpl needs to include references to those stylesheets in Bugzilla page headers. This is the fourth and final task in the first stage of the Bugzilla CSS plan: * classify pages by installation and CGI: bug 253480; * move existing files to their new locations: bug 256207; * make checksetup.pl create the skins/custom/ hierarchy: bug 256208; * make header.html.tmpl include custom stylesheets: this bug;
Assignee | ||
Comment 1•20 years ago
|
||
Assignee | ||
Comment 2•20 years ago
|
||
Comment on attachment 162886 [details] [diff] [review] patch v1: implements feature This five-line patch adds support for custom stylesheets to Bugzilla. It's the last of the four tasks in round one of the CSS plan that still needs doing. The others are all in 2.20 already. Gerv, any chance you can review this adjunct to the other two you recently reviewed before you head off for a month? Detail: This patch adds skins/custom/<foo>.css references to pages with skins/standard/<foo>.css references. custom/ comes after standard/ so it takes precedence. The patch doesn't support multiple third-party skins as alternate stylesheets, and it leaves the standard stylesheets as "persistent" stylesheets (applied always no matter what alternate stylesheet has been selected) since the standard stylesheets only need to become preferred (applied by default, but not applied when the user selects an alternate skin) once we support multiple third-party skins as alternate stylesheets. In other words, this patch is simple, but things will become more complex in the future.
Attachment #162886 -
Flags: review?(gerv)
Comment 3•20 years ago
|
||
Comment on attachment 162886 [details] [diff] [review] patch v1: implements feature r=gerv. Like the implementation. Gerv
Attachment #162886 -
Flags: review?(gerv) → review+
Assignee | ||
Comment 4•20 years ago
|
||
Thanks Gerv! Fix checked in, and stage one of CSS plan is complete. Checking in template/en/default/global/header.html.tmpl; /cvsroot/mozilla/webtools/bugzilla/template/en/default/global/header.html.tmpl,v <-- header.html.tmpl new revision: 1.30; previous revision: 1.29 done
Flags: approval+
Target Milestone: --- → Bugzilla 2.20
Assignee | ||
Updated•20 years ago
|
Status: NEW → RESOLVED
Closed: 20 years ago
Resolution: --- → FIXED
Updated•11 years ago
|
QA Contact: matty_is_a_geek → default-qa
You need to log in
before you can comment on or make changes to this bug.
Description
•