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

🐞🔨 Pagination: Fix missing translation error for Next and Previous links #2217

Merged
merged 1 commit into from
Feb 20, 2024

Conversation

zspencer
Copy link
Member

@zspencer zspencer commented Feb 17, 2024

With the Pagy 7 upgrade; the helper I wrote so we could add spacing between the Pagination links broke.

This switches out from doing an in-ruby style override to an in-css style override.

After

Screenshot 2024-02-17 at 10 29 24 AM

Before

Screenshot 2024-02-17 at 10 26 16 AM

@zspencer zspencer requested review from a team February 17, 2024 18:26
- #2216

With the Pagy 7 upgrade; the helper I wrote so we could add spacing
between the Pagination links broke.

This switches out from doing an in-ruby style override to an in-css
style override; which should be less fraught.
@zspencer zspencer force-pushed the journal/fix-pagination-links branch from 7e9afc3 to 96cde68 Compare February 17, 2024 18:30
@zspencer zspencer changed the title 🐞🔨 Pagination: Apply Pagy themes via CSS 🐞🔨 Pagination: Fix missing translation error for Next and Previous links Feb 17, 2024
@zspencer zspencer merged commit de7f62a into main Feb 20, 2024
3 checks passed
@zspencer zspencer deleted the journal/fix-pagination-links branch February 20, 2024 18:27
zspencer added a commit to zinc-collective/tobias that referenced this pull request Mar 25, 2024
…us` links (zinc-collective#2217)

- zinc-collective#2216

With the Pagy 7 upgrade; the helper I wrote so we could add spacing
between the Pagination links broke.

This switches out from doing an in-ruby style override to an in-css
style override; which should be less fraught.
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