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

added item pagination with 10 items per page and 4 buttons to select from along with arrows #35

Open
wants to merge 2 commits into
base: main
Choose a base branch
from

Conversation

ElianMalessy
Copy link
Contributor

resolves #25

here is what the pagination looks like:
image

@stevem-zhou stevem-zhou self-requested a review May 27, 2024 05:17
Copy link
Collaborator

@stevem-zhou stevem-zhou left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

aight i think this looks really good, do you think you can fix a lil bit of stuff before the meeting.

there's a bunch of extra space when you keep scrolling to the bottom, and every time you click on a new page, it should bring it back to the top

image

@ElianMalessy
Copy link
Contributor Author

I've added the scrolling functionality and I got rid of the extra space. I was getting errors of undefined lat on the map while I was developing so I made a little change on the Map.jsx file to fix it.

Copy link
Collaborator

@stevem-zhou stevem-zhou left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

since we have agreed to go with pagination, please fix the bugs that we have found during the meeting:

  • - search bar not working with pagination
  • - number of pages would change depending on search
  • - rerender the page numbers
  • - filtering as well
  • - button to go straight to first item or last
  • - … button to specify page (user can input page number)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Pagination of items
2 participants