refactor: rename nextOf and previousOf to nextItem and previousItem

reallygoodsoftware · Nov 9, 2023 · c3dfe15 · c3dfe15
1 parent 049e353
commit c3dfe15
Show file tree

Hide file tree

Showing 3 changed files with 11 additions and 11 deletions.
diff --git a/index.html b/index.html
@@ -365,8 +365,8 @@ <h3 class="font-bold font-mono">Multi Select:</h3>
             :enter="selectedTags = selectedTags.add(selectedTag);
                     filteredTags = filteredTags.remove(selectedTag);
                     selectedTag = filteredTags.first"
-            :keyup.up="selectedTag = filteredTags.previousOf(selectedTag)"
-            :keyup.down="selectedTag = filteredTags.nextOf(selectedTag)"
+            :keyup.up="selectedTag = filteredTags.previousItem(selectedTag)"
+            :keyup.down="selectedTag = filteredTags.nextItem(selectedTag)"
           />
 
           <div :clickout="filteredTags = []; selectedTag = null" :class="filteredTags.length ? '' : 'hidden'" class="bg-white rounded">
@@ -423,8 +423,8 @@ <h3 class="font-bold font-mono">Multi Select:</h3>
    :enter=&quot;selectedTags = selectedTags.add(selectedTag);
    filteredTags = filteredTags.remove(selectedTag);
    selectedTag = filteredTags.first&quot;
-   :keyup.up=&quot;selectedTag = filteredTags.previousOf(selectedTag)&quot;
-   :keyup.down=&quot;selectedTag = filteredTags.nextOf(selectedTag)&quot;
+   :keyup.up=&quot;selectedTag = filteredTags.previousItem(selectedTag)&quot;
+   :keyup.down=&quot;selectedTag = filteredTags.nextItem(selectedTag)&quot;
 /&gt;
 
 &lt;div :clickout=&quot;filteredTags = []; selectedTag = null&quot; :class=&quot;filteredTags.length ? '' : 'hidden' &quot;  class=&quot;bg-white rounded&quot;&gt;

diff --git a/lib/array.js b/lib/array.js
@@ -11,12 +11,12 @@ export default class MiniArray extends Array {
     return this.at(-1);
   }
 
-  nextOf(item) {
+  nextItem(item) {
     const nextIndex = this.indexOf(item) + 1;
     return nextIndex >= this.length ? this.first : this.at(nextIndex);
   }
 
-  previousOf(item) {
+  previousItem(item) {
     const previousIndex = this.indexOf(item) - 1;
     return previousIndex < 0 ? this.last : this.at(previousIndex);
   }

diff --git a/readme.md b/readme.md
@@ -160,17 +160,17 @@ Here are the custom array methods which are available for you to use:
   array = ['Tag 1', 'Tag 2', 'Tag 3', 'Tag 4']
   array.remove('Tag 2') // returns ['Tag 1', 'Tag 3', 'Tag 4']
   ```
-- `nextOf` - gets the next item in the array.
-  Usage: `array.nextOf('item')`
+- `nextItem` - gets the next item in the array.
+  Usage: `array.nextItem('item')`
   ```js
   array = ['Tag 1', 'Tag 2', 'Tag 3', 'Tag 4']
-  array.nextOf('Tag 2') // returns 'Tag 3'
+  array.nextItem('Tag 2') // returns 'Tag 3'
   ```
-- `previousOf` - gets the next item in the array.
+- `previousItem` - gets the next item in the array.
   Usage: `array.previousOf('item')`
   ```js
   array = ['Tag 1', 'Tag 2', 'Tag 3', 'Tag 4']
-  array.previousOf('Tag 2') // returns 'Tag 1'
+  array.previousItem('Tag 2') // returns 'Tag 1'
   ```
 
 To trigger a re-render you need to update the variable: