Skip to content

Tapan Fixed the Pagination UI in Members Table for Project Reports#2201

Merged
one-community merged 1 commit into
developmentfrom
Tapan_Pagination_UI_Members_Table_Fix
Aug 2, 2024
Merged

Tapan Fixed the Pagination UI in Members Table for Project Reports#2201
one-community merged 1 commit into
developmentfrom
Tapan_Pagination_UI_Members_Table_Fix

Conversation

@TapanPathak
Copy link
Copy Markdown

Description

Screenshot 2024-04-26 at 4 45 30 PM

Pagination Overlaps with table values.

Related PRS (if any):

N/A

Main changes explained:

  • Fixed UI for handling overlap of pagination and table values
  • Fixed CSS styling for responsiveness

How to test:

  1. check into current branch
  2. do npm install and ... to run this PR locally
  3. Clear site data/cache
  4. log as admin/owner user
  5. Go to Reports -> Projects -> Any project with 7 or more members
  6. Make Screen Smaller
  7. Check that pagination doesn't overlaps with table values

Screenshots or videos of changes:

Before -

Screen.Recording.2024-03-23.at.10.56.25.AM.1.mov

After -

Screen.Recording.2024-04-26.at.3.57.35.PM.mov

Copy link
Copy Markdown

@linkhanthtel linkhanthtel left a comment

Choose a reason for hiding this comment

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

There is no pagination overlapping in project members table. Nice work!

2201.mov

Copy link
Copy Markdown
Contributor

@aaryaneil aaryaneil left a comment

Choose a reason for hiding this comment

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

Tested the PR and the pagination doesn't overlap when the screen size is reduced.

Screen.Recording.2024-04-27.at.7.25.30.AM.mov

Copy link
Copy Markdown

@MinSun0814 MinSun0814 left a comment

Choose a reason for hiding this comment

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

There is no overlapping when resizing the window now.

Screen.Recording.2024-05-02.at.13.09.16.mov

Copy link
Copy Markdown
Contributor

@TareqMia TareqMia left a comment

Choose a reason for hiding this comment

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

Changes look good and there is no overlap when making the screen size smaller 👍

Screen.Recording.2024-05-12.at.1.21.29.AM.mov

Copy link
Copy Markdown
Contributor

@peter6866 peter6866 left a comment

Choose a reason for hiding this comment

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

Hi, I have reviewed your PR and it works with smaller screen sizes.
image
image

@one-community one-community added Needs New Developer This is a PR that is partially developed but needs someone new to take it over and finish it. do not review Do not review or look at code without full context labels Jul 17, 2024
@one-community one-community merged commit 9d82c3b into development Aug 2, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

do not review Do not review or look at code without full context Needs New Developer This is a PR that is partially developed but needs someone new to take it over and finish it.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

7 participants