Skip to content

Commit 3608949

Browse files
alanleedevfacebook-github-bot
alanleedev
authored and
facebook-github-bot
committed
Remove UIManager.{show|dismiss}PopupMenu (#44096)
Summary: These Android only APIs have been deprecated and are being removed for 0.75 release. Changelog: [Android][Removed] - UIManager.showPopupMenu() and UIManager.dismissPopupMenu() have been removed Differential Revision: D56041827
1 parent 7047563 commit 3608949

9 files changed

Lines changed: 1 addition & 247 deletions

File tree

‎packages/react-native/Libraries/ReactNative/BridgelessUIManager.js‎

Lines changed: 1 addition & 21 deletions
Original file line numberDiff line numberDiff line change
@@ -160,27 +160,7 @@ const UIManagerJSUnusedInNewArchAPIs = {
160160
* them from React Native.
161161
*/
162162
const UIManagerJSDeprecatedPlatformAPIs = Platform.select({
163-
android: {
164-
// TODO(T175424986): Remove UIManager.showPopupMenu() in React Native v0.75.
165-
showPopupMenu: (
166-
reactTag: number,
167-
items: Array<string>,
168-
error: (error: Object) => void,
169-
success: (event: string, selected?: number) => void,
170-
): void => {
171-
raiseSoftError(
172-
'showPopupMenu',
173-
'Please use the <PopupMenuAndroid /> component instead.',
174-
);
175-
},
176-
// TODO(T175424986): Remove UIManager.dismissPopupMenu() in React Native v0.75.
177-
dismissPopupMenu: (): void => {
178-
raiseSoftError(
179-
'dismissPopupMenu',
180-
'Please use the <PopupMenuAndroid /> component instead.',
181-
);
182-
},
183-
},
163+
android: {},
184164
});
185165

186166
const UIManagerJSPlatformAPIs = Platform.select({

‎packages/react-native/Libraries/ReactNative/UIManagerProperties.js‎

Lines changed: 0 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -30,7 +30,6 @@ module.exports = [
3030
'clearJSResponder',
3131
'configureNextLayoutAnimation',
3232
'createView',
33-
'dismissPopupMenu',
3433
'dispatchViewManagerCommand',
3534
'findSubviewIn',
3635
'getConstantsForViewManager',
@@ -45,10 +44,8 @@ module.exports = [
4544
'setChildren',
4645
'setJSResponder',
4746
'setLayoutAnimationEnabledExperimental',
48-
'showPopupMenu',
4947
'updateView',
5048
'viewIsDescendantOf',
51-
'PopupMenu',
5249
'LazyViewManagersEnabled',
5350
'ViewManagerNames',
5451
'StyleConstants',

‎packages/react-native/ReactAndroid/api/ReactAndroid.api‎

Lines changed: 0 additions & 8 deletions
Original file line numberDiff line numberDiff line change
@@ -4220,7 +4220,6 @@ public class com/facebook/react/uimanager/NativeViewHierarchyManager {
42204220
protected final fun addRootViewGroup (ILandroid/view/View;)V
42214221
public fun clearJSResponder ()V
42224222
public fun createView (Lcom/facebook/react/uimanager/ThemedReactContext;ILjava/lang/String;Lcom/facebook/react/uimanager/ReactStylesDiffMap;)V
4223-
public fun dismissPopupMenu ()V
42244223
public fun dispatchCommand (IILcom/facebook/react/bridge/ReadableArray;)V
42254224
public fun dispatchCommand (ILjava/lang/String;Lcom/facebook/react/bridge/ReadableArray;)V
42264225
protected fun dropView (Landroid/view/View;)V
@@ -4237,7 +4236,6 @@ public class com/facebook/react/uimanager/NativeViewHierarchyManager {
42374236
public fun setChildren (ILcom/facebook/react/bridge/ReadableArray;)V
42384237
public fun setJSResponder (IIZ)V
42394238
public fun setLayoutAnimationEnabled (Z)V
4240-
public fun showPopupMenu (ILcom/facebook/react/bridge/ReadableArray;Lcom/facebook/react/bridge/Callback;Lcom/facebook/react/bridge/Callback;)V
42414239
public fun updateInstanceHandle (IJ)V
42424240
public fun updateLayout (IIIIII)V
42434241
public fun updateProperties (ILcom/facebook/react/uimanager/ReactStylesDiffMap;)V
@@ -4946,7 +4944,6 @@ public class com/facebook/react/uimanager/UIImplementation {
49464944
protected fun createRootShadowNode ()Lcom/facebook/react/uimanager/ReactShadowNode;
49474945
protected fun createShadowNode (Ljava/lang/String;)Lcom/facebook/react/uimanager/ReactShadowNode;
49484946
public fun createView (ILjava/lang/String;ILcom/facebook/react/bridge/ReadableMap;)V
4949-
public fun dismissPopupMenu ()V
49504947
public fun dispatchViewManagerCommand (IILcom/facebook/react/bridge/ReadableArray;)V
49514948
public fun dispatchViewManagerCommand (ILjava/lang/String;Lcom/facebook/react/bridge/ReadableArray;)V
49524949
public fun dispatchViewUpdates (I)V
@@ -4982,7 +4979,6 @@ public class com/facebook/react/uimanager/UIImplementation {
49824979
public fun setLayoutUpdateListener (Lcom/facebook/react/uimanager/UIImplementation$LayoutUpdateListener;)V
49834980
public fun setViewHierarchyUpdateDebugListener (Lcom/facebook/react/uimanager/debug/NotThreadSafeViewHierarchyUpdateDebugListener;)V
49844981
public fun setViewLocalData (ILjava/lang/Object;)V
4985-
public fun showPopupMenu (ILcom/facebook/react/bridge/ReadableArray;Lcom/facebook/react/bridge/Callback;Lcom/facebook/react/bridge/Callback;)V
49864982
public fun synchronouslyUpdateViewOnUIThread (ILcom/facebook/react/uimanager/ReactStylesDiffMap;)V
49874983
public fun updateNodeSize (III)V
49884984
public fun updateRootView (III)V
@@ -5026,7 +5022,6 @@ public class com/facebook/react/uimanager/UIManagerModule : com/facebook/react/b
50265022
public fun configureNextLayoutAnimation (Lcom/facebook/react/bridge/ReadableMap;Lcom/facebook/react/bridge/Callback;Lcom/facebook/react/bridge/Callback;)V
50275023
public static fun createConstants (Ljava/util/List;Ljava/util/Map;Ljava/util/Map;)Ljava/util/Map;
50285024
public fun createView (ILjava/lang/String;ILcom/facebook/react/bridge/ReadableMap;)V
5029-
public fun dismissPopupMenu ()V
50305025
public fun dispatchCommand (IILcom/facebook/react/bridge/ReadableArray;)V
50315026
public fun dispatchCommand (ILjava/lang/String;Lcom/facebook/react/bridge/ReadableArray;)V
50325027
public fun dispatchViewManagerCommand (ILcom/facebook/react/bridge/Dynamic;Lcom/facebook/react/bridge/ReadableArray;)V
@@ -5070,7 +5065,6 @@ public class com/facebook/react/uimanager/UIManagerModule : com/facebook/react/b
50705065
public fun setLayoutAnimationEnabledExperimental (Z)V
50715066
public fun setViewHierarchyUpdateDebugListener (Lcom/facebook/react/uimanager/debug/NotThreadSafeViewHierarchyUpdateDebugListener;)V
50725067
public fun setViewLocalData (ILjava/lang/Object;)V
5073-
public fun showPopupMenu (ILcom/facebook/react/bridge/ReadableArray;Lcom/facebook/react/bridge/Callback;Lcom/facebook/react/bridge/Callback;)V
50745068
public fun startSurface (Landroid/view/View;Ljava/lang/String;Lcom/facebook/react/bridge/WritableMap;II)I
50755069
public fun stopSurface (I)V
50765070
public fun synchronouslyUpdateViewOnUIThread (ILcom/facebook/react/bridge/ReadableMap;)V
@@ -5101,7 +5095,6 @@ public class com/facebook/react/uimanager/UIViewOperationQueue {
51015095
public fun enqueueClearJSResponder ()V
51025096
public fun enqueueConfigureLayoutAnimation (Lcom/facebook/react/bridge/ReadableMap;Lcom/facebook/react/bridge/Callback;)V
51035097
public fun enqueueCreateView (Lcom/facebook/react/uimanager/ThemedReactContext;ILjava/lang/String;Lcom/facebook/react/uimanager/ReactStylesDiffMap;)V
5104-
public fun enqueueDismissPopupMenu ()V
51055098
public fun enqueueDispatchCommand (IILcom/facebook/react/bridge/ReadableArray;)V
51065099
public fun enqueueDispatchCommand (ILjava/lang/String;Lcom/facebook/react/bridge/ReadableArray;)V
51075100
public fun enqueueFindTargetForTouch (IFFLcom/facebook/react/bridge/Callback;)V
@@ -5114,7 +5107,6 @@ public class com/facebook/react/uimanager/UIViewOperationQueue {
51145107
public fun enqueueSetChildren (ILcom/facebook/react/bridge/ReadableArray;)V
51155108
public fun enqueueSetJSResponder (IIZ)V
51165109
public fun enqueueSetLayoutAnimationEnabled (Z)V
5117-
public fun enqueueShowPopupMenu (ILcom/facebook/react/bridge/ReadableArray;Lcom/facebook/react/bridge/Callback;Lcom/facebook/react/bridge/Callback;)V
51185110
public fun enqueueUIBlock (Lcom/facebook/react/uimanager/UIBlock;)V
51195111
protected fun enqueueUIOperation (Lcom/facebook/react/uimanager/UIViewOperationQueue$UIOperation;)V
51205112
public fun enqueueUpdateExtraData (ILjava/lang/Object;)V

‎packages/react-native/ReactAndroid/src/main/java/com/facebook/react/uimanager/NativeViewHierarchyManager.java‎

Lines changed: 0 additions & 83 deletions
Original file line numberDiff line numberDiff line change
@@ -12,12 +12,9 @@
1212
import android.graphics.RectF;
1313
import android.util.SparseArray;
1414
import android.util.SparseBooleanArray;
15-
import android.view.Menu;
16-
import android.view.MenuItem;
1715
import android.view.View;
1816
import android.view.ViewGroup;
1917
import android.view.ViewParent;
20-
import android.widget.PopupMenu;
2118
import androidx.annotation.Nullable;
2219
import com.facebook.common.logging.FLog;
2320
import com.facebook.react.R;
@@ -81,7 +78,6 @@ public class NativeViewHierarchyManager {
8178
private final RectF mBoundingBox = new RectF();
8279

8380
private boolean mLayoutAnimationEnabled;
84-
private PopupMenu mPopupMenu;
8581
private HashMap<Integer, Set<Integer>> mPendingDeletionsForTag;
8682

8783
public NativeViewHierarchyManager(ViewManagerRegistry viewManagers) {
@@ -862,85 +858,6 @@ public synchronized void dispatchCommand(
862858
viewManager.receiveCommand(view, commandId, args);
863859
}
864860

865-
/**
866-
* Show a {@link PopupMenu}.
867-
*
868-
* <p>This is deprecated, please use the <PopupMenuAndroid /> component instead.
869-
*
870-
* <p>TODO(T175424986): Remove UIManager.showPopupMenu() in React Native v0.75.
871-
*
872-
* @param reactTag the tag of the anchor view (the PopupMenu is displayed next to this view); this
873-
* needs to be the tag of a native view (shadow views can not be anchors)
874-
* @param items the menu items as an array of strings
875-
* @param success will be called with the position of the selected item as the first argument, or
876-
* no arguments if the menu is dismissed
877-
*/
878-
@Deprecated
879-
public synchronized void showPopupMenu(
880-
int reactTag, ReadableArray items, Callback success, Callback error) {
881-
UiThreadUtil.assertOnUiThread();
882-
View anchor = mTagsToViews.get(reactTag);
883-
if (anchor == null) {
884-
error.invoke("Can't display popup. Could not find view with tag " + reactTag);
885-
return;
886-
}
887-
mPopupMenu = new PopupMenu(getReactContextForView(reactTag), anchor);
888-
889-
Menu menu = mPopupMenu.getMenu();
890-
for (int i = 0; i < items.size(); i++) {
891-
menu.add(Menu.NONE, Menu.NONE, i, items.getString(i));
892-
}
893-
894-
PopupMenuCallbackHandler handler = new PopupMenuCallbackHandler(success);
895-
mPopupMenu.setOnMenuItemClickListener(handler);
896-
mPopupMenu.setOnDismissListener(handler);
897-
898-
mPopupMenu.show();
899-
}
900-
901-
/**
902-
* This is deprecated, please use the <PopupMenuAndroid /> component instead.
903-
*
904-
* <p>TODO(T175424986): Remove UIManager.dismissPopupMenu() in React Native v0.75.
905-
*
906-
* <p>Dismiss the last opened PopupMenu {@link PopupMenu}.
907-
*/
908-
@Deprecated
909-
public void dismissPopupMenu() {
910-
if (mPopupMenu != null) {
911-
mPopupMenu.dismiss();
912-
}
913-
}
914-
915-
private static class PopupMenuCallbackHandler
916-
implements PopupMenu.OnMenuItemClickListener, PopupMenu.OnDismissListener {
917-
918-
final Callback mSuccess;
919-
boolean mConsumed = false;
920-
921-
private PopupMenuCallbackHandler(Callback success) {
922-
mSuccess = success;
923-
}
924-
925-
@Override
926-
public void onDismiss(PopupMenu menu) {
927-
if (!mConsumed) {
928-
mSuccess.invoke(UIManagerModuleConstants.ACTION_DISMISSED);
929-
mConsumed = true;
930-
}
931-
}
932-
933-
@Override
934-
public boolean onMenuItemClick(MenuItem item) {
935-
if (!mConsumed) {
936-
mSuccess.invoke(UIManagerModuleConstants.ACTION_ITEM_SELECTED, item.getOrder());
937-
mConsumed = true;
938-
return true;
939-
}
940-
return false;
941-
}
942-
}
943-
944861
/**
945862
* @return Themed React context for view with a given {@param reactTag} - it gets the context
946863
* directly from the view using {@link View#getContext}.

‎packages/react-native/ReactAndroid/src/main/java/com/facebook/react/uimanager/UIImplementation.java‎

Lines changed: 0 additions & 30 deletions
Original file line numberDiff line numberDiff line change
@@ -741,36 +741,6 @@ public void dispatchViewManagerCommand(
741741
mOperationsQueue.enqueueDispatchCommand(reactTag, commandId, commandArgs);
742742
}
743743

744-
/**
745-
* Show a PopupMenu.
746-
*
747-
* <p>This is deprecated, please use the <PopupMenuAndroid /> component instead.
748-
*
749-
* <p>TODO(T175424986): Remove UIManager.showPopupMenu() in React Native v0.75.
750-
*
751-
* @param reactTag the tag of the anchor view (the PopupMenu is displayed next to this view); this
752-
* needs to be the tag of a native view (shadow views can not be anchors)
753-
* @param items the menu items as an array of strings
754-
* @param error will be called if there is an error displaying the menu
755-
* @param success will be called with the position of the selected item as the first argument, or
756-
* no arguments if the menu is dismissed
757-
*/
758-
@Deprecated
759-
public void showPopupMenu(int reactTag, ReadableArray items, Callback error, Callback success) {
760-
boolean viewExists = checkOrAssertViewExists(reactTag, "showPopupMenu");
761-
if (!viewExists) {
762-
return;
763-
}
764-
765-
mOperationsQueue.enqueueShowPopupMenu(reactTag, items, error, success);
766-
}
767-
768-
/** TODO(T175424986): Remove UIManager.dismissPopupMenu() in React Native v0.75. */
769-
@Deprecated
770-
public void dismissPopupMenu() {
771-
mOperationsQueue.enqueueDismissPopupMenu();
772-
}
773-
774744
public void sendAccessibilityEvent(int tag, int eventType) {
775745
mOperationsQueue.enqueueSendAccessibilityEvent(tag, eventType);
776746
}

‎packages/react-native/ReactAndroid/src/main/java/com/facebook/react/uimanager/UIManagerModule.java‎

Lines changed: 0 additions & 31 deletions
Original file line numberDiff line numberDiff line change
@@ -615,37 +615,6 @@ public void dispatchCommand(int reactTag, String commandId, @Nullable ReadableAr
615615
mUIImplementation.dispatchViewManagerCommand(reactTag, commandId, commandArgs);
616616
}
617617

618-
/**
619-
* Show a PopupMenu.
620-
*
621-
* <p>This is deprecated, please use the <PopupMenuAndroid /> component instead.
622-
*
623-
* <p>TODO(T175424986): Remove UIManager.showPopupMenu() in React Native v0.75.
624-
*
625-
* @param reactTag the tag of the anchor view (the PopupMenu is displayed next to this view); this
626-
* needs to be the tag of a native view (shadow views can not be anchors)
627-
* @param items the menu items as an array of strings
628-
* @param error will be called if there is an error displaying the menu
629-
* @param success will be called with the position of the selected item as the first argument, or
630-
* no arguments if the menu is dismissed
631-
*/
632-
@ReactMethod
633-
@Deprecated
634-
public void showPopupMenu(int reactTag, ReadableArray items, Callback error, Callback success) {
635-
mUIImplementation.showPopupMenu(reactTag, items, error, success);
636-
}
637-
638-
/**
639-
* This is deprecated, please use the <PopupMenuAndroid /> component instead.
640-
*
641-
* <p>TODO(T175424986): Remove UIManager.dismissPopupMenu() in React Native v0.75.
642-
*/
643-
@ReactMethod
644-
@Deprecated
645-
public void dismissPopupMenu() {
646-
mUIImplementation.dismissPopupMenu();
647-
}
648-
649618
/**
650619
* LayoutAnimation API on Android is currently experimental. Therefore, it needs to be enabled
651620
* explicitly in order to avoid regression in existing application written for iOS using this API.

‎packages/react-native/ReactAndroid/src/main/java/com/facebook/react/uimanager/UIManagerModuleConstants.java‎

Lines changed: 0 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -104,11 +104,6 @@ public static Map<String, Object> getConstants() {
104104
"unspecified",
105105
PointerEvents.AUTO.ordinal())));
106106

107-
constants.put(
108-
"PopupMenu",
109-
MapBuilder.of(
110-
ACTION_DISMISSED, ACTION_DISMISSED, ACTION_ITEM_SELECTED, ACTION_ITEM_SELECTED));
111-
112107
constants.put(
113108
"AccessibilityEventTypes",
114109
MapBuilder.of(

‎packages/react-native/ReactAndroid/src/main/java/com/facebook/react/uimanager/UIViewOperationQueue.java‎

Lines changed: 0 additions & 59 deletions
Original file line numberDiff line numberDiff line change
@@ -343,44 +343,6 @@ public int getRetries() {
343343
}
344344
}
345345

346-
/**
347-
* This is deprecated, please use the <PopupMenuAndroid /> component instead.
348-
*
349-
* <p>TODO(T175424986): Remove UIManager.showPopupMenu() in React Native v0.75.
350-
*/
351-
@Deprecated
352-
private final class ShowPopupMenuOperation extends ViewOperation {
353-
354-
private final ReadableArray mItems;
355-
private final Callback mError;
356-
private final Callback mSuccess;
357-
358-
public ShowPopupMenuOperation(int tag, ReadableArray items, Callback error, Callback success) {
359-
super(tag);
360-
mItems = items;
361-
mError = error;
362-
mSuccess = success;
363-
}
364-
365-
@Override
366-
public void execute() {
367-
mNativeViewHierarchyManager.showPopupMenu(mTag, mItems, mSuccess, mError);
368-
}
369-
}
370-
371-
/**
372-
* This is deprecated, please use the <PopupMenuAndroid /> component instead.
373-
*
374-
* <p>TODO(T175424986): Remove UIManager.dismissPopupMenu() in React Native v0.75.
375-
*/
376-
@Deprecated
377-
private final class DismissPopupMenuOperation implements UIOperation {
378-
@Override
379-
public void execute() {
380-
mNativeViewHierarchyManager.dismissPopupMenu();
381-
}
382-
}
383-
384346
/** A spec for animation operations (add/remove) */
385347
private abstract static class AnimationOperation implements UIViewOperationQueue.UIOperation {
386348

@@ -715,27 +677,6 @@ public void enqueueUpdateExtraData(int reactTag, Object extraData) {
715677
mOperations.add(new UpdateViewExtraData(reactTag, extraData));
716678
}
717679

718-
/**
719-
* This is deprecated, please use the <PopupMenuAndroid /> component instead.
720-
*
721-
* <p>TODO(T175424986): Remove UIManager.showPopupMenu() in React Native v0.75.
722-
*/
723-
@Deprecated
724-
public void enqueueShowPopupMenu(
725-
int reactTag, ReadableArray items, Callback error, Callback success) {
726-
mOperations.add(new ShowPopupMenuOperation(reactTag, items, error, success));
727-
}
728-
729-
/**
730-
* This is deprecated, please use the <PopupMenuAndroid /> component instead.
731-
*
732-
* <p>TODO(T175424986): Remove UIManager.dismissPopupMenu() in React Native v0.75.
733-
*/
734-
@Deprecated
735-
public void enqueueDismissPopupMenu() {
736-
mOperations.add(new DismissPopupMenuOperation());
737-
}
738-
739680
public void enqueueCreateView(
740681
ThemedReactContext themedContext,
741682
int viewReactTag,

‎packages/react-native/src/private/specs/modules/NativeUIManager.js‎

Lines changed: 0 additions & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -101,13 +101,6 @@ export interface Spec extends TurboModule {
101101
+getDefaultEventTypes?: () => Array<string>;
102102
+setLayoutAnimationEnabledExperimental?: (enabled: boolean) => void;
103103
+sendAccessibilityEvent?: (reactTag: number, eventType: number) => void;
104-
+showPopupMenu?: (
105-
reactTag: number,
106-
items: Array<string>,
107-
error: (error: Object) => void,
108-
success: (event: string, selected?: number) => void,
109-
) => void;
110-
+dismissPopupMenu?: () => void;
111104

112105
// ios only
113106
+lazilyLoadView?: (name: string) => Object; // revisit return

0 commit comments

Comments
 (0)
⚡