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 2056485: Add pagination to tables if there are more than 10 items #1172

Merged
merged 1 commit into from Mar 4, 2022

Conversation

rawagner
Copy link
Contributor

Screenshot from 2022-02-25 11-15-18

@nirfarkas this adds pagination for every AI table in ACM and OCM when we have more than 10 items to show

@rawagner rawagner changed the title Add pagination to tables if there are more than 10 items Bug 2056485: Add pagination to tables if there are more than 10 items Feb 28, 2022
mareklibra
mareklibra previously approved these changes Feb 28, 2022
@nirfarkas
Copy link
Contributor

nirfarkas commented Mar 1, 2022

Looks great!
In OCM, it's a bit different:
image

@rawagner
Copy link
Contributor Author

rawagner commented Mar 1, 2022

PR updated, the pagination is not added to the top right corner too, if the table has a toolbar

Screenshot from 2022-03-01 15-27-12

Screenshot from 2022-03-01 15-27-28

@rawagner
Copy link
Contributor Author

rawagner commented Mar 1, 2022

/hold

@openshift-ci openshift-ci bot added the do-not-merge/hold Indicates that a PR should not merge because someone has issued a /hold command. label Mar 1, 2022
@nirfarkas
Copy link
Contributor

You have my blessing (:
Looking great!

@rawagner
Copy link
Contributor Author

rawagner commented Mar 3, 2022

/hold cancel

@openshift-ci openshift-ci bot removed the do-not-merge/hold Indicates that a PR should not merge because someone has issued a /hold command. label Mar 3, 2022
@jkilzi jkilzi requested a review from batzionb March 3, 2022 14:39
@openshift-ci
Copy link

openshift-ci bot commented Mar 4, 2022

[APPROVALNOTIFIER] This PR is NOT APPROVED

This pull-request has been approved by: jtomasek, rawagner

The full list of commands accepted by this bot can be found here.

Needs approval from an approver in each of these files:

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

@jtomasek jtomasek merged commit ef86f38 into openshift-assisted:master Mar 4, 2022
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.

None yet

4 participants