Closed (fixed)
Project:
Ridiculously Responsive Social Sharing Buttons
Version:
8.x-2.x-dev
Component:
Code
Priority:
Normal
Category:
Feature request
Assigned:
Unassigned
Reporter:
Created:
15 Nov 2016 at 18:30 UTC
Updated:
18 Oct 2017 at 08:00 UTC
Jump to comment: Most recent
Comments
Comment #2
adamps commentedYes, it's a good idea, thanks.
Could even make it so that you could add as many sets of buttons as you want. Each one could be configured separately as follow/share with a choice of buttons. The node form could have an input to choose which set of buttons, and each set of buttons could have a block.
Patches welcome. I've set to v2.x, because new features will likely go in the new branch. Sorry, I haven't written up any instructions for it yet.
Comment #3
himanshu-dixit commented@AdamPS I guess we could implement this feature in the 8.x-2.x branch first. What do you think?
Comment #4
adamps commentedHere's what I have in mind
Comment #5
adamps commented@himanshu-dixit Now would be a good time to start this work. The rest of the code is fairly stable, and it's early enough still that we could avoid the migration code.
**** Please do not check code into 8.x-2.x until it has been fully tested and reviewed ****
You can either create a feature branch in git on Drupal.org or use your own git repository and post a patch for review.
Comment #6
himanshu-dixit commentedAssigning this issue to myself. I would provide the patch soon for this issue (by using my own sperate git repo)
Comment #7
ajayg commentedDo we really need edit? If creating the block requires only checking relevant checkboxes, we can skip edit. If someone wants change they can simply delete and create a new one. Unless I am missing a feature you had in mind with edit. What exactly the user could do in edit? A) change the order B) disable a button C) change the follow or share text. What else ?
I am thinking loudly here. If you create a block(drupal core block) with the button selection set in the order and name that block, that is all you need. Because multiple blocks can be created and assigned to multiple regions. Showing which block to show on which content type is handled by core. Deleting blocks handled by core.
Module will be much simpler. Same block can be used in Panels etc.
Comment #8
ajayg commentedHere is a case that is for providing edit feature.
By default the Google analytics module does not provide/record social share events. These had to be implemented by website owners.
So after each button you can allow to attach a custom input box to send event to google and admin can enter whatever javascript they wish to enter (By default it would be blank.)
ga('send', 'social', 'Facebook', 'like', 'http://myownpersonaldomain.com');More details here
https://developers.google.com/analytics/devguides/collection/analyticsjs...
Comment #9
adamps commented@ajayg edit is definitely required. /admin/config/content/rrssb is a complex form. Deleting and creating again to make a small change is not acceptable, especially as that would likely break any content types and blocks that were using the button set. Everywhere on Drupal you see it: create, edit and delete.
Comment #10
adamps commented@himanshu-dixit Have you had any time to make progress?
I think we are ready to make a RC release now. After that I would still welcome this feature, but there would need to be a hook_update to migrate existing sites. I am happy to wait a few weeks if you that would give you time to complete the code. Let me know - thanks!
Comment #11
adamps commentedI am a site that needs this so have started work.
Comment #13
adamps commentedNB If updatedb fails with "Failed: Incomplete or missing schema for rrssb.button_set.default" please rerun and it should work
I will add this text to the release note when I create the next release.