View Issue Details

IDProjectCategoryView StatusLast Update
0000286Composrrecommendpublic2011-09-19 12:00
ReporterChris GrahamAssigned ToChris Graham 
SeverityFeature-request 
Status resolvedResolutionfixed 
Product Version 
Fixed in Version 
Summary0000286: Google +1 button
DescriptionAdd to main_screen_actions block
TagsNo tags attached.
Time estimation (hours)1
Sponsorship open


If users wish to sponsor this issue (optional, some issues are implemented without sponsorship)...
If users wish to sponsor this issue (optional, some issues are implemented without sponsorship)...

Activities

Chris Graham

2011-09-19 12:00

administrator   ~0000189

Last edited: 2011-09-19 12:01

View 2 revisions

I've done this with proper valid XHTML (the Google supplied code won't validate).

I've followed the advice regarding placement of the Google script tag, which they advise for reasons of performance (if the script tag was mid-page, it would have a small effect on page rendering speed).
However, it has not been hard-coded into FOOTER.tpl (which would have been a terrible assumption - not all screens/sites will have the button). Instead, I've added a bit of extra syntax to Tempcode to allow a template (BLOCK_MAIN_SCREEN_ACTIONS.tpl) to defer a bit of it's code down to the footer.

Actual placement of the button was almost trivial. It was about 1px too high in the template, so I made a wrapper div. That means ideally (to make things look perfect) this needs adding to global.css:


#screen_actions .google_plusone {
    margin-top: 1px;
}

Issue History

Date Modified Username Field Change