fix incorrect activeIndex when handling Focus.Previous

RobinMalfait · RobinMalfait · commit bc4a74494774 · 2024-01-06T01:51:07.000+01:00
This is already available in the React version, now let's sync it in the
Vue version as well.
diff --git a/packages/@headlessui-vue/src/utils/calculate-active-index.ts b/packages/@headlessui-vue/src/utils/calculate-active-index.ts
@@ -48,6 +48,9 @@ export function calculateActiveIndex<TItem>(
     }
 
     case Focus.Previous: {
+      // If nothing is active, focus the last relevant item
+      if (activeIndex === -1) activeIndex = items.length
+
       for (let i = activeIndex - 1; i >= 0; --i) {
         if (!resolvers.resolveDisabled(items[i], i, items)) {
           return i

Original file line number	Diff line number	Diff line change
`@@ -48,6 +48,9 @@ export function calculateActiveIndex<TItem>(`
`48`	`48`	`}`
`49`	`49`
`50`	`50`	`case Focus.Previous: {`
	`51`	`+ // If nothing is active, focus the last relevant item`
	`52`	`+ if (activeIndex === -1) activeIndex = items.length`
	`53`	`+`
`51`	`54`	`for (let i = activeIndex - 1; i >= 0; --i) {`
`52`	`55`	`if (!resolvers.resolveDisabled(items[i], i, items)) {`
`53`	`56`	`return i`