Closed (fixed)
Project:
ShareThis
Version:
7.x-2.8
Component:
Miscellaneous
Priority:
Major
Category:
Task
Assigned:
Unassigned
Reporter:
Created:
26 Jul 2011 at 16:57 UTC
Updated:
13 Jul 2014 at 07:10 UTC
Jump to comment: Most recent, Most recent file
Comments
Comment #1
markduce commentedSubscribing...
Comment #2
jvalencia commentedI'm slightly confused, could you outline what you did vs. what you expect to happen?
Thanks!
Comment #3
markduce commentedSorry, I just put 'subscribing' so that when someone answers your question I get a notification as I don't know the answer
Comment #4
wcarpent commentedOn my site, ShareThis buttons will appear on static pages as they would be expected to. However, I have a particular node type which is displayed through panels and I'm unable to display the ShareThis buttons on this node type or through the panels configuration. When I attempt to "add content" to a panel for this node type, I'm unable to add the ShareThis buttons. I would like to be able to put ShareThis buttons into a panel.
Comment #5
alonk27 commentedSubscribing...
Comment #6
alonk27 commentedI find it so hard to believe that no one knows anything about that. How can it be that something like "Share this" would not integrate with "Panels"??????
I'm searching for an answer for two weeks now...
Can anyone help?
Thanks.
Comment #7
jvalencia commentedI'm looking into it and have filed a support request with Panels
Any insight would be appreciated, as I'm not sure how to procede.
Comment #8
alonk27 commentedThank you.
Comment #9
7wonders commentedYou can do it the old fashioned way as a workaround.
1. Go here - http://sharethis.com/publishers/get-sharing-button and pick your button
2. Click get the button
3. Open up your theme html.tpl.php and put in the script in your head.
4. Put the span code in a mini panel custom code.
5. Add mini panel to panel.
Done.
Comment #10
alonk27 commentedYou have no idea how much you helped me. Thank you very much!!!! it is working!!!
Comment #11
courtney.mob commentedsubscribe
Comment #12
jvalencia commented7.2 branch addresses this. Look under miscellaneous when you add to panels, and add the ShareThis block.
Let me know if this works for you.
Comment #13
cwithout commentedThis worked in Panels for me in the 7.x-2.2 release.
But does not work in the current 7.x-2.x-dev.
Comment #14
jvalencia commentedComment #15
Anonymous (not verified) commentedPatch is coming soon to integrate with Panels.
Comment #16
Anonymous (not verified) commentedComment #17
Anonymous (not verified) commentedComment #18
Anonymous (not verified) commentedneeds review :)
Comment #19
gumdrop commentedTried all patches and view 3 patch and still doesn't show in panels
Comment #20
jherencia commented@gumdrop please try this:
The AM patch is to apply with git am.
Comment #21
brunodboPatch in #20 works for me, thanks!
Got a minor whitespace error on line 48 though - not sure how to fix that (when I tried to remove the extra line at the end, I got a 'patch corrupt' error and it wouldn't apply anymore.
Comment #22
jherencia commentedI've improved the pane to be able to configure between:
Comment #23
jansenCreative commentedComment #24
luksakFor a quick workaround you could just embed a view in your panel which displays the ShareThis links. This is how I solved it for the moment.
Comment #25
drupal_was_my_past commented#22 didn't apply cleanly for me, but (when I manually applied the patch) it functionality is great.
Comment #26
jherencia commentedRerolled.
Comment #27
rerooting commentedLine #559 needs a / to open the comment block.
Otherwise, it works great! Thanks! When will this be rolled into a release?
Comment #28
bastos71 commentedHi everybody !
I'm working on a project and I would like to apply the patch. But how could I simply do it ?
Thank you for answers :)
Comment #29
jherencia commentedThank you @rerooting :). Here is a new reroll.
Comment #30
rerooting commented@Bastos71 - you can wget it to the module root folder and do a few things:
patch -p1 < 1230146-29.patchIs the old way which works well when you aren't controlling an individual repo for the module. Otherwise, you *should* be able to use git apply.
I don't know much about the git am method that folks are using for patches here, but I'm sure it's a good approach as well!
This resource has always been indispensable for me: http://drupal.org/patch/apply
Comment #31
rerooting commentedI've used this several times, with the most recent minor tweak it works flawlessly. Is that enough for it to be RTBC? I think so. Would love to see this committed, meanwhile its going into several of my .make files!
Comment #32
Gaofengzzz commentedThanks very much @jherencia for your patch.
After more than two weeks, with no particular objections or further issues on this ticket, I went ahead and got it committed against the 7.x-2.x branch at d511cb7.
I allowed myself to re-roll this patch and mark this issue as fixed for now, but feel free to re-open it, or post a new ticket, at any time if you have any further objections with this patch (we would surely be happy to hear your feedback).
Special thanks to @wcarpent, @Valor, @rerooting for re-implementing, Re-rolling, testing this patch.
Thanks again to everyone for your help, reviews, feedback and comments on this issue.
Cheers!
Comment #33
rerooting commentedThanks for the commit! Just pulled it on a project, it appears I'm getting an AJAX error whenever I try to add the pane. Not sure what it's being caused by (hard to debug), I'll have to let you know what I figure out.
Comment #34
Anonymous (not verified) commentedSo is this patch in the current (July 3, 2013) dev version on the Drupal site or not?
Comment #35
jherencia commentedThis patch is in the dev version of this module (https://drupal.org/project/sharethis) which is 7.x-2.x-dev (2013-Jun-26).
Comment #37
user654 commented.
Comment #38
reallyordinary commentedThis isn't working. Just set up on fresh D7.28 install with latest panels and latest 7.x-2.8 version of ShareThis module. Shows up fine when not using panels. When using a panels node view page, nope. Nothing.
EDIT: I take it back - right, you have to set Location to "block" in ShareThis settings. Then it shows up fine in panels. Apologies.
Comment #39
reallyordinary commented