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): Tighten margins and reduce pages shown to make usable on 320px #5269

Merged
merged 2 commits into from
Nov 12, 2024

Conversation

dougmacknz
Copy link
Contributor

Why

Pagination currently overflows 320px

image

What

  • Tighten up some margins and padding
  • Reduce the 'sibling' page buttons that are shown down to 0 from 1 on small viewport
Screen.Recording.2024-11-12.at.2.08.07.PM.mov

@dougmacknz dougmacknz requested a review from a team as a code owner November 12, 2024 03:09
Copy link

changeset-bot bot commented Nov 12, 2024

🦋 Changeset detected

Latest commit: 3526455

The changes in this PR will be included in the next version bump.

This PR includes changesets to release 1 package
Name Type
@kaizen/components Patch

Not sure what this means? Click here to learn what changesets are.

Click here if you're a maintainer who wants to add another changeset to this PR

@dougmacknz dougmacknz force-pushed the KZN-2739/pagination-responsiveness branch from abf84d5 to 5659587 Compare November 12, 2024 03:10
Copy link
Contributor

github-actions bot commented Nov 12, 2024

✨ Here is your branch preview! ✨

Last updated for commit 3526455: Merge branch 'main' into KZN-2739/pagination-responsiveness

@dougmacknz dougmacknz force-pushed the KZN-2739/pagination-responsiveness branch from 5659587 to a1c6434 Compare November 12, 2024 03:16
@@ -51,15 +51,3 @@
}
}
}

// Truncate indicator
.truncateIndicatorWrapper {
Copy link
Contributor Author

@dougmacknz dougmacknz Nov 12, 2024

Choose a reason for hiding this comment

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

Wasn't being used at all, It got split out and copied into it's own component and I guess they forgot to delete this

@dougmacknz dougmacknz merged commit e4358c8 into main Nov 12, 2024
19 of 20 checks passed
@dougmacknz dougmacknz deleted the KZN-2739/pagination-responsiveness branch November 12, 2024 05:49
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.

2 participants