Closed
Bug 851591
Opened 12 years ago
Closed 11 years ago
Add a main-end hook into global/footer.html.tmpl to be able to display additional data before displaying the page footer
Categories
(Bugzilla :: Extensions, enhancement)
Tracking
()
RESOLVED
FIXED
Bugzilla 4.4
People
(Reporter: LpSolit, Assigned: LpSolit)
Details
Attachments
(1 file)
1.62 KB,
patch
|
dkl
:
review+
|
Details | Diff | Splinter Review |
The current hooks in global/footer.html.tmpl all affect the page footer itself but do not let a developer add stuff before the page footer is displayed, see e.g. bug 842045 and bug 842159. I add a 'main-end' hook before the page footer and also add a new template to the Example extension to show how to get the name of the template being processed (I use the existing 'end' hook for that, to not clutter the page itself).
Attachment #725506 -
Flags: review?(timello)
Assignee | ||
Comment 1•12 years ago
|
||
Comment on attachment 725506 [details] [diff] [review]
patch, v1
timello doesn't seem to be around anymore.
Attachment #725506 -
Flags: review?(dkl)
Comment 2•11 years ago
|
||
Comment on attachment 725506 [details] [diff] [review]
patch, v1
Review of attachment 725506 [details] [diff] [review]:
-----------------------------------------------------------------
I think you meant to name the example template hook extensions/Example/template/en/default/hook/global/footer-main-end.html.tmpl instead. Fix on commit. r=dkl
Attachment #725506 -
Flags: review?(dkl) → review+
Updated•11 years ago
|
Flags: approval?
Flags: approval4.4?
Assignee | ||
Comment 3•11 years ago
|
||
(In reply to David Lawrence [:dkl] from comment #2)
> I think you meant to name the example template hook
> extensions/Example/template/en/default/hook/global/footer-main-end.html.tmpl
> instead.
No. As I said in comment 0, I don't want to display data in the page itself, but only below the page footer.
Assignee | ||
Updated•11 years ago
|
Attachment #725506 -
Flags: review?(timello)
Assignee | ||
Updated•11 years ago
|
Flags: approval?
Flags: approval4.4?
Flags: approval4.4+
Flags: approval+
Assignee | ||
Comment 4•11 years ago
|
||
Committing to: bzr+ssh://lpsolit%40gmail.com@bzr.mozilla.org/bugzilla/trunk/
added extensions/Example/template/en/default/hook/global/footer-end.html.tmpl
modified template/en/default/global/footer.html.tmpl
Committed revision 8614.
Committing to: bzr+ssh://lpsolit%40gmail.com@bzr.mozilla.org/bugzilla/4.4/
added extensions/Example/template/en/default/hook/global/footer-end.html.tmpl
modified template/en/default/global/footer.html.tmpl
Committed revision 8547.
Status: ASSIGNED → RESOLVED
Closed: 11 years ago
Resolution: --- → FIXED
Comment 5•11 years ago
|
||
(In reply to Frédéric Buclin from comment #3)
> (In reply to David Lawrence [:dkl] from comment #2)
> > I think you meant to name the example template hook
> > extensions/Example/template/en/default/hook/global/footer-main-end.html.tmpl
> > instead.
>
> No. As I said in comment 0, I don't want to display data in the page itself,
> but only below the page footer.
Ah ok. Missed that point in comment 0. I tried the template in both locations and it worked all the same.
dkl
You need to log in
before you can comment on or make changes to this bug.
Description
•