Closed
Bug 163541
Opened 22 years ago
Closed 22 years ago
header.html.tmpl should set h1 be the empty string
Categories
(Bugzilla :: Bugzilla-General, defect)
Tracking
()
RESOLVED
FIXED
Bugzilla 2.18
People
(Reporter: myk, Assigned: myk)
Details
Attachments
(2 files)
1.10 KB,
patch
|
bbaetz
:
review+
bbaetz
:
review+
|
Details | Diff | Splinter Review |
793 bytes,
patch
|
bbaetz
:
review+
bbaetz
:
review+
|
Details | Diff | Splinter Review |
header.html.tmpl currently doesn't let the primary page header (h1) be the empty string, because the DEFAULT directive thinks "" == undefined and sets h1 = title in that case. This causes problems with, f.e. setting a title on the custom b.m.o index page without that title appearing as the primary page header. header.html.tmpl should let the calling template specify that h1 is empty.
Assignee | ||
Comment 1•22 years ago
|
||
This patch handles h1 outside the DEFAULT tag to avoid DEFAULT doing the wrong thing.
Assignee | ||
Comment 2•22 years ago
|
||
This patch has been applied to b.m.o.
Comment 3•22 years ago
|
||
Is this behaviour as designed, or a bug? And if the latter, have we reported it? Gerv
Comment 4•22 years ago
|
||
Comment on attachment 95916 [details] [diff] [review] patch v1: fixes problem looks reasonable r=bbaetz x2
Attachment #95916 -
Flags: review+
Assignee | ||
Comment 5•22 years ago
|
||
This is behavior as designed. Checking in header.html.tmpl; /cvsroot/mozilla/webtools/bugzilla/template/en/default/global/header.html.tmpl,v <-- header.html.tmpl new revision: 1.16; previous revision: 1.15 done
Status: NEW → RESOLVED
Closed: 22 years ago
Resolution: --- → FIXED
Assignee | ||
Comment 6•22 years ago
|
||
Reopening. Something strange is going on. On my local install this patch works fine, but on b.m.o. it makes primary headers stop appearing. Changing the conditional from the shortcut form to the classic form makes it work: [% h1 = title IF !h1.defined %] --> [% IF !h1.defined %][% h1 = title %][% END %]
Status: RESOLVED → REOPENED
Resolution: FIXED → ---
Assignee | ||
Comment 7•22 years ago
|
||
Comment 8•22 years ago
|
||
Comment on attachment 95997 [details] [diff] [review] patch v2: changes conditional structure Really odd. r=bbaetz x2
Attachment #95997 -
Flags: review+
Assignee | ||
Comment 9•22 years ago
|
||
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.17; previous revision: 1.16 done
Status: REOPENED → RESOLVED
Closed: 22 years ago → 22 years ago
Resolution: --- → FIXED
Updated•21 years ago
|
Target Milestone: --- → Bugzilla 2.18
![]() |
||
Comment 10•18 years ago
|
||
(In reply to comment #3) > Is this behaviour as designed, or a bug? And if the latter, have we reported > it? > > Gerv > Was there ever a response to this?
Updated•18 years ago
|
Assignee: justdave → myk
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
•