- Issue created by @hamid.ali
- ๐ฎ๐ณIndia Rajan Kumar@2026
Rajan Kumar@2026 โ made their first commit to this issueโs fork.
- Status changed to Needs review
5 months ago 9:35am 20 June 2024 - ๐ฎ๐ณIndia Rajan Kumar@2026
Hii @Hamid.ali
I have created the MR and fixed this issue. Please review and verify it.
- Status changed to Needs work
5 months ago 10:27am 20 June 2024 - ๐ฎ๐ณIndia hamid.ali
@Rajan Kumar@2026 applied changes from your MR firstly main pager and the mini pager look way too different in appearance. Also you have added the css directly into css file instead of the sass file. Kindly follow the coding standards along with the visual changes. I am attaching screenshots for your reference. Also before making any changes kindly apply the changes from the parent issue MR so that you get idea how the mini pager should look. Thanks
- ๐ฎ๐ณIndia Rajan Kumar@2026
Rajan Kumar@2026 โ changed the visibility of the branch 3455881-Mini-pager-style to hidden.
- ๐ฎ๐ณIndia hamid.ali
I reviewed the patch, Everything looks good to me apart from the alignment page number text. Which isn't aligned with the buttons, to fix the same you can add the align-items centre property to the ul.js-pager-items. I am adding screenshot for my assessment