Replies: 23 comments 70 replies
This comment has been hidden.
This comment has been hidden.
This comment has been hidden.
This comment has been hidden.
This comment has been hidden.
This comment has been hidden.
-
Will this method work for inputs placed at the bottom or in the middle of the bottomSheet? (Android works good already with softInputMode=adjustResize but on iOS the inputs can go behind the keyboard.) EDIT: @gorhom I didn't read carefully your status update. Seems like you already planned the implementation I was talking about with |
Beta Was this translation helpful? Give feedback.
This comment has been hidden.
This comment has been hidden.
This comment has been hidden.
This comment has been hidden.
-
Any chance this could be patched into the 3.0.0 release so it'd be compatible with Expo? I'll upgrade to Expo 41 and Reanimated 2 as soon as I can, but this is unusably bugged otherwise (I'm using it for a bottom sheet of comments and a comment input) |
Beta Was this translation helpful? Give feedback.
This comment has been hidden.
This comment has been hidden.
This comment has been hidden.
This comment has been hidden.
This comment has been hidden.
This comment has been hidden.
-
thanks everyone for participating in this discussion and helping me out shape up this feature <3 |
Beta Was this translation helpful? Give feedback.
-
Now that Expo SDK 41 Beta is out, I finally get to play with this. Great job as always. I love that you have When your snap points are: [60, '30%'] When your snap points are: [60, '30%', '100%'] bskeyboard.mov |
Beta Was this translation helpful? Give feedback.
-
@gorhom I can't seem to get this working. I'm probably missing something, but I'm not sure what. Here is a simple example: import React, {Node, useRef} from 'react';
import {
StyleSheet,
Text,
TextInput,
TouchableOpacity,
View,
} from 'react-native';
import {BottomSheetModal, BottomSheetModalProvider} from '@gorhom/bottom-sheet';
const App: () => Node = () => {
const sheetRef = useRef(null);
const handleShowPress = () => {
sheetRef.current && sheetRef.current.present();
};
return (
<BottomSheetModalProvider>
<View style={styles.container}>
<TouchableOpacity style={styles.button} onPress={handleShowPress}>
<Text>Show sheet</Text>
</TouchableOpacity>
<BottomSheetModal
ref={sheetRef}
snapPoints={['25%']}
keyboardBehavior="interactive"
keyboardBlurBehavior="restore">
<TextInput placeholder="Type something here..." />
</BottomSheetModal>
</View>
</BottomSheetModalProvider>
);
}; I've also set up a repo with this example: https://github.com/evrimfeyyaz/bottom-sheet-keyboard-test What am I missing? |
Beta Was this translation helpful? Give feedback.
-
Possible BUG: White space at bottom of BottomSheet when using snap point of 100% This took a white to hunt down 🤯
|
Beta Was this translation helpful? Give feedback.
-
Another issue with Android. Same code as above, but change the snap point to [60, 250, "90%"]. See video below. First, it opens to a strange spot - below the first snap point. somehow managing to click the BottomSheetTextInput expands and then clicking the search button on the keyboard makes the bottom sheet disappear (same as above). Next - we refresh the app. The bottom sheet opens to the strange point. We drag the bottom sheet up and back down - it then snaps to the correct first snap point (60). Then two things happen:
androidKeyboard2.mp4 |
Beta Was this translation helpful? Give feedback.
-
Hi @gorhom 👋 how do u update to this version? |
Beta Was this translation helpful? Give feedback.
-
Possible BUG: Presenting modal after clicking inside The expand button is:
In the video below:
So far so good Now the bug:
RPReplay_Final1618335364.movCode:
|
Beta Was this translation helpful? Give feedback.
-
on a screen with multiple modals, we currently need to write methods for each modal if we want to dismiss an optionally open Keyboard. Like so:
It's not a big deal and it's perfectly valid. However, I can't think of a scenario where one would want to keep the Keyboard open when switching between modals. Can you @gorhom? |
Beta Was this translation helpful? Give feedback.
-
@gorhom @kickbk Also, how am I able to use a Screen.Recording.2021-05-07.at.9.20.50.PM.mov |
Beta Was this translation helpful? Give feedback.
-
Is there any way we can get around |
Beta Was this translation helpful? Give feedback.
-
It'd be nice if I could use my own TextInput component rather than the library's one. I use react-native-paper's TextInput but it seems it cannot be used for this feature. |
Beta Was this translation helpful? Give feedback.
-
whats the easiest solution to handle keyboard ? as its should scroll up |
Beta Was this translation helpful? Give feedback.
-
which solution is best to keyboard handling? |
Beta Was this translation helpful? Give feedback.
-
Feature Request
Why it is needed
Keyboard handling is a bit trickier with bottom sheet internal animated variables, although #97 could resolve it for some cases, however it needs to be handled internally to not conflict with BottomSheet interactions.
Possible implementation
BottomSheet
was inspired by the implementation of Apple Maps, so I would keep that as the main goal to achieve with this feature.Untitled.mp4
Code sample
We could provide a pre-integrated overridable
TextInput
that will internally interacts withBottomSheet
, so user won't need to handle anything else.Beta Was this translation helpful? Give feedback.
All reactions