Merge pull request #110 from vyskocilpavel/social_width

Fixed the width of showed tagged idps

Author: vyskocilpavel

CHANGELOG.md

@@ -5,6 +5,9 @@ All notable changes to this project will be documented in this file.
 #### Added
 - Added facility capabilities to PerunEntitlement
 
+#### Fixed
+- Fixed the width of showed tagged idps in case the count of idps is equal to (x * 3) + 1
+
 ## [v3.8.0]
 #### Changed
 - Releasing forwardedEduPersonEntitlement is now optional (forwardedEduPersonEntitlement are released by default)

lib/Disco.php

@@ -483,10 +483,12 @@ public static function showTaggedIdPs($t, $tag, $showSignInWith = false)
             }
             $html .= '</div>';
         }
-        if (($idpCount % 3) !== 0) {
+
+        $remainIdpsCount = ($idpCount - ($fullRowCount * 3)) % 3;
+        if ($remainIdpsCount !== 0) {
             $html .= '<div class="row">';
-            for ($i = 0; $i < $idpCount % 3; $i++) {
-                $html .= '<div class="col-md-' . (12 / ($idpCount % 3))  . '">';
+            for ($i = 0; $i < $remainIdpsCount; $i++) {
+                $html .= '<div class="' . self::getCssClass($remainIdpsCount) . '">';
                 $html .= '<div class="metalist list-group">';
                 $html .= self::showTaggedEntry($t, $idps[array_keys($idps)[$counter]], $showSignInWith);
                 $html .= '</div>';
@@ -499,6 +501,14 @@ public static function showTaggedIdPs($t, $tag, $showSignInWith = false)
         return $html;
     }
 
+    protected static function getCssClass($remainIdpsCount)
+    {
+        if ($remainIdpsCount === 1) {
+            return 'col-md-4 col-md-offset-4';
+        }
+        return 'col-md-6';
+    }
+
     public static function showEntriesScript()
     {
         $script = '<script type="text/javascript">