Skip to content

Commit

Permalink
Revert "feat: implement freezeOnBlur (#207)"
Browse files Browse the repository at this point in the history
This reverts commit c9f13ad.
  • Loading branch information
okwasniewski committed Dec 18, 2024
1 parent 6e19b1a commit 7855b14
Show file tree
Hide file tree
Showing 10 changed files with 7 additions and 190 deletions.
8 changes: 4 additions & 4 deletions apps/example/ios/Podfile.lock
Original file line number Diff line number Diff line change
Expand Up @@ -1209,7 +1209,7 @@ PODS:
- ReactCommon/turbomodule/bridging
- ReactCommon/turbomodule/core
- Yoga
- react-native-bottom-tabs (0.7.6):
- react-native-bottom-tabs (0.7.3):
- DoubleConversion
- glog
- RCT-Folly (= 2024.01.01.00)
Expand All @@ -1222,7 +1222,7 @@ PODS:
- React-graphics
- React-ImageManager
- React-jsi
- react-native-bottom-tabs/common (= 0.7.6)
- react-native-bottom-tabs/common (= 0.7.3)
- React-NativeModulesApple
- React-RCTFabric
- React-rendererdebug
Expand All @@ -1234,7 +1234,7 @@ PODS:
- SDWebImageSVGCoder (>= 1.7.0)
- SwiftUIIntrospect (~> 1.0)
- Yoga
- react-native-bottom-tabs/common (0.7.6):
- react-native-bottom-tabs/common (0.7.3):
- DoubleConversion
- glog
- RCT-Folly (= 2024.01.01.00)
Expand Down Expand Up @@ -1943,7 +1943,7 @@ SPEC CHECKSUMS:
React-logger: d79b704bf215af194f5213a6b7deec50ba8e6a9b
React-Mapbuffer: b982d5bba94a8bc073bda48f0d27c9b28417fae3
React-microtasksnativemodule: 8fa285fed833a04a754bf575f8ded65fc240b88d
react-native-bottom-tabs: 084cfd4d4b1e74c03f4196b3f62d39445882f45f
react-native-bottom-tabs: b6b3dc2e971c860a0a6d763701929d1899f666a0
react-native-safe-area-context: 73505107f7c673cd550a561aeb6271f152c483b6
React-nativeconfig: 8c83d992b9cc7d75b5abe262069eaeea4349f794
React-NativeModulesApple: b8465afc883f5bf3fe8bac3767e394d581a5f123
Expand Down
5 changes: 0 additions & 5 deletions apps/example/src/App.tsx
Original file line number Diff line number Diff line change
Expand Up @@ -27,7 +27,6 @@ import TintColorsExample from './Examples/TintColors';
import NativeBottomTabsEmbeddedStacks from './Examples/NativeBottomTabsEmbeddedStacks';
import NativeBottomTabsSVGs from './Examples/NativeBottomTabsSVGs';
import NativeBottomTabsRemoteIcons from './Examples/NativeBottomTabsRemoteIcons';
import NativeBottomTabsFreezeOnBlur from './Examples/NativeBottomTabsFreezeOnBlur';

