From fdb13b15356219f03a83fe0a08f22fd7343b8a52 Mon Sep 17 00:00:00 2001
From: Andrew Kazakov <tosk-nn@yandex.ru>
Date: Thu, 7 Nov 2024 13:36:17 +0300
Subject: [PATCH] fix(Pagination): ignore clicks if pagination button is
 disabled (#1415)

---
 .../src/Pagination/Pagination.test.ts         | 68 +++++++++++++++++++
 .../src/Pagination/PaginationFirst.vue        |  7 +-
 .../src/Pagination/PaginationLast.vue         |  7 +-
 .../src/Pagination/PaginationListItem.vue     |  8 ++-
 .../src/Pagination/PaginationNext.vue         |  7 +-
 .../src/Pagination/PaginationPrev.vue         |  7 +-
 .../src/Pagination/story/_Pagination.vue      | 37 ++++++----
 7 files changed, 118 insertions(+), 23 deletions(-)

diff --git a/packages/radix-vue/src/Pagination/Pagination.test.ts b/packages/radix-vue/src/Pagination/Pagination.test.ts
index d2506f0d9..8d140af7d 100644
--- a/packages/radix-vue/src/Pagination/Pagination.test.ts
+++ b/packages/radix-vue/src/Pagination/Pagination.test.ts
@@ -56,6 +56,74 @@ describe('given default Pagination', () => {
   })
 })
 
