Comments

jmolivas’s picture

This is a small update to the Drupal follow module in order to support Google+ network, to see a demo of this in action just take a look at this blog http://jmolivas.com on the block "Follow me on"

Github
https://github.com/jmolivas/follow

Orignal post entry on my blog
http://jmolivas.com/drupal-follow-google-plus

rwinikates’s picture

Any chance of backporting this to the 6.x branch?

EDIT: I did this locally and will work on packaging it up to contribute back in a bit.

yareckon’s picture

yo, could you provide this as a patchfile to 7.x-dev so that those of us who want your code don't have to work off of the github version?

yareckon’s picture

StatusFileSize
new1.15 KB

I guess I can do that too :)

yareckon’s picture

StatusFileSize
new4.05 KB
new5.03 KB

guess folks need the icons for the icons folders too.. There's not a really good way to maintain a patch including binary files is there. Your use to github starts to make sense...

AlexisWilke’s picture

Status: Active » Needs review
StatusFileSize
new4.74 KB
new4.02 KB
new1.04 KB

There is my own patch for 6.x

The icons are a real problem as usual... 8-(

The first is the small icon, the second is the large one. The name is wrong in comparison to the patch, it should be:

  icon-googleplus.png

Thank you.
Alexis Wilke

AlexisWilke’s picture

Oh! And I added the attribute rel="me" as mentioned in this video:

http://www.youtube.com/watch?v=FgFb6Y-UJUI&feature=digest

I'm not too sure, but I guess the rel="me" should not be added if we're creating a link for the site (versus a link for the user.) I guess a test is necessary to know whether that attribute should be added or not. What do you think?

Thank you.
Alexis Wilke

metakel’s picture

Is it possible to add service on our own?

Sina Weibo (the Mainland China adoption of Twitter) is very popular in the Chinese population. Some clients requests to add this on their sites.

q0rban’s picture

Hi metakel!

First thing to note, you should search the issue queue before posting, and create a separate issue if you can't find what you're looking for. This thread is solely about adding Google +.

That said, please see the follow.api.php file inside the Follow directory for details on how to add custom networks. Also see #640084: Custom networks. Thanks!

sirkitree’s picture

Hrm, looks like Google Plus changed their icon to these:

64:
Only local images are allowed.

32:
Only local images are allowed.

16:
Only local images are allowed.

See here: https://developers.google.com/+/plugins/badge/config

ericduran’s picture

StatusFileSize
new1.51 KB

And here you go.

An updated patches, also cleaned up the whitespace, use the new images, and fix the readme.txt

ericduran’s picture

StatusFileSize
new1.51 KB

A better one.

ericduran’s picture

And now here's a proper git format-patch with the binary images in them.

sirkitree’s picture

Status: Needs review » Reviewed & tested by the community

Works! Using it on Lullabot.com ;)

sirkitree’s picture

Also, I applied cleanly to the 6.x-1.x branch

q0rban’s picture

@Eric, if this passes Drupal CS, go ahead and commit it.

robloach’s picture

Status: Reviewed & tested by the community » Fixed

After some testing, I committed to both Drupal 7 and Drupal 6... I scaled the size of the icons to more closely match the sizes of the other icons that are shipped with the module at 24x24 and 36x36. The small icon is also used by default. You can see that deviation in this commit.

Status: Fixed » Closed (fixed)

Automatically closed -- issue fixed for 2 weeks with no activity.

Gessuit’s picture

I know its been awhile on this post since someone commented, and this may not be the best spot to ask this question. But what do I do with the patchfile Yareckon posted? How do I get it working along with the follow module? I'm a novice at Drupal

AlexisWilke’s picture

Gessuit,

You may just want to use his proposed icons if that's what you wanted. (#5)

For that you'd just smack the existing icons with those and off you go! (hit reload / clear your caches to make sure you get the images updated.)

Thank you.
Alexis Wilke