const FourTabsIgnoreSafeArea = () => {
return <FourTabs ignoresTopSafeArea />;
Expand Down Expand Up @@ -103,10 +102,6 @@ const examples = [
name: 'Four Tabs - Transparent scroll edge appearance',
platform: 'ios',
},
{
component: NativeBottomTabsFreezeOnBlur,
name: 'Native Bottom Tabs with freezeOnBlur',
},
{
component: FourTabsOpaqueScrollEdgeAppearance,
name: 'Four Tabs - Opaque scroll edge appearance',
Expand Down
115 changes: 0 additions & 115 deletions apps/example/src/Examples/NativeBottomTabsFreezeOnBlur.tsx

This file was deleted.

9 changes: 1 addition & 8 deletions packages/react-native-bottom-tabs/package.json
Original file line number Diff line number Diff line change
Expand Up @@ -77,18 +77,11 @@
"react": "18.3.1",
"react-native": "0.75.4",
"react-native-builder-bob": "^0.32.1",
"react-native-screens": "4.3.0",
"typescript": "^5.2.2"
},
"peerDependencies": {
"react": "*",
"react-native": "*",
"react-native-screens": ">=3.29.0"
},
"peerDependenciesMeta": {
"react-native-screens": {
"optional": true
}
"react-native": "*"
},
"react-native-builder-bob": {
"source": "src",
Expand Down
26 changes: 0 additions & 26 deletions packages/react-native-bottom-tabs/src/Screen.tsx

This file was deleted.

16 changes: 2 additions & 14 deletions packages/react-native-bottom-tabs/src/TabView.tsx
Original file line number Diff line number Diff line change
Expand Up @@ -15,7 +15,6 @@ import type { ImageSource } from 'react-native/Libraries/Image/ImageSource';
import TabViewAdapter from './TabViewAdapter';
import useLatestCallback from 'use-latest-callback';
import type { BaseRoute, NavigationState } from './types';
import { Screen } from './Screen';

const isAppleSymbol = (icon: any): icon is { sfSymbol: string } =>
icon?.sfSymbol;
Expand Down Expand Up @@ -117,13 +116,6 @@ interface Props<Route extends BaseRoute> {
*/
getTestID?: (props: { route: Route }) => string | undefined;

/**
* Get freezeOnBlur for the current screen. Uses false by default.
* Defaults to `true` when `enableFreeze()` is run at the top of the application.
*
*/
getFreezeOnBlur?: (props: { route: Route }) => boolean | undefined;

/**
* Background color of the tab bar.
*/
Expand Down Expand Up @@ -168,7 +160,6 @@ const TabView = <Route extends BaseRoute>({
tabBarInactiveTintColor: inactiveTintColor,
getLazy = ({ route }: { route: Route }) => route.lazy,
getLabelText = ({ route }: { route: Route }) => route.title,
getFreezeOnBlur = ({ route }: { route: Route }) => route.freezeOnBlur,
getIcon = ({ route, focused }: { route: Route; focused: boolean }) =>
route.unfocusedIcon
? focused
Expand Down Expand Up @@ -320,14 +311,11 @@ const TabView = <Route extends BaseRoute>({
const focused = route.key === focusedKey;
const opacity = focused ? 1 : 0;
const zIndex = focused ? 0 : -1;
const freezeOnBlur = getFreezeOnBlur({ route });

return (
<Screen
<View
key={route.key}
collapsable={false}
visible={focused}
freezeOnBlur={freezeOnBlur}
pointerEvents={focused ? 'auto' : 'none'}
accessibilityElementsHidden={!focused}
importantForAccessibility={
Expand All @@ -343,7 +331,7 @@ const TabView = <Route extends BaseRoute>({
route,
jumpTo,
})}
</Screen>
</View>
);
})}
</TabViewAdapter>
Expand Down
1 change: 0 additions & 1 deletion packages/react-native-bottom-tabs/src/types.ts
Original file line number Diff line number Diff line change
Expand Up @@ -15,7 +15,6 @@ export type BaseRoute = {
activeTintColor?: string;
hidden?: boolean;
testID?: string;
freezeOnBlur?: boolean;
};

export type NavigationState<Route extends BaseRoute> = {
Expand Down
9 changes: 0 additions & 9 deletions packages/react-navigation/src/types.ts
Original file line number Diff line number Diff line change
Expand Up @@ -91,14 +91,6 @@ export type NativeBottomTabNavigationOptions = {
* TestID for the tab.
*/
tabBarButtonTestID?: string;

/**
* Whether inactive screens should be suspended from re-rendering. Defaults to `false`.
* Defaults to `true` when `enableFreeze()` is run at the top of the application.
*
* Only supported on iOS and Android.
*/
freezeOnBlur?: boolean;
};

export type NativeBottomTabDescriptor = Descriptor<
Expand All @@ -125,6 +117,5 @@ export type NativeBottomTabNavigationConfig = Partial<
| 'onTabLongPress'
| 'getActiveTintColor'
| 'getTestID'
| 'getFreezeOnBlur'
>
>;
3 changes: 0 additions & 3 deletions packages/react-navigation/src/views/NativeBottomTabView.tsx
Original file line number Diff line number Diff line change
Expand Up @@ -45,9 +45,6 @@ export default function NativeBottomTabView({
const options = descriptors[route.key]?.options;
return options?.tabBarItemHidden === true;
}}
getFreezeOnBlur={({ route }) =>
descriptors[route.key]?.options.freezeOnBlur
}
getTestID={({ route }) =>
descriptors[route.key]?.options.tabBarButtonTestID
}
Expand Down
5 changes: 0 additions & 5 deletions yarn.lock
Original file line number Diff line number Diff line change
Expand Up @@ -15614,17 +15614,12 @@ __metadata:
react: 18.3.1
react-native: 0.75.4
react-native-builder-bob: ^0.32.1
react-native-screens: 4.3.0
sf-symbols-typescript: ^2.0.0
typescript: ^5.2.2
use-latest-callback: ^0.2.1
peerDependencies:
react: "*"
react-native: "*"
react-native-screens: ">=3.29.0"
peerDependenciesMeta:
react-native-screens:
optional: true
languageName: unknown
linkType: soft

Expand Down

0 comments on commit 7855b14

Please sign in to comment.