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

fix(pagination): add row padding on small screens and demo stories #1405

Conversation

HendrikThePendric
Copy link
Contributor

@HendrikThePendric HendrikThePendric commented Sep 14, 2023

Relates to DHIS2-15465


Description

This is a very unobtrusive PR that only adds a row-gap CSS rule to the pagination container to prevent the button "rows" touching when the pagination container is narrow. Implementation was discussed with Joe.


Checklist

  • API docs are generated
  • Tests were added
  • Storybook demos were added

NB: API docs and tests weren't relevant for this PR.


Screenshots

BEFORE
Screenshot 2023-09-14 at 12 28 09

AFTER
Screenshot 2023-09-14 at 12 26 31

@HendrikThePendric HendrikThePendric requested a review from a team as a code owner September 14, 2023 10:26
@dhis2-bot
Copy link
Contributor

🚀 Deployed on https://pr-1405--dhis2-ui.netlify.app

@dhis2-bot dhis2-bot temporarily deployed to netlify September 14, 2023 10:34 Inactive
@cypress
Copy link

cypress bot commented Sep 14, 2023

Passing run #3031 ↗︎

0 583 0 0 Flakiness 0

Details:

fix(pagination): add row padding on small screens and demo stories
Project: ui Commit: b373859316
Status: Passed Duration: 09:55 💡
Started: Sep 14, 2023 10:33 AM Ended: Sep 14, 2023 10:43 AM

This comment has been generated by cypress-bot as a result of this project's GitHub integration settings.

@HendrikThePendric HendrikThePendric merged commit 91a4e6f into master Sep 14, 2023
21 checks passed
@HendrikThePendric HendrikThePendric deleted the DHIS2-15465-pagination-row-padding-in-narrow-containers branch September 14, 2023 14:49
@dhis2-bot
Copy link
Contributor

🎉 This PR is included in version 8.14.3 🎉

The release is available on:

Your semantic-release bot 📦🚀

@dhis2-bot
Copy link
Contributor

🎉 This PR is included in version 8.15.0-alpha.3 🎉

The release is available on:

Your semantic-release bot 📦🚀

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

Successfully merging this pull request may close these issues.

4 participants