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)

enhancement
Not set
normal

Tracking

()

RESOLVED FIXED
Bugzilla 4.4

People

(Reporter: LpSolit, Assigned: LpSolit)

Details

Attachments

(1 file)

Attached patch patch, v1Splinter 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)
Comment on attachment 725506 [details] [diff] [review] patch, v1 timello doesn't seem to be around anymore.
Attachment #725506 - Flags: review?(dkl)
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+
Flags: approval?
Flags: approval4.4?
(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.
Attachment #725506 - Flags: review?(timello)
Flags: approval?
Flags: approval4.4?
Flags: approval4.4+
Flags: approval+
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
(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.

Attachment

General

Created:
Updated:
Size: