refactor(MenuView): replace NSMutableSet with NSHashTable for disabled views

Use NSHashTable with weak references to prevent memory leaks when views are deallocated.

Add safety checks during iteration to handle weak references properly.

Author: sbaiahmed1

ios/MenuView.mm

@@ -20,7 +20,7 @@ @implementation MenuView {
     UIColor *_checkedColor;
     UIColor *_uncheckedColor;
     BOOL _isChildViewButton;
-    NSMutableSet<UIView *> *_disabledViews;
+    NSHashTable<UIView *> *_disabledViews;
 }
 
 + (ComponentDescriptorProvider)componentDescriptorProvider
@@ -33,7 +33,7 @@ - (instancetype)initWithFrame:(CGRect)frame
     if (self = [super initWithFrame:frame]) {
         static const auto defaultProps = std::make_shared<const MenuViewProps>();
         _props = defaultProps;
-        _disabledViews = [[NSMutableSet alloc] init];
+        _disabledViews = [NSHashTable weakObjectsHashTable];
         _isChildViewButton = NO;
     }
 
@@ -121,9 +121,13 @@ - (void)disableUserInteractionRecursively:(UIView *)view
 
 - (void)restoreUserInteractionForDisabledViews
 {
-    for (UIView *view in _disabledViews) {
-        // Only restore if the view is still in the view hierarchy
-        if (view.superview != nil) {
+    // Create a copy of the objects to iterate over since NSHashTable with weak references
+    // can have objects deallocated during iteration
+    NSArray<UIView *> *viewsToRestore = [_disabledViews allObjects];
+    
+    for (UIView *view in viewsToRestore) {
+        // The view might have been deallocated (weak reference), so check if it's still valid
+        if (view && view.superview != nil) {
             view.userInteractionEnabled = YES;
         }
     }