Pagination needs better styling.

Created on 7 September 2023, over 1 year ago

🐛 Bug report
Status

Active

Version

1.0

Component

Code

Created by

🇮🇳India alok_singh

Live updates comments and jobs are added and updated live.
Sign in to follow issues

Merge Requests

Comments & Activities

  • Issue created by @alok_singh
  • Status changed to Needs review over 1 year ago
  • Status changed to Needs work 9 months ago
  • @alok_singh, verified MR5 and I can see the pagination is broken on mobile.

  • Assigned to manishvijay02
  • Status changed to Active 9 months ago
  • Status changed to Needs review 9 months ago
  • 🇮🇳India manishvijay02

    I have made changes in the CSS files accordingly and made the pagination responsive for all the devices. I have attached the screenshot for the refrence.

  • @Manishvijay02, getting this error after applying MR5 -
    5.patch:817: trailing whitespace.
    }
    5.patch:849: trailing whitespace.
    border:2px solid #f91546;
    5.patch:909: trailing whitespace.
    #news-letter .btn.btn-theme{
    5.patch:918: trailing whitespace.
    }
    5.patch:1101: trailing whitespace.
    }
    warning: css/custom.css has type 100644, expected 100755
    error: patch failed: css/custom.css:229
    error: css/custom.css: patch does not apply

  • 🇮🇳India manishvijay02

    @Tanuja Bohra
    The MR patch looks outdated as we have made a some commits. Please do not apply the MR patch, try to apply the MR diff or take pull from this branch directly and then test the code and styling.

  • Issue was unassigned.
  • Status changed to RTBC 3 days ago
  • Hi @manishvijay02,

    Thank you for your work on this! I’ve reviewed your MR, and it successfully applies styles to the pagination as intended. To provide clarity, I’ve attached before and after screenshots for reference.

    I’m moving this issue to RTBC++.

Production build 0.71.5 2024