search

sha5010 / vim.xlam

Vim experience in Excel: An add-in that brings Vim’s key bindings to Excel, enhancing navigation and efficiency.
MIT License
62 stars 5 forks source link

`nextWorkbook` と `prevWorkbook` で無限ループが発生 #21

Closed sha5010 closed 2 years ago

sha5010 commented 2 years ago

https://github.com/sha5010/vim.xlam/blob/d45fa53c0db21ba4372823bde2231b00708b79ab/src/functions/UsefulCmd.bas#L235-L259

i の計算が 1ずれているので修正が必要。

next

-     i = getWorkbookIndex(ActiveWorkbook) - 1 
+     i = getWorkbookIndex(ActiveWorkbook)
      Do 
-         i = ((i + 1) Mod Workbooks.Count) + 1
+         i = (i Mod Workbooks.Count) + 1

prev

-     i = getWorkbookIndex(ActiveWorkbook) - 1 
+     i = getWorkbookIndex(ActiveWorkbook)
      Do 
-         i = ((i - 1 + Workbooks.Count) Mod Workbooks.Count) + 1
+         i = ((i - 2 + Workbooks.Count) Mod Workbooks.Count) + 1