From 322d12101e0ebec0468e5ce054c60a9b5ec11e65 Mon Sep 17 00:00:00 2001
From: Oliver Switzer <oliver@geometer.io>
Date: Thu, 28 Sep 2023 11:32:23 -0400
Subject: [PATCH] Flop.Phoenix.pagination change

* add ability to specify that you do not wanter to render next and previous links
---
 lib/flop_phoenix.ex            |  5 ++++-
 lib/flop_phoenix/pagination.ex |  1 +
 test/flop_phoenix_test.exs     | 19 +++++++++++++++++--
 3 files changed, 22 insertions(+), 3 deletions(-)

diff --git a/lib/flop_phoenix.ex b/lib/flop_phoenix.ex
index 276d31d..b06bb1c 100644
--- a/lib/flop_phoenix.ex
+++ b/lib/flop_phoenix.ex
@@ -186,7 +186,8 @@ defmodule Flop.Phoenix do
     Default: `#{inspect(Pagination.default_opts()[:wrapper_attrs])}`.
   """
   @type pagination_option ::
-          {:current_link_attrs, keyword}
+          {:render_next_and_previous_links, boolean()}
+          | {:current_link_attrs, keyword}
           | {:disabled_class, String.t()}
           | {:ellipsis_attrs, keyword}
           | {:ellipsis_content, Phoenix.HTML.safe() | binary}
@@ -429,6 +430,7 @@ defmodule Flop.Phoenix do
     ~H"""
     <nav :if={@meta.errors == [] && @meta.total_pages > 1} {@opts[:wrapper_attrs]}>
       <.pagination_link
+        :if={@opts[:render_next_and_previous_links]}
         disabled={!@meta.has_previous_page?}
         disabled_class={@opts[:disabled_class]}
         event={@event}
@@ -441,6 +443,7 @@ defmodule Flop.Phoenix do
         <%= @opts[:previous_link_content] %>
       </.pagination_link>
       <.pagination_link
+        :if={@opts[:render_next_and_previous_links]}
         disabled={!@meta.has_next_page?}
         disabled_class={@opts[:disabled_class]}
         event={@event}
diff --git a/lib/flop_phoenix/pagination.ex b/lib/flop_phoenix/pagination.ex
index 2c15b8c..0a0b18a 100644
--- a/lib/flop_phoenix/pagination.ex
+++ b/lib/flop_phoenix/pagination.ex
@@ -8,6 +8,7 @@ defmodule Flop.Phoenix.Pagination do
   @spec default_opts() :: [Flop.Phoenix.pagination_option()]
   def default_opts do
     [
+      render_next_and_previous_links: true,
       current_link_attrs: [
         class: "pagination-link is-current",
         aria: [current: "page"]
diff --git a/test/flop_phoenix_test.exs b/test/flop_phoenix_test.exs
index 8b593ea..e7d7b30 100644
--- a/test/flop_phoenix_test.exs
+++ b/test/flop_phoenix_test.exs
@@ -101,6 +101,22 @@ defmodule Flop.PhoenixTest do
              """) == []
     end
 
+    test "does not render next and previous links if user specifies not to" do
+      assigns = %{meta: build(:meta_on_first_page)}
+
+      html =
+        parse_heex(~H"""
+        <Flop.Phoenix.pagination
+          meta={@meta}
+          on_paginate={%JS{}}
+          opts={[render_next_and_previous_links: false]}
+        />
+        """)
+
+      assert [] = Floki.find(html, ".pagination-next")
+      assert [] = Floki.find(html, ".pagination-previous")
+    end
+
     test "allows to overwrite wrapper class" do
       assigns = %{meta: build(:meta_on_first_page)}
 
@@ -3023,8 +3039,7 @@ defmodule Flop.PhoenixTest do
       assert [
                {"div", [{"data-test-id", "thead-label-component"}],
                 ["\n  Custom\n"]}
-             ] =
-               Floki.find(html, ~s([data-test-id="thead-label-component"]))
+             ] = Floki.find(html, ~s([data-test-id="thead-label-component"]))
     end
 
     test "renders multiple inputs for the same field", %{