Alright.. I looked into our code a bit more on how this works. Our SCLogin module tries to fetch the optimal picture from Facebook to display based on your height/width settings. When you're values are 'small', we pick what's called a pre-defined 'small' size. As you set larger values, we use larger predfined settings like 'large'. The 'small' value is always going to return a square picture. The larger values will return variable size pictures. The reason we scale up is so that the picture isn't too pixellated, but it also means that the picture is no longer square.
With that said, it looks like Facebook has added some new ways to crop pictures since the last time we checked. If interested in *testing* some changes, please edit the /modules/mod_sclogin/helper.php file. At line 195, you'll see:
$fbAvatarURL = 'https://graph.facebook.com/'.$fbUserId.'/picture?type='.$fbAvatarSize;
Edit that and change it to:
$fbAvatarURL = 'https://graph.facebook.com/'.$fbUserId.'/picture?width='.$this->params->get("profileWidth").'&height='.$this->params->get("profileHeight");;That will use the parameters from the module and tell FB exactly what to fetch instead of using their predefined sizes, which is better overall. It's likely something we'll implement in the upcoming SCLogin release as well, but would love to hear you're feedback on the above.
Let me know how that goes, and good luck,
Alex