Summary


This PR adds some social packages. See the PR for testing instructions, usage details and other information :)https://github.com/joomla/joomla-cms/pull/1550
Opened On:
17 Jul 2013, 13:41 by George Wilson
Closed On:
23 Aug 2014, 7:52
Status:
Closed

Responses

Posted on 28 Jul 2013, 15:18 by Elin Waring
I like these a lot and they are a good example of how jlayouts give good flexibility/reusabilty in the place they should, a layout file.
Posted on 16 Aug 2013, 16:05 by George Wilson

Ready for testing. Relevent JHtml methods have also been added in.
Posted on 24 Aug 2013, 19:12 by George Wilson

Added better test instructions
Posted on 26 Aug 2013, 17:36 by George Wilson

All methods have been re-written to use JInput removing about 10 lines of code off of each layout - thanks to Janich for the brill idea :)
Posted on 3 Sep 2013, 4:56 by Elin Waring

I'm getiting

Strict Standards: call_user_func_array() expects parameter 1 to be a valid callback, non-static method JHtmlSocial::facebook() should not be called statically in C:\Program Files (x86)\EasyPHP-12.1\www\joomla-cms2\libraries\cms\html\html.php on line 238

Multiple times (but for the different layouts) when pasting as described.
Posted on 3 Sep 2013, 8:32 by George Wilson

Oops. Fixed. My bad!
Posted on 3 Oct 2013, 20:32 by George Wilson

After chatting about this with Elin just now - setting this Ready for Review
Posted on 8 Oct 2013, 13:40 by George Wilson

https://github.com/joomla/joomla-cms/pull/2173 this PR samples the jlayout for com content. You can enable or disable this for any  article (note they are disabled by default)

also Facebook like button will not show on a localhost. It's just the way it goes unfortunately :/
Posted on 23 Aug 2014, 7:52 by Brian Teeman

Thank you for your contribution. We have now moved our tracking process to http://issues.joomla.org/ so please do not update the issue here. The link on the new tracker for this issue is  http://issues.joomla.org/tracker/joomla-cms/1550