Closed
Bug 163541
Opened 23 years ago
Closed 23 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•23 years ago
|
||
This patch handles h1 outside the DEFAULT tag to avoid DEFAULT doing the wrong
thing.
Assignee | ||
Comment 2•23 years ago
|
||
This patch has been applied to b.m.o.
Comment 3•23 years ago
|
||
Is this behaviour as designed, or a bug? And if the latter, have we reported it?
Gerv
Comment 4•23 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•23 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: 23 years ago
Resolution: --- → FIXED
Assignee | ||
Comment 6•23 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•23 years ago
|
||
Comment 8•23 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•23 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: 23 years ago → 23 years ago
Resolution: --- → FIXED
Updated•23 years ago
|
Target Milestone: --- → Bugzilla 2.18
![]() |
||
Comment 10•20 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•19 years ago
|
Assignee: justdave → myk
Updated•13 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
•