- Issue created by @hamid.ali
- Status changed to Needs review
6 months ago 9:34am 16 May 2024 - 🇮🇳India hamid.ali
Adding the patch for the pagination fix along with that removal of commented code in the CSS. Adding the screenshot below for the post patch:
- Status changed to Needs work
6 months ago 10:32am 16 May 2024 - 🇮🇳India Gautam_105@
@Hamid.ali ,
I've tested your patch and it's working fine with the full pager. However, when using the mini pager, we are encountering the same issue: "Pagination needs to be styled." So, i am moving in need to work. - First commit to issue fork.
- Assigned to Muskan0215
Styled the pagination and also highlighted the active page, check the attached image for reference
- Issue was unassigned.
- Status changed to Needs review
6 months ago 5:35am 17 May 2024 @Muskan0215, I applied the MR5 on 10.3.x and the mini pager doesn't look good and is not centered and aligned properly on the mobile.
As per #11 screenshot , I don't see highlighted active state after MR5.- Status changed to Needs work
6 months ago 7:48am 17 May 2024 - Assigned to Muskan0215
@Tanuja Bohra , I have center aligned the mini pager, please check the MR now.
Also, i have updated the design as seen in screenshot #13, since the active page should be bold as asked in the requirement.- Issue was unassigned.
- Status changed to Needs review
6 months ago 9:07am 17 May 2024 - 🇮🇳India Gautam_105@
@Muskan0215 , i have tested your MR it's working fine for me.
full pager
mini pager
- Status changed to RTBC
6 months ago 11:00am 17 May 2024