Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Bug with Pagination as current page selection is not visible. #719

Closed
adnankarim opened this issue Apr 25, 2023 · 7 comments · Fixed by #725
Closed

Bug with Pagination as current page selection is not visible. #719

adnankarim opened this issue Apr 25, 2023 · 7 comments · Fixed by #725
Labels
🐛 bug Something isn't working confirmed This bug was confirmed

Comments

@adnankarim
Copy link

Describe the bug

When using the Pagination component in Flowbite React with Tailwind CSS in a NextJS project, the background or interface change for the current page indicator does not appear to be working.

Screenshots
If applicable, add screenshots to help explain your problem.
image

System information:

  • Device: Core i7
  • Resolution: 1080p
  • OS: Windows 11
  • Browser: Chrome
  • Version: latest

Project information:

  • Tailwind: 3.3.1
  • Flowbite: ^1.6.5
  • Flowbite React: ^0.4.3
  • Type: CRA
@tulup-conner
Copy link
Collaborator

It's working at https://flowbite-react.com/pagination - Try upgrading to v0.4.4? There were some changes to this component recently, but I don't know why you'd be having this issue regardless.

@adnankarim
Copy link
Author

adnankarim commented Apr 26, 2023

I have updated the library still it is not showing selected page UI interface.

All Students
  </div>
  <div className=" px-4 container bg-white mx-auto z-[30] rounded-[10px]">
  <SearchFilterInput />
  <JourneyTable
    data={myData}
    columns={columns}
    selectable
    onRowClick={(rowData: any) => {
      console.log("Row clicked", rowData), alert(rowData);
    }}
  />

  <div className="flex items-center justify-center text-center py-5">
    <Pagination
      currentPage={currentPage}
      layout="pagination"
      onPageChange={onPageChange}
      showIcons={true}
      totalPages={1000}
      previousLabel=""
      nextLabel=""
    />
  </div>
  </div>
</div>

image

@Timtendo12
Copy link

It's working at https://flowbite-react.com/pagination - Try upgrading to v0.4.4? There were some changes to this component recently, but I don't know why you'd be having this issue regardless.

I have the same issue, It seems like the standard bg-white is overpowering the bg-blue
image

@adriangornypl
Copy link

adriangornypl commented May 1, 2023

Same problem here :) @Timtendo12 exactly as he described.

-- edit --

As a hotfix for anyone needing urgent fix I'd recommend applying the !important styles to globals -> usually shouldn't conflict with others, can also be limited to custom pagination class:


.bg-blue-50 {
    --tw-bg-opacity: 1 !important;
    background-color: rgb(235 245 255 / var(--tw-bg-opacity)) !important;
}

.text-blue-600 {
    --tw-text-opacity: 1 !important;
    color: rgb(28 100 242 / var(--tw-text-opacity)) !important;
}

@tulup-conner tulup-conner added 🐛 bug Something isn't working confirmed This bug was confirmed labels May 4, 2023
@IuriPires
Copy link
Contributor

@adnankarim already opened a PR to resolve this issue: #725 waiting review.

@tcrz
Copy link

tcrz commented Jul 5, 2023

Still facing this same issue described by @Timtendo12

@tulup-conner
Copy link
Collaborator

Still facing this same issue described by @Timtendo12

The flowbite-react.com documentation for Pagination is working correctly, so I'm not really sure what to tell you in this case. If you can provide a reproduction of the issue with more specific detail and possibly explore why it might be happening to you specifically please feel free to create a new issue for it.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
🐛 bug Something isn't working confirmed This bug was confirmed
Projects
None yet
Development

Successfully merging a pull request may close this issue.

6 participants