-
-
Notifications
You must be signed in to change notification settings - Fork 3.6k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
[#31516] Start a JLayout Social #1550
Conversation
I love this. Where do you think we should store parameters? |
Just to put up what we're just talking about on skype. Personally a new section where the Component options are set (so Global Config but not actually in Global Config). Called Layouts. Starting with one tab (social). This then gets expanded as more JLayouts are introduced. |
Feel free to borrow code out of https://github.com/mbabker/Yet-Another-Social-Plugin/blob/master/yetanothersocial.php if need be, I've spent a little time figuring out the language implementations. Could be improved on, but it's a good start. |
Hmm what you have is interesting there but is specific to an article - but Having said that I might well hijack those language setting functions you Thanks Michael - actually that's going to be a great help!! On 16 July 2013 01:49, Michael Babker notifications@github.com wrote:
|
@mbabker thanks for the plugin link. It actually gave me some great ideas to refactor the Layouts. Hopefully in a much better position now! |
Have you looked at: Maybe now is too late but I love the idea of only require 3KB for sharing. I'd never use the "old" share buttons but this jQuery plugin is awesome. |
In a word - no :P I did initially for G+ and facebook use the inline javascript. But I was unhappy with the fact you were including it with every use of the button. For example in cases where you might have the fb like button in a module and then again in a blog post. Loading it in the head with JDocument gave me a way out of that. If you can find a way around this though I'm more than happy to consider it. I'd rather get it right now then face the consequences later on! P.S. Also just reading the comments of your link there that plugin breaks on mobiles (well definitely iPhones - I just tested it on mine) and on IE10 (just tested that as well). I get the idea though :) |
Yes. I went to the github project to check issues and the plugin seems abandoned :( About your PR I tested it and all seems to work fine. In my opinion layouts are in the best place and hashtag language worked fine here. So no issues Are Twitter translations really required? I tested it in spanish and worked perfect before creating the language file. |
RE: language files - I'm not sure. Some of them I don't think are. The twitter website puts these things in the link tags https://twitter.com/about/resources/buttons but then just the data-language tag seems to set everything up fine (good to hear the language on the hashtags worked for you!) Either way I still need the file for my two error messages if people don't include a hashtag for the hashtag layout and a user for the follow/mention layout. |
For the code try render(array()); $layout2 = new JLayoutFile('social.twitter.share'); echo $layout2->render(array()); $layout3 = new JLayoutFile('social.facebook'); echo $layout3->render(array()); $layout4 = new JLayoutFile('social.twitter.follow'); echo $layout4->render(array('user'=>'GW1992')); ?>I'd suggest making a super JLayout that then includes all of them in a block. I'd be game for including these default off as a simple on/ff block. I do think we need to think about complex storing of params but for right now we could do it on a per site basis and let webmaster, implementers and developers go to town with more complex. |
Also I was thinking as a default user if one isn't set perhaps we should use the Joomla twitter feed rather than throwing an error? |
Will work on super JLayout block |
Ok as discussed on skype I'm not going to have a master layout. However I am going to put in a JHtml class to call these functions. Just committed these now. |
What is the status with this @wilsonge |
feature still pending awaiting testers. i'll move it across to target staging later |
Fix comment about the default memory limit in daemon.php
@test The PR cannot be applied: |
Also Covers com_categories quotes forgotten
obvious mistake Thanks @smanzi Quotes forgotten
Quotes forgotten
Well somethings very wrong with your git then :P Because none of those files do exist yet. They are supposed to be created by this PR |
Because this is so old it's targeted at master I've created a fresh version targeting staging at #5252 - try that and see how you go. Closing this one as a result |
This starts the process of making some features not already in the CMS available as JLayouts. This starts with:
Tracker: http://joomlacode.org/gf/project/joomla/tracker/?action=TrackerItemEdit&tracker_item_id=31516
Screenshot
Note including them in articles is _NOT_ included in this PR - this is just demonstrating their use
Layout Params
Google+1
Facebook Like
Twitter
Follow
Share
To be written up (see code for now)
Hashtag
To be written up (see code for now)
Mention
To be written up (see code for now)
Issues
Testers
Suggested code: