Topic-icon Custom Google button error

Active Subscriptions:

None
10 years 7 months ago - 10 years 7 months ago #36185 by drjjw
Hello,

In JFB 5.1.1 for J2.5.14, I am using custom FB buttons with no trouble in the SC Login module. But in for the Google button, I get a blank img src:

<a class="show" id="sc_gologin" href="javascript:void(0)" onclick="jfbc.login.google();"><img src="/"></a>

The link used is active, and have tried in relative and absolute configuration.

Jordan
Last edit: 10 years 7 months ago by drjjw.
The topic has been locked.
Support Specialist
10 years 7 months ago #36192 by alzander
Replied by alzander on topic Custom Google button error
Jordan,
Sorry about that. Looks like a silly bug on our part with the custom image option in the SCLogin module for the Google image. To fix, please edit the /modules/mod_sclogin/helper.php file. At line 284, you'll see:
$params['googleLinkImage'] = $goLoginButtonLinkImage;
Update that to:
$params['goLinkImage'] = $goLoginButtonLinkImage;

Do the same at line 308, which is for the Reconnect buttons (when a user is logged in and hasn't already connected to Google).

Let me know how that goes, and sorry for the trouble.

Thanks,
Alex
The topic has been locked.
Active Subscriptions:

None
10 years 7 months ago #36195 by drjjw
Replied by drjjw on topic Custom Google button error
Not a problem Alex,

That solved it.

Jordan
The topic has been locked.
Support Specialist
10 years 7 months ago #36207 by alzander
Replied by alzander on topic Custom Google button error
Awesome. Stupid bug, but easy fix. We like those issues.

Glad that got it going for you. As always, should you need anything else, just let us know.

Thanks,
Alex
The topic has been locked.
Active Subscriptions:

None
10 years 7 months ago #36314 by lukmel
Replied by lukmel on topic Custom Google button error
This solutions helped me too! Thanks.
The topic has been locked.
Support Specialist
10 years 7 months ago #36331 by alzander
Replied by alzander on topic Custom Google button error
lukmel,
Awesome. Glad to hear that helped, and apologies again for the silly typo on our part. Again, this will be included in the next release, currently due out around mid-September.

Thanks,
Alex
The topic has been locked.