Topic-icon Social Find Us module issue

Active Subscriptions:

None
10 years 2 months ago - 10 years 2 months ago #57472 by timwil
Amongst the stuff that comes with JFBConnect, I discovered there is a module called Social Find Us but I am having a problem styling it.

Originally the module outputs looked like below.


but I wanted it to look like


I changed the padding in style.css to {padding:0 5px 0 0} but when I do this, I get the below (the icons appearing to wrap on 2 lines)


When I looked at the HTML, I see the problem is the div width being set to 160px but when I searched mod_socialfindus files, I can't see where this value has come from because the word "horizontal" does not exist.

To show the actual width of the module, here is a screenshot where Firebug is highlighting the module width. Also the second screenshot was done "on the fly" using Firebug where I disabled the width setting.


Please could you enlighten me.
Last edit: 10 years 2 months ago by timwil.
The topic has been locked.
Active Subscriptions:

None
10 years 2 months ago #57473 by timwil
Replied by timwil on topic Social Find Us module issue
Bl**dy typical... I spend half an hour posting a question then I find the answer.

It's in mod_scsocialfindus.php and that you have semi-dynamically set the width by
$width = 0;
if(!empty($facebookLink)) $width += 32;
if(!empty($googleLink)) $width += 32;
if(!empty($instagramLink)) $width += 32;
if(!empty($twitterLink)) $width += 32;
if(!empty($linkedinLink)) $width += 32;
if(!empty($pinterestLink)) $width += 32;
if(!empty($flickrLink)) $width += 32;
if(!empty($rssLink)) $width += 32;
if(!empty($youTubeLink)) $width += 32;
which I have simply changed to
$width = 0;
if(!empty($facebookLink)) $width += 42;
if(!empty($googleLink)) $width += 42;
if(!empty($instagramLink)) $width += 42;
if(!empty($twitterLink)) $width += 42;
if(!empty($linkedinLink)) $width += 42;
if(!empty($pinterestLink)) $width += 42;
if(!empty($flickrLink)) $width += 42;
if(!empty($rssLink)) $width += 42;
if(!empty($youTubeLink)) $width += 42;
My changes worked and the module now displays perfectly.
The topic has been locked.
Support Specialist
10 years 2 months ago #57477 by alzander
Glad you found the solution.. the way we do it is not ideal for allowing overrides, as you can tell. We'll try to update this in the future as we improve the SCFindUs module further.

Thanks,
Alex
The topic has been locked.