Commit 95a2b75
Android: Add new configuration for
Summary:
Following the [RFC](react-native-community/discussions-and-proposals#925), this PR adds new `DevMenuConfiguration` object and extends `ReactHost` API for passing settings to the particular `DevSupportManager`. The `DevMenuConfiguration` includes:
- isDevMenuEnabled,
- isShakeGestureEnabled,
- areKeyboardShortcutsEnabled,
## Changelog:
[ANDROID][ADDED] - Add new configuration for `RCTDevMenu`
Test Plan:
Tested with different configuration on `DevMenuConfiguration`:
<details>
<summary>Click to view code</summary>
```kt
override fun onCreate(savedInstanceState: Bundle?) {
super.onCreate(savedInstanceState)
fullyDrawnReporter.addReporter()
maybeUpdateBackgroundColor()
reactDelegate?.reactHost?.let { reactHost ->
val devMenuConfiguration = DevMenuConfiguration(
isDevMenuEnabled = true,
isShakeGestureEnabled = false,
areKeyboardShortcutsEnabled = false,
)
reactHost.setDevMenuConfiguration(devMenuConfiguration)
}
// register insets listener to update margins on the ReactRootView to avoid overlap w/ system
// bars
reactDelegate?.reactRootView?.let { rootView ->
val insetsType: Int =
WindowInsetsCompat.Type.systemBars() or WindowInsetsCompat.Type.displayCutout()
val windowInsetsListener = { view: View, windowInsets: WindowInsetsCompat ->
val insets = windowInsets.getInsets(insetsType)
(view.layoutParams as FrameLayout.LayoutParams).apply {
setMargins(insets.left, insets.top, insets.right, insets.bottom)
}
WindowInsetsCompat.CONSUMED
}
ViewCompat.setOnApplyWindowInsetsListener(rootView, windowInsetsListener)
}
}
```
</details>
Differential Revision: D81684310
Pulled By: coadoDevSupportManager (facebook#53557)1 parent 7d73d39 commit 95a2b75
File tree
9 files changed
+135
-9
lines changed- packages
- react-native/ReactAndroid
- api
- src/main/java/com/facebook/react
- devsupport
- interfaces
- runtime
- rn-tester/android/app/src/main/java/com/facebook/react/uiapp
9 files changed
+135
-9
lines changed| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
244 | 244 | | |
245 | 245 | | |
246 | 246 | | |
| 247 | + | |
247 | 248 | | |
248 | 249 | | |
249 | 250 | | |
| |||
1899 | 1900 | | |
1900 | 1901 | | |
1901 | 1902 | | |
| 1903 | + | |
| 1904 | + | |
| 1905 | + | |
| 1906 | + | |
| 1907 | + | |
| 1908 | + | |
| 1909 | + | |
| 1910 | + | |
| 1911 | + | |
| 1912 | + | |
| 1913 | + | |
| 1914 | + | |
| 1915 | + | |
| 1916 | + | |
| 1917 | + | |
| 1918 | + | |
| 1919 | + | |
1902 | 1920 | | |
1903 | 1921 | | |
1904 | 1922 | | |
| |||
1940 | 1958 | | |
1941 | 1959 | | |
1942 | 1960 | | |
| 1961 | + | |
1943 | 1962 | | |
1944 | 1963 | | |
1945 | 1964 | | |
1946 | 1965 | | |
1947 | 1966 | | |
| 1967 | + | |
1948 | 1968 | | |
1949 | 1969 | | |
1950 | 1970 | | |
1951 | 1971 | | |
1952 | 1972 | | |
1953 | 1973 | | |
| 1974 | + | |
1954 | 1975 | | |
1955 | 1976 | | |
1956 | 1977 | | |
| |||
1969 | 1990 | | |
1970 | 1991 | | |
1971 | 1992 | | |
| 1993 | + | |
1972 | 1994 | | |
1973 | 1995 | | |
1974 | 1996 | | |
| 1997 | + | |
1975 | 1998 | | |
1976 | 1999 | | |
1977 | 2000 | | |
1978 | 2001 | | |
1979 | 2002 | | |
| 2003 | + | |
1980 | 2004 | | |
1981 | 2005 | | |
1982 | 2006 | | |
| |||
2033 | 2057 | | |
2034 | 2058 | | |
2035 | 2059 | | |
| 2060 | + | |
2036 | 2061 | | |
2037 | 2062 | | |
2038 | 2063 | | |
| 2064 | + | |
2039 | 2065 | | |
2040 | 2066 | | |
2041 | 2067 | | |
2042 | 2068 | | |
2043 | 2069 | | |
| 2070 | + | |
2044 | 2071 | | |
2045 | 2072 | | |
2046 | 2073 | | |
| |||
2057 | 2084 | | |
2058 | 2085 | | |
2059 | 2086 | | |
| 2087 | + | |
2060 | 2088 | | |
2061 | 2089 | | |
2062 | 2090 | | |
| 2091 | + | |
2063 | 2092 | | |
| 2093 | + | |
2064 | 2094 | | |
2065 | 2095 | | |
2066 | 2096 | | |
| |||
2134 | 2164 | | |
2135 | 2165 | | |
2136 | 2166 | | |
| 2167 | + | |
2137 | 2168 | | |
2138 | 2169 | | |
2139 | 2170 | | |
| 2171 | + | |
2140 | 2172 | | |
2141 | 2173 | | |
2142 | 2174 | | |
2143 | 2175 | | |
2144 | 2176 | | |
| 2177 | + | |
2145 | 2178 | | |
2146 | 2179 | | |
2147 | 2180 | | |
| |||
2158 | 2191 | | |
2159 | 2192 | | |
2160 | 2193 | | |
| 2194 | + | |
2161 | 2195 | | |
2162 | 2196 | | |
2163 | 2197 | | |
| 2198 | + | |
2164 | 2199 | | |
| 2200 | + | |
2165 | 2201 | | |
2166 | 2202 | | |
2167 | 2203 | | |
| |||
3073 | 3109 | | |
3074 | 3110 | | |
3075 | 3111 | | |
| 3112 | + | |
3076 | 3113 | | |
3077 | 3114 | | |
3078 | 3115 | | |
| |||
Lines changed: 5 additions & 1 deletion
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
393 | 393 | | |
394 | 394 | | |
395 | 395 | | |
396 | | - | |
| 396 | + | |
| 397 | + | |
| 398 | + | |
| 399 | + | |
| 400 | + | |
397 | 401 | | |
398 | 402 | | |
399 | 403 | | |
| |||
Lines changed: 4 additions & 0 deletions
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
14 | 14 | | |
15 | 15 | | |
16 | 16 | | |
| 17 | + | |
17 | 18 | | |
18 | 19 | | |
19 | 20 | | |
| |||
189 | 190 | | |
190 | 191 | | |
191 | 192 | | |
| 193 | + | |
| 194 | + | |
| 195 | + | |
192 | 196 | | |
Lines changed: 16 additions & 0 deletions
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
| 1 | + | |
| 2 | + | |
| 3 | + | |
| 4 | + | |
| 5 | + | |
| 6 | + | |
| 7 | + | |
| 8 | + | |
| 9 | + | |
| 10 | + | |
| 11 | + | |
| 12 | + | |
| 13 | + | |
| 14 | + | |
| 15 | + | |
| 16 | + | |
Lines changed: 40 additions & 8 deletions
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
130 | 130 | | |
131 | 131 | | |
132 | 132 | | |
| 133 | + | |
| 134 | + | |
| 135 | + | |
| 136 | + | |
| 137 | + | |
| 138 | + | |
| 139 | + | |
| 140 | + | |
| 141 | + | |
| 142 | + | |
| 143 | + | |
| 144 | + | |
| 145 | + | |
| 146 | + | |
| 147 | + | |
| 148 | + | |
133 | 149 | | |
134 | 150 | | |
135 | 151 | | |
| |||
173 | 189 | | |
174 | 190 | | |
175 | 191 | | |
| 192 | + | |
176 | 193 | | |
177 | 194 | | |
178 | 195 | | |
| |||
187 | 204 | | |
188 | 205 | | |
189 | 206 | | |
| 207 | + | |
| 208 | + | |
| 209 | + | |
190 | 210 | | |
191 | 211 | | |
192 | 212 | | |
| |||
332 | 352 | | |
333 | 353 | | |
334 | 354 | | |
335 | | - | |
| 355 | + | |
| 356 | + | |
| 357 | + | |
| 358 | + | |
| 359 | + | |
| 360 | + | |
336 | 361 | | |
337 | 362 | | |
338 | 363 | | |
| |||
848 | 873 | | |
849 | 874 | | |
850 | 875 | | |
| 876 | + | |
| 877 | + | |
| 878 | + | |
| 879 | + | |
| 880 | + | |
| 881 | + | |
| 882 | + | |
| 883 | + | |
| 884 | + | |
| 885 | + | |
| 886 | + | |
851 | 887 | | |
852 | 888 | | |
853 | 889 | | |
| |||
857 | 893 | | |
858 | 894 | | |
859 | 895 | | |
860 | | - | |
861 | | - | |
862 | | - | |
863 | | - | |
864 | | - | |
| 896 | + | |
| 897 | + | |
865 | 898 | | |
866 | 899 | | |
867 | 900 | | |
| |||
915 | 948 | | |
916 | 949 | | |
917 | 950 | | |
918 | | - | |
919 | | - | |
| 951 | + | |
920 | 952 | | |
921 | 953 | | |
922 | 954 | | |
| |||
Lines changed: 12 additions & 0 deletions
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
72 | 72 | | |
73 | 73 | | |
74 | 74 | | |
| 75 | + | |
| 76 | + | |
| 77 | + | |
| 78 | + | |
| 79 | + | |
| 80 | + | |
| 81 | + | |
| 82 | + | |
| 83 | + | |
| 84 | + | |
| 85 | + | |
| 86 | + | |
75 | 87 | | |
76 | 88 | | |
77 | 89 | | |
| |||
Lines changed: 3 additions & 0 deletions
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
36 | 36 | | |
37 | 37 | | |
38 | 38 | | |
| 39 | + | |
| 40 | + | |
| 41 | + | |
39 | 42 | | |
40 | 43 | | |
41 | 44 | | |
| |||
Lines changed: 7 additions & 0 deletions
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
39 | 39 | | |
40 | 40 | | |
41 | 41 | | |
| 42 | + | |
42 | 43 | | |
43 | 44 | | |
44 | 45 | | |
| |||
365 | 366 | | |
366 | 367 | | |
367 | 368 | | |
| 369 | + | |
| 370 | + | |
| 371 | + | |
| 372 | + | |
| 373 | + | |
| 374 | + | |
368 | 375 | | |
369 | 376 | | |
370 | 377 | | |
| |||
Lines changed: 11 additions & 0 deletions
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
19 | 19 | | |
20 | 20 | | |
21 | 21 | | |
| 22 | + | |
22 | 23 | | |
23 | 24 | | |
24 | 25 | | |
| |||
66 | 67 | | |
67 | 68 | | |
68 | 69 | | |
| 70 | + | |
| 71 | + | |
| 72 | + | |
| 73 | + | |
| 74 | + | |
| 75 | + | |
| 76 | + | |
| 77 | + | |
| 78 | + | |
| 79 | + | |
69 | 80 | | |
70 | 81 | | |
71 | 82 | | |
| |||
0 commit comments