[openstreetmap/openstreetmap-website] Add pagination to Issues page (PR #5057)
Andy Allan
notifications at github.com
Wed Aug 7 13:31:55 UTC 2024
I agree with Tom here, the SPA-like behaviour should be implemented using the Turbo framework, not with custom javascript.
Otherwise, I'm happy to see pagination used for these pages. I would suggest just implementing the pagination first, then returning to the concept of using turbo drive separately, and ideally for all the paginated pages at the same time.
Using Turbo will also avoid having to define an extra `page` route, which doesn't fit into our resourceful-routing approach.
--
Reply to this email directly or view it on GitHub:
https://github.com/openstreetmap/openstreetmap-website/pull/5057#issuecomment-2273484629
You are receiving this because you are subscribed to this thread.
Message ID: <openstreetmap/openstreetmap-website/pull/5057/c2273484629 at github.com>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.openstreetmap.org/pipermail/rails-dev/attachments/20240807/9eb340bd/attachment.htm>
More information about the rails-dev
mailing list