Fix and optimize widget

Author: yidas

src/widgets/Pagination.php

@@ -215,6 +215,9 @@ public function run()
             $linkAttributes .= "{$key}=\"{$value}\" ";
         }
 
+        $isFirst = ($page<=1) ? true : false;
+        $isLast = ($this->pagination->pageCount<=$page) ? true : false;
+
         include __DIR__ . '/views/bootstrap.php';
 
         return;

src/widgets/views/bootstrap.php

@@ -1,11 +1,41 @@
 <nav aria-label="Page navigation example">
   <ul class="pagination <?php if($this->alignCenter):?>justify-content-center<?php endif?>">
-    <?php if($this->firstPageLabel!==null):?><li class="<?=$this->pageCssClass?> <?=$this->firstPageCssClass?> <?php if($page<=1):?>disabled<?php endif ?>"><a <?=$linkAttributes?> href="<?=$this->pagination->createUrl(1);?>"><?=$this->firstPageLabel?></a></li><?php endif ?>
-    <li class="<?=$this->pageCssClass?> <?=$this->prevPageCssClass?> <?php if($page<=1):?>disabled<?php endif ?>"><a <?=$linkAttributes?> href="<?=$this->pagination->createUrl($page-1);?>"><?=$this->prevPageLabel?></a></li>
+    <?php if($this->firstPageLabel):?>
+    <li class="<?=$this->pageCssClass?> <?=$this->firstPageCssClass?> <?php if($isFirst):?>disabled<?php endif ?>">
+      <?php if($isFirst):?>
+      <span><?=$this->firstPageLabel?></span>
+      <?php else: ?>
+      <a <?=$linkAttributes?> href="<?=$this->pagination->createUrl(1);?>"><?=$this->firstPageLabel?></a>
+      <?php endif ?>
+    </li>
+    <?php endif ?>
+    <li class="<?=$this->pageCssClass?> <?=$this->prevPageCssClass?> <?php if($isFirst):?>disabled<?php endif ?>">
+      <?php if($isFirst):?>
+      <span><?=$this->prevPageLabel?></span>
+      <?php else: ?>
+      <a <?=$linkAttributes?> href="<?=$this->pagination->createUrl($page-1);?>"><?=$this->prevPageLabel?></a>
+      <?php endif ?>
+    </li>
     <?php foreach ($this->_buttonStack as $key => $btnPage): ?>
-    <li class="<?=$this->pageCssClass?> <?php if(($page)==$btnPage):?>active<?php endif ?>"><a <?=$linkAttributes?> href="<?=$this->pagination->createUrl($btnPage);?>"><?=$btnPage?></a></li>
+    <li class="<?=$this->pageCssClass?> <?php if(($page)==$btnPage):?>active<?php endif ?>">
+      <a <?=$linkAttributes?> href="<?=$this->pagination->createUrl($btnPage);?>"><?=$btnPage?></a>
+    </li>
     <?php endforeach ?>
-    <li class="<?=$this->pageCssClass?> <?=$this->nextPageCssClass?> <?php if($this->pagination->pageCount<=$page):?>disabled<?php endif ?>"><a <?=$linkAttributes?> href="<?=$this->pagination->createUrl($page+1);?>"><?=$this->nextPageLabel?></a></li>
-    <?php if($this->lastPageCssClass!==null):?><li class="<?=$this->pageCssClass?> <?=$this->lastPageCssClass?> <?php if($this->pagination->pageCount<=$page):?>disabled<?php endif ?>"><a <?=$linkAttributes?> href="<?=$this->pagination->createUrl($this->pagination->pageCount);?>"><?=$this->lastPageLabel?></a></li><?php endif ?>
+    <li class="<?=$this->pageCssClass?> <?=$this->nextPageCssClass?> <?php if($isLast):?>disabled<?php endif ?>">
+      <?php if($isLast):?>
+      <span><?=$this->nextPageLabel?></span>
+      <?php else: ?>
+      <a <?=$linkAttributes?> href="<?=$this->pagination->createUrl($page+1);?>"><?=$this->nextPageLabel?></a>
+      <?php endif ?>
+    </li>
+    <?php if($this->lastPageLabel):?>
+    <li class="<?=$this->pageCssClass?> <?=$this->lastPageCssClass?> <?php if($isLast):?>disabled<?php endif ?>">
+      <?php if($isLast):?>
+      <span><?=$this->lastPageLabel?></span>
+      <?php else: ?>
+      <a <?=$linkAttributes?> href="<?=$this->pagination->createUrl($this->pagination->pageCount);?>"><?=$this->lastPageLabel?></a>
+      <?php endif ?>
+    </li>
+    <?php endif ?>
   </ul>
 </nav>