-
Notifications
You must be signed in to change notification settings - Fork 1.1k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
feat: New message composer #5205
Conversation
# Conflicts: # __tests__/containers/RoomItem/__snapshots__/RoomItem.stories.storyshot # app/lib/constants/colors.ts # e2e/tests/room/02-room.spec.ts # e2e/tests/room/05-threads.spec.ts
Co-authored-by: reinaldonetof <reinaldonetof@hotmail.com>
…h... useCallback isn't really working?
# Conflicts: # .circleci/config.yml # __tests__/containers/Button/__snapshots__/Button.stories.storyshot # __tests__/containers/LoginServices/__snapshots__/LoginServices.stories.storyshot # __tests__/containers/RoomHeader/__snapshots__/RoomHeader.stories.storyshot # __tests__/containers/RoomItem/__snapshots__/RoomItem.stories.storyshot # __tests__/containers/message/__snapshots__/Message.stories.storyshot # __tests__/views/CannedResponsesListView/__snapshots__/CannedResponseItem.stories.storyshot # __tests__/views/ThreadMessagesView/__snapshots__/Item.stories.storyshot # app/containers/ActionSheet/ActionSheet.tsx # app/containers/Status/index.tsx # app/containers/UIKit/UiKitMessage.stories.tsx # app/containers/UIKit/index.tsx # app/containers/message/index.tsx # app/i18n/locales/en.json # app/i18n/locales/pt-BR.json # app/lib/methods/handleMediaDownload.ts # app/lib/services/restApi.ts # app/views/RoomView/List/index.tsx # app/views/RoomView/index.tsx # e2e/tests/room/05-threads.spec.ts # e2e/tests/room/11-autoTranslate.spec.ts # ios/RocketChatRN.xcodeproj/project.pbxproj # jest.setup.js # patches/react-native-ui-lib+7.2.1.patch # yarn.lock
# Conflicts: # app/views/RoomView/List/NavBottomFAB.tsx # app/views/RoomView/List/index.tsx # app/views/RoomView/index.tsx # ios/RocketChatRN.xcodeproj/project.pbxproj
9c0e6c5
to
2af66ec
Compare
d24014c
to
36c2c15
Compare
e25cd85
to
b508213
Compare
e37a983
to
16d00c9
Compare
….ReactNative into feat.new-composer
return ( | ||
<> | ||
<BaseButton | ||
onPress={() => editCancel?.()} |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
onPress={() => editCancel?.()} | |
onPress={editCancel} |
return null; | ||
} | ||
|
||
// TODO: Use RNGH |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
👀
backgroundColor: colors.buttonBackgroundPrimaryDefault | ||
} | ||
]} | ||
onPress={() => onPress()} |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
onPress={() => onPress()} | |
onPress={onPress} |
bug when inserting emojis within multiple characters: |
This is not a bug, however this "jump" that occurs when clicking on the input could be avoided if you disabled the input and only focused when clicking on it. |
b79f8df
to
be981d5
Compare
# Conflicts: # __tests__/containers/message/__snapshots__/Message.stories.storyshot # app/lib/constants/colors.ts
# Conflicts: # app/containers/MessageBox/index.tsx
Proposed changes
Issue(s)
Closes #5301
How to test or reproduce
Screenshots
Types of changes
Checklist
Further comments