Merge pull request #102 from Lynesth/patch-15

Fix centering ascii art

Author: AydinHassan

src/MenuItem/AsciiArtItem.php

@@ -73,8 +73,7 @@ public function getRows(MenuStyle $style, bool $selected = false) : array
                     break;
                 case self::POSITION_CENTER:
                 default:
-                    $row = rtrim($row);
-                    $padding = $padding - ($this->artLength - mb_strlen($row));
+                    $padding = $padding - ($this->artLength - $length);
                     $left = ceil($padding/2);
                     $right = $padding - $left;
                     $row = sprintf('%s%s%s', str_repeat(' ', $left), $row, str_repeat(' ', $right));

test/MenuItem/AsciiArtItemTest.php

@@ -96,6 +96,15 @@ public function testGetRowsCenterAligned() : void
             ],
             $item->getRows($menuStyle)
         );
+
+        $item = new AsciiArtItem("    //    \n//////////", AsciiArtItem::POSITION_CENTER);
+        $this->assertEquals(
+            [
+                "    //    ",
+                "//////////",
+            ],
+            $item->getRows($menuStyle)
+        );
     }
 
     public function testGetRowsCenterAlignedWithOddWidth() : void
@@ -115,6 +124,15 @@ public function testGetRowsCenterAlignedWithOddWidth() : void
             ],
             $item->getRows($menuStyle)
         );
+
+        $item = new AsciiArtItem("    //    \n//////////", AsciiArtItem::POSITION_CENTER);
+        $this->assertEquals(
+            [
+                "     //    ",
+                " //////////",
+            ],
+            $item->getRows($menuStyle)
+        );
     }
 
     public function testHideAndShowItemExtraHasNoEffect() : void