fix(tags): suggested changes done

Author: blunteshwar

tools/reactive-controllers/src/FocusGroup.ts

@@ -154,19 +154,16 @@ export class FocusGroupController<T extends HTMLElement>
             this._listenerScope
         );
     }
-    /* In  handleItemMutation() method the first if condition is checking if the element is not focused or if the element's children's length is not decreasing then it means no element has been deleted and we must return.
-    Then we are checking if the deleted element was the focused one before the deletion if so then we need to proceed else we casn return;
-     */
+    /*  In  handleItemMutation() method the first if condition is checking if the element is not focused or if the element's children's length is not decreasing then it means no element has been deleted and we must return.
+        Then we are checking if the deleted element was the focused one before the deletion if so then we need to proceed else we casn return;
+    */
     handleItemMutation(): void {
-        if (
-            this._currentIndex == -1 ||
-            this.elements.length <= this._elements.length
-        )
-            return;
-        this.clearElementCache();
+        if (this._currentIndex == -1) return;
+        const lengthOfElementsBeforeMutation = this.elements.length;
         const focusedElement = this.elements[this.currentIndex];
+        this.clearElementCache();
+        if (lengthOfElementsBeforeMutation <= this.elements.length) return;
         if (this.elements.includes(focusedElement)) return;
-
         const moveToNextElement = this.currentIndex !== this.elements.length;
         const diff = moveToNextElement ? 1 : -1;
         if (moveToNextElement) {