+const ALL_PAGINATION_BUTTONS_AS_A_PROPS = {
+  first: { as: 'a' },
+  prev: { as: 'a' },
+  listItem: { as: 'a' },
+  next: { as: 'a' },
+  last: { as: 'a' },
+}
+
+describe('given Pagination with <a> as buttons', () => {
+  let wrapper: VueWrapper<InstanceType<typeof Pagination>>
+
+  beforeEach(() => {
+    document.body.innerHTML = ''
+    wrapper = mount(Pagination, { attachTo: document.body, props: { ...ALL_PAGINATION_BUTTONS_AS_A_PROPS } })
+  })
+
+  it('should pass axe accessibility tests', async () => {
+    expect(await axe(wrapper.element)).toHaveNoViolations()
+  })
+
+  it('should not unselect page 1 after clicking on Prev Page trigger', async () => {
+    await wrapper.find('[aria-label="Previous Page"]').trigger('click')
+    expect(wrapper.find('[aria-label="Page 1"]').attributes('data-selected')).toBe('true')
+  })
+
+  it('should not unselect last page after clicking on Next Page trigger', async () => {
+    await wrapper.find('[aria-label="Last Page"]').trigger('click')
+    const lastPageLabel = wrapper.find('[data-selected="true"]').attributes('aria-label')
+    await wrapper.find('[aria-label="Next Page"]').trigger('click')
+    expect(wrapper.find('[data-selected="true"]').attributes('aria-label')).toBe(lastPageLabel)
+  })
+})
+
+describe('given Pagination with <a> as buttons and disabled', () => {
+  let wrapper: VueWrapper<InstanceType<typeof Pagination>>
+
+  const INITIAL_PAGE = 2 // Do not set to first or last page
+
+  beforeEach(async () => {
+    document.body.innerHTML = ''
+    wrapper = mount(Pagination, { attachTo: document.body, props: { ...ALL_PAGINATION_BUTTONS_AS_A_PROPS } })
+    await wrapper.find(`[aria-label="Page ${INITIAL_PAGE}"]`).trigger('click')
+    wrapper.setProps({ root: { disabled: true } })
+  })
+
+  it('should pass axe accessibility tests', async () => {
+    expect(await axe(wrapper.element)).toHaveNoViolations()
+  })
+
+  it('should ignore clicking on First Page trigger', async () => {
+    await wrapper.find('[aria-label="First Page"]').trigger('click')
+
+    expect(wrapper.find('[data-selected="true"]').attributes('aria-label')).toBe(`Page ${INITIAL_PAGE}`)
+  })
+
+  it('should ignore clicking on Last Page trigger', async () => {
+    await wrapper.find('[aria-label="Last Page"]').trigger('click')
+
+    expect(wrapper.find('[data-selected="true"]').attributes('aria-label')).toBe(`Page ${INITIAL_PAGE}`)
+  })
+
+  it('should ignore clicking on any non-selected page', async () => {
+    await wrapper.find('[aria-label="Page 1"]').trigger('click')
+
+    expect(wrapper.find('[data-selected="true"]').attributes('aria-label')).toBe(`Page ${INITIAL_PAGE}`)
+  })
+})
+
 describe('given show-edges Pagination', () => {
   let wrapper: VueWrapper<InstanceType<typeof Pagination>>
 
diff --git a/packages/radix-vue/src/Pagination/PaginationFirst.vue b/packages/radix-vue/src/Pagination/PaginationFirst.vue
index 453ff6504..83ba07038 100644
--- a/packages/radix-vue/src/Pagination/PaginationFirst.vue
+++ b/packages/radix-vue/src/Pagination/PaginationFirst.vue
@@ -6,6 +6,7 @@ export interface PaginationFirstProps extends PrimitiveProps {}
 </script>
 
 <script setup lang="ts">
+import { computed } from 'vue'
 import { Primitive } from '@/Primitive'
 import { injectPaginationRootContext } from './PaginationRoot.vue'
 
@@ -13,6 +14,8 @@ const props = withDefaults(defineProps<PaginationFirstProps>(), { as: 'button' }
 
 const rootContext = injectPaginationRootContext()
 useForwardExpose()
+
+const disabled = computed((): boolean => rootContext.page.value === 1 || rootContext.disabled.value)
 </script>
 
 <template>
@@ -20,8 +23,8 @@ useForwardExpose()
     v-bind="props"
     aria-label="First Page"
     :type="as === 'button' ? 'button' : undefined"
-    :disabled="rootContext.page.value === 1 || rootContext.disabled.value"
-    @click="rootContext.onPageChange(1)"
+    :disabled
+    @click="!disabled && rootContext.onPageChange(1)"
   >
     <slot>First page</slot>
   </Primitive>
diff --git a/packages/radix-vue/src/Pagination/PaginationLast.vue b/packages/radix-vue/src/Pagination/PaginationLast.vue
index 39c9c93be..0b5aca033 100644
--- a/packages/radix-vue/src/Pagination/PaginationLast.vue
+++ b/packages/radix-vue/src/Pagination/PaginationLast.vue
@@ -6,6 +6,7 @@ export interface PaginationLastProps extends PrimitiveProps {}
 </script>
 
 <script setup lang="ts">
+import { computed } from 'vue'
 import { Primitive } from '@/Primitive'
 import { injectPaginationRootContext } from './PaginationRoot.vue'
 
@@ -13,6 +14,8 @@ const props = withDefaults(defineProps<PaginationLastProps>(), { as: 'button' })
 
 const rootContext = injectPaginationRootContext()
 useForwardExpose()
+
+const disabled = computed((): boolean => rootContext.page.value === rootContext.pageCount.value || rootContext.disabled.value)
 </script>
 
 <template>
@@ -20,8 +23,8 @@ useForwardExpose()
     v-bind="props"
     aria-label="Last Page"
     :type="as === 'button' ? 'button' : undefined"
-    :disabled="rootContext.page.value === rootContext.pageCount.value || rootContext.disabled.value"
-    @click="rootContext.onPageChange(rootContext.pageCount.value)"
+    :disabled
+    @click="!disabled && rootContext.onPageChange(rootContext.pageCount.value)"
   >
     <slot>Last page</slot>
   </Primitive>
diff --git a/packages/radix-vue/src/Pagination/PaginationListItem.vue b/packages/radix-vue/src/Pagination/PaginationListItem.vue
index b9102bfd0..e90baabfa 100644
--- a/packages/radix-vue/src/Pagination/PaginationListItem.vue
+++ b/packages/radix-vue/src/Pagination/PaginationListItem.vue
@@ -9,8 +9,8 @@ export interface PaginationListItemProps extends PrimitiveProps {
 </script>
 
 <script setup lang="ts">
-import { Primitive } from '@/Primitive'
 import { computed } from 'vue'
+import { Primitive } from '@/Primitive'
 import { injectPaginationRootContext } from './PaginationRoot.vue'
 
 const props = withDefaults(defineProps<PaginationListItemProps>(), { as: 'button' })
@@ -18,6 +18,8 @@ useForwardExpose()
 
 const rootContext = injectPaginationRootContext()
 const isSelected = computed(() => rootContext.page.value === props.value)
+
+const disabled = computed((): boolean => rootContext.disabled.value)
 </script>
 
 <template>
@@ -27,9 +29,9 @@ const isSelected = computed(() => rootContext.page.value === props.value)
     :aria-label="`Page ${value}`"
     :aria-current="isSelected ? 'page' : undefined"
     :data-selected="isSelected ? 'true' : undefined"
-    :disabled="rootContext.disabled.value"
+    :disabled
     :type="as === 'button' ? 'button' : undefined"
-    @click="rootContext.onPageChange(value)"
+    @click="!disabled && rootContext.onPageChange(value)"
   >
     <slot>{{ value }}</slot>
   </Primitive>
diff --git a/packages/radix-vue/src/Pagination/PaginationNext.vue b/packages/radix-vue/src/Pagination/PaginationNext.vue
index 63bbd8302..0117b0652 100644
--- a/packages/radix-vue/src/Pagination/PaginationNext.vue
+++ b/packages/radix-vue/src/Pagination/PaginationNext.vue
@@ -6,6 +6,7 @@ export interface PaginationNextProps extends PrimitiveProps {}
 </script>
 
 <script setup lang="ts">
+import { computed } from 'vue'
 import { Primitive } from '@/Primitive'
 import { injectPaginationRootContext } from './PaginationRoot.vue'
 
@@ -13,6 +14,8 @@ const props = withDefaults(defineProps<PaginationNextProps>(), { as: 'button' })
 
 useForwardExpose()
 const rootContext = injectPaginationRootContext()
+
+const disabled = computed((): boolean => rootContext.page.value === rootContext.pageCount.value || rootContext.disabled.value)
 </script>
 
 <template>
@@ -20,8 +23,8 @@ const rootContext = injectPaginationRootContext()
     v-bind="props"
     aria-label="Next Page"
     :type="as === 'button' ? 'button' : undefined"
-    :disabled="rootContext.page.value === rootContext.pageCount.value || rootContext.disabled.value"
-    @click="rootContext.onPageChange(rootContext.page.value + 1)"
+    :disabled
+    @click="!disabled && rootContext.onPageChange(rootContext.page.value + 1)"
   >
     <slot>Next page</slot>
   </Primitive>
diff --git a/packages/radix-vue/src/Pagination/PaginationPrev.vue b/packages/radix-vue/src/Pagination/PaginationPrev.vue
index dd98e773d..584b96b01 100644
--- a/packages/radix-vue/src/Pagination/PaginationPrev.vue
+++ b/packages/radix-vue/src/Pagination/PaginationPrev.vue
@@ -6,6 +6,7 @@ export interface PaginationPrevProps extends PrimitiveProps {}
 </script>
 
 <script setup lang="ts">
+import { computed } from 'vue'
 import { Primitive } from '@/Primitive'
 import { injectPaginationRootContext } from './PaginationRoot.vue'
 
@@ -13,6 +14,8 @@ const props = withDefaults(defineProps<PaginationPrevProps>(), { as: 'button' })
 
 useForwardExpose()
 const rootContext = injectPaginationRootContext()
+
+const disabled = computed((): boolean => rootContext.page.value === 1 || rootContext.disabled.value)
 </script>
 
 <template>
@@ -20,8 +23,8 @@ const rootContext = injectPaginationRootContext()
     v-bind="props"
     aria-label="Previous Page"
     :type="as === 'button' ? 'button' : undefined"
-    :disabled="rootContext.page.value === 1 || rootContext.disabled?.value"
-    @click="rootContext.onPageChange(rootContext.page.value - 1)"
+    :disabled
+    @click="!disabled && rootContext.onPageChange(rootContext.page.value - 1)"
   >
     <slot>Prev page</slot>
   </Primitive>
diff --git a/packages/radix-vue/src/Pagination/story/_Pagination.vue b/packages/radix-vue/src/Pagination/story/_Pagination.vue
index 2037e0731..f9d25aabd 100644
--- a/packages/radix-vue/src/Pagination/story/_Pagination.vue
+++ b/packages/radix-vue/src/Pagination/story/_Pagination.vue
@@ -1,28 +1,40 @@
 <script setup lang="ts">
-import { PaginationEllipsis, PaginationFirst, PaginationLast, PaginationList, PaginationListItem, PaginationNext, PaginationPrev, PaginationRoot, type PaginationRootProps } from '..'
+import { computed } from 'vue'
+import { PaginationEllipsis, type PaginationEllipsisProps, PaginationFirst, type PaginationFirstProps, PaginationLast, type PaginationLastProps, PaginationList, PaginationListItem, type PaginationListItemProps, type PaginationListProps, PaginationNext, type PaginationNextProps, PaginationPrev, type PaginationPrevProps, PaginationRoot, type PaginationRootProps } from '..'
 
-const props = withDefaults(defineProps<{
-  total?: PaginationRootProps['total']
-  showEdges?: PaginationRootProps['showEdges']
-}>(), {
-  total: 100,
-})
+const props = defineProps<{
+  root?: PaginationRootProps
+  list?: PaginationListProps
+
+  first?: PaginationFirstProps
+  prev?: PaginationPrevProps
+
+  listItem?: Partial<PaginationListItemProps>
+  ellipsis?: PaginationEllipsisProps
+
+  next?: PaginationNextProps
+  last?: PaginationLastProps
+}>()
+
+const rootProps = computed(() => ({ total: 100, ...props.root }))
 </script>
 
 <template>
-  <PaginationRoot v-bind="props">
+  <PaginationRoot v-bind="rootProps">
     <PaginationList
       v-slot="{ items }"
+      v-bind="props.list"
       class="flex items-center gap-2 "
     >
-      <PaginationFirst />
-      <PaginationPrev />
+      <PaginationFirst v-bind="props.first" />
+      <PaginationPrev v-bind="props.prev" />
       <template v-for="(page, index) in items">
         <PaginationListItem
           v-if="page.type === 'page'"
           :key="index"
           class="border rounded px-4 py-2 data-[selected]:bg-grass8"
           :value="page.value"
+          v-bind="props.listItem"
         >
           {{ page.value }}
         </PaginationListItem>
@@ -31,12 +43,13 @@ const props = withDefaults(defineProps<{
           :key="page.type"
           :index="index"
           class="border rounded px-4 py-2 "
+          v-bind="props.ellipsis"
         >
           &#8230;
         </PaginationEllipsis>
       </template>
-      <PaginationNext />
-      <PaginationLast />
+      <PaginationNext v-bind="props.next" />
+      <PaginationLast v-bind="props.last" />
     </PaginationList>
   </PaginationRoot>
 </template>