Skip to content

Commit

Permalink
fix:Multiple image attachments in one message do not work correctly R…
Browse files Browse the repository at this point in the history
  • Loading branch information
Vansh5632 committed Dec 13, 2024
1 parent f12e9eb commit 7bf0bfb
Show file tree
Hide file tree
Showing 51 changed files with 320 additions and 327 deletions.
4 changes: 2 additions & 2 deletions .detoxrc.js
Original file line number Diff line number Diff line change
Expand Up @@ -19,13 +19,13 @@ module.exports = {
type: 'ios.app',
binaryPath: 'ios/build/Build/Products/Debug-iphonesimulator/Rocket.Chat Experimental.app',
build:
'xcodebuild -workspace ios/RocketChatRN.xcworkspace -scheme RocketChatRN -configuration Debug -destination \'generic/platform=iphonesimulator\' -derivedDataPath ios/build'
"xcodebuild -workspace ios/RocketChatRN.xcworkspace -scheme RocketChatRN -configuration Debug -destination 'generic/platform=iphonesimulator' -derivedDataPath ios/build"
},
'ios.release': {
type: 'ios.app',
binaryPath: 'ios/build/Build/Products/Release-iphonesimulator/Rocket.Chat Experimental.app',
build:
'xcodebuild -workspace ios/RocketChatRN.xcworkspace -scheme RocketChatRN -configuration Release -destination \'generic/platform=iphonesimulator\' -derivedDataPath ios/build'
"xcodebuild -workspace ios/RocketChatRN.xcworkspace -scheme RocketChatRN -configuration Release -destination 'generic/platform=iphonesimulator' -derivedDataPath ios/build"
},
'android.debug': {
type: 'android.apk',
Expand Down
2 changes: 1 addition & 1 deletion android/app/build.gradle
Original file line number Diff line number Diff line change
Expand Up @@ -93,7 +93,7 @@ android {
minSdkVersion rootProject.ext.minSdkVersion
targetSdkVersion rootProject.ext.targetSdkVersion
versionCode VERSIONCODE as Integer
versionName "4.56.0"
versionName "4.55.0"
vectorDrawables.useSupportLibrary = true
if (!isFoss) {
manifestPlaceholders = [BugsnagAPIKey: BugsnagAPIKey as String]
Expand Down
3 changes: 1 addition & 2 deletions app/containers/ActionSheet/BottomSheetContent.tsx
Original file line number Diff line number Diff line change
Expand Up @@ -28,8 +28,7 @@ const BottomSheetContent = React.memo(({ options, hasCancel, hide, children, onL
<Touch
onPress={hide}
style={[styles.button, { backgroundColor: colors.surfaceHover }]}
accessibilityLabel={I18n.t('Cancel')}
>
accessibilityLabel={I18n.t('Cancel')}>
<Text style={[styles.text, { color: colors.fontDefault }]}>{I18n.t('Cancel')}</Text>
</Touch>
) : null;
Expand Down
3 changes: 1 addition & 2 deletions app/containers/Chip/index.tsx
Original file line number Diff line number Diff line change
Expand Up @@ -57,8 +57,7 @@ const Chip = ({ avatar, text, onPress, testID, style }: IChip) => {
onPress={() => onPress?.()}
android_ripple={{
color: colors.surfaceNeutral
}}
>
}}>
<View style={styles.container}>
{avatar ? <Avatar text={avatar} size={28} style={styles.avatar} /> : null}
<View style={styles.textContainer}>
Expand Down
6 changes: 2 additions & 4 deletions app/containers/CollapsibleText/index.tsx
Original file line number Diff line number Diff line change
Expand Up @@ -67,15 +67,13 @@ const CollapsibleText = ({ msg, style = [], linesToTruncate = 1 }: ICollapsibleT
} else {
setShowTruncated(false);
}
}}
>
}}>
{m}
{truncatedText ? (
<Text
testID='collapsible-text-show-less'
onPress={() => setShowTruncated(true)}
style={[styles.textInfo, { color: colors.fontHint }]}
>
style={[styles.textInfo, { color: colors.fontHint }]}>
{` ${I18n.t('Show_less')}`}
</Text>
) : null}
Expand Down
6 changes: 2 additions & 4 deletions app/containers/EmojiPicker/Footer.tsx
Original file line number Diff line number Diff line change
Expand Up @@ -18,8 +18,7 @@ const Footer = ({ onSearchPressed, onBackspacePressed }: IFooterProps): React.Re
styles.footerButtonsContainer,
{ backgroundColor: isIOS && pressed ? colors.buttonBackgroundSecondaryPress : 'transparent' }
]}
testID='emoji-picker-search'
>
testID='emoji-picker-search'>
<CustomIcon size={24} name='search' />
</Pressable>

Expand All @@ -30,8 +29,7 @@ const Footer = ({ onSearchPressed, onBackspacePressed }: IFooterProps): React.Re
styles.footerButtonsContainer,
{ backgroundColor: isIOS && pressed ? colors.buttonBackgroundSecondaryPress : 'transparent' }
]}
testID='emoji-picker-backspace'
>
testID='emoji-picker-backspace'>
<CustomIcon size={24} name='backspace' />
</Pressable>
</View>
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -49,8 +49,7 @@ const IncomingCallHeader = React.memo(
{
marginTop: insets.top
}
]}
>
]}>
<CallHeader
title={i18n.t('Incoming_call_from')}
cam={cam}
Expand All @@ -69,8 +68,7 @@ const IncomingCallHeader = React.memo(
setAudio(!audio);
hideNotification();
}}
style={styles.closeButton}
>
style={styles.closeButton}>
<CustomIcon name='close' size={20} />
</Touchable>
<Touchable
Expand All @@ -80,8 +78,7 @@ const IncomingCallHeader = React.memo(
hideNotification();
dispatch(cancelCall({ callId }));
}}
style={styles.cancelButton}
>
style={styles.cancelButton}>
<Text style={styles.buttonText}>{i18n.t('decline')}</Text>
</Touchable>
<Touchable
Expand All @@ -91,8 +88,7 @@ const IncomingCallHeader = React.memo(
hideNotification();
dispatch(acceptCall({ callId }));
}}
style={styles.acceptButton}
>
style={styles.acceptButton}>
<Text style={styles.buttonText}>{i18n.t('accept')}</Text>
</Touchable>
</View>
Expand Down
3 changes: 1 addition & 2 deletions app/containers/LoginServices/ButtonService.tsx
Original file line number Diff line number Diff line change
Expand Up @@ -18,8 +18,7 @@ const ButtonService = ({ name, authType, onPress, backgroundColor, buttonText, i
activeOpacity={0.5}
underlayColor={colors.fontWhite}
accessible
accessibilityLabel={accessibilityLabel}
>
accessibilityLabel={accessibilityLabel}>
<View style={styles.serviceButtonContainer}>
{authType === 'oauth' || authType === 'apple' ? <CustomIcon name={icon} size={24} style={styles.serviceIcon} /> : null}
<Text style={[styles.serviceText, { color: colors.fontTitlesLabels }]}>{buttonText}</Text>
Expand Down
3 changes: 1 addition & 2 deletions app/containers/MessageComposer/components/EmojiSearchbar.tsx
Original file line number Diff line number Diff line change
Expand Up @@ -64,8 +64,7 @@ export const EmojiSearchbar = (): React.ReactElement | null => {
style={({ pressed }: { pressed: boolean }) => [styles.backButton, { opacity: pressed ? 0.7 : 1 }]}
onPress={openEmojiKeyboard}
hitSlop={BUTTON_HIT_SLOP}
testID='openback-emoji-keyboard'
>
testID='openback-emoji-keyboard'>
<CustomIcon name='chevron-left' size={24} />
</Pressable>
<View style={styles.inputContainer}>
Expand Down
3 changes: 1 addition & 2 deletions app/containers/Passcode/Base/Button.tsx
Original file line number Diff line number Diff line change
Expand Up @@ -25,8 +25,7 @@ const Button = React.memo(({ style, text, disabled, onPress, icon }: IPasscodeBu
underlayColor={colors.buttonBackgroundSecondaryDefault}
rippleColor={colors.buttonBackgroundSecondaryPress}
enabled={!disabled}
onPress={press}
>
onPress={press}>
{icon ? (
<CustomIcon name={icon} size={36} />
) : (
Expand Down
12 changes: 4 additions & 8 deletions app/containers/RoomItem/Actions.tsx
Original file line number Diff line number Diff line change
Expand Up @@ -39,8 +39,7 @@ export const LeftActions = React.memo(({ transX, isRead, width, onToggleReadPres
{ width: width * 2, backgroundColor: colors.badgeBackgroundLevel2, right: '100%' },
viewHeight,
animatedStyles
]}
>
]}>
<View style={[styles.actionLeftButtonContainer, viewHeight]}>
<RectButton style={styles.actionButton} onPress={onToggleReadPress}>
<CustomIcon
Expand Down Expand Up @@ -123,8 +122,7 @@ export const RightActions = React.memo(({ transX, favorite, width, toggleFav, on
},
viewHeight,
animatedFavStyles
]}
>
]}>
<RectButton style={[styles.actionButton, { backgroundColor: colors.statusFontWarning }]} onPress={toggleFav}>
<CustomIcon
size={isCondensed ? CONDENSED_ICON_SIZE : EXPANDED_ICON_SIZE}
Expand All @@ -143,12 +141,10 @@ export const RightActions = React.memo(({ transX, favorite, width, toggleFav, on
},
isCondensed && { height: ROW_HEIGHT_CONDENSED },
animatedHideStyles
]}
>
]}>
<RectButton
style={[styles.actionButton, { backgroundColor: colors.buttonBackgroundSecondaryPress }]}
onPress={onHidePress}
>
onPress={onHidePress}>
<CustomIcon
size={isCondensed ? CONDENSED_ICON_SIZE : EXPANDED_ICON_SIZE}
name='unread-on-top-disabled'
Expand Down
3 changes: 1 addition & 2 deletions app/containers/RoomItem/Touchable.tsx
Original file line number Diff line number Diff line change
Expand Up @@ -224,8 +224,7 @@ const Touchable = ({
testID={testID}
style={{
backgroundColor: isFocused ? colors.surfaceTint : colors.surfaceRoom
}}
>
}}>
{children}
</Touch>
</Animated.View>
Expand Down
2 changes: 1 addition & 1 deletion app/containers/SearchBox/index.tsx
Original file line number Diff line number Diff line change
Expand Up @@ -23,7 +23,7 @@ const SearchBox = ({ onChangeText, onSubmitEditing, testID }: TextInputProps): J
}, []);

return (
<View testID='searchbox' style={{ backgroundColor: colors.surfaceRoom }} >
<View testID='searchbox' style={{ backgroundColor: colors.surfaceRoom }}>
<FormTextInput
autoCapitalize='none'
autoCorrect={false}
Expand Down
3 changes: 1 addition & 2 deletions app/containers/ThreadDetails.tsx
Original file line number Diff line number Diff line change
Expand Up @@ -72,8 +72,7 @@ const ThreadDetails = ({ item, user, badgeColor, toggleFollowThread, style }: IT
<Text
testID={`thread-count-${count}`}
style={[styles.detailText, { color: themes[theme].fontSecondaryInfo }]}
numberOfLines={1}
>
numberOfLines={1}>
{count}
</Text>
</View>
Expand Down
2 changes: 1 addition & 1 deletion app/containers/Toast.tsx
Original file line number Diff line number Diff line change
Expand Up @@ -46,7 +46,7 @@ const Toast = (): React.ReactElement => {
ref={getToastRef}
position='center'
style={[styles.toast, { backgroundColor: colors.surfaceDark }]}
textStyle={[styles.text, { color: theme === "light" ? colors.fontWhite : colors.fontPureBlack }]}
textStyle={[styles.text, { color: theme === 'light' ? colors.fontWhite : colors.fontPureBlack }]}
opacity={0.9}
/>
);
Expand Down
9 changes: 5 additions & 4 deletions app/containers/UIKit/DatePicker.tsx
Original file line number Diff line number Diff line change
Expand Up @@ -65,11 +65,12 @@ export const DatePicker = ({ element, language, action, context, loading, value,
<Touchable
onPress={() => onShow(!show)}
style={{ backgroundColor: themes[theme].surfaceRoom }}
background={Touchable.Ripple(themes[theme].surfaceNeutral)}
>
background={Touchable.Ripple(themes[theme].surfaceNeutral)}>
<View
style={[styles.input, { borderColor: error ? themes[theme].buttonBackgroundDangerDefault : themes[theme].strokeLight }]}
>
style={[
styles.input,
{ borderColor: error ? themes[theme].buttonBackgroundDangerDefault : themes[theme].strokeLight }
]}>
<Text style={[styles.inputText, { color: error ? themes[theme].fontDanger : themes[theme].fontTitlesLabels }]}>
{currentDate.toLocaleDateString(language)}
</Text>
Expand Down
3 changes: 1 addition & 2 deletions app/containers/UIKit/MultiSelect/Chips.tsx
Original file line number Diff line number Diff line change
Expand Up @@ -31,8 +31,7 @@ const Chip = ({ item, onSelect, style }: IChip) => {
onPress={() => onSelect(item)}
style={[styles.chip, { backgroundColor: colors.surfaceHover }, style]}
background={Touchable.Ripple(colors.surfaceNeutral)}
testID={`multi-select-chip-${item.value}`}
>
testID={`multi-select-chip-${item.value}`}>
<>
{item.imageUrl ? <FastImage style={styles.chipImage} source={{ uri: item.imageUrl }} /> : null}
<Text numberOfLines={1} style={[styles.chipText, { color: colors.fontTitlesLabels }]}>
Expand Down
3 changes: 1 addition & 2 deletions app/containers/UIKit/MultiSelect/Input.tsx
Original file line number Diff line number Diff line change
Expand Up @@ -24,8 +24,7 @@ const Input = ({ children, onPress, loading, inputStyle, placeholder, disabled,
onPress={onPress}
style={[{ backgroundColor: colors.surfaceRoom }, styles.inputBorder, inputStyle]}
background={Touchable.Ripple(colors.surfaceNeutral)}
disabled={disabled}
>
disabled={disabled}>
<View style={[styles.input, styles.inputBorder, { borderColor: colors.strokeLight }, innerInputStyle]}>
{placeholder ? <Text style={[styles.pickerText, { color: colors.fontSecondaryInfo }]}>{placeholder}</Text> : children}
{loading ? (
Expand Down
9 changes: 3 additions & 6 deletions app/containers/UIKit/Overflow.tsx
Original file line number Diff line number Diff line change
Expand Up @@ -30,8 +30,7 @@ const Option = ({ option: { text, value }, onOptionPress, parser, theme }: IOpti
<Touchable
onPress={() => onOptionPress({ value })}
background={Touchable.Ripple(themes[theme].surfaceNeutral)}
style={styles.option}
>
style={styles.option}>
<Text>{parser.text(text)}</Text>
</Touchable>
);
Expand Down Expand Up @@ -65,8 +64,7 @@ export const Overflow = ({ element, loading, action, parser }: IOverflow) => {
background={Touchable.Ripple(themes[theme].surfaceNeutral)}
onPress={() => onShow(!show)}
hitSlop={BUTTON_HIT_SLOP}
style={styles.menu}
>
style={styles.menu}>
{!loading ? (
<CustomIcon size={18} name='kebab' color={themes[theme].fontDefault} />
) : (
Expand All @@ -78,8 +76,7 @@ export const Overflow = ({ element, loading, action, parser }: IOverflow) => {
// fromView exists in Popover Component
/* @ts-ignore*/
fromView={touchable[blockId]}
onRequestClose={() => onShow(false)}
>
onRequestClose={() => onShow(false)}>
<Options options={options} onOptionPress={onOptionPress} parser={parser} theme={theme} />
</Popover>
</>
Expand Down
3 changes: 1 addition & 2 deletions app/containers/UIKit/UiKitMessage.stories.tsx
Original file line number Diff line number Diff line change
Expand Up @@ -41,8 +41,7 @@ export default {
onDiscussionPress: () => {},
onReactionLongPress: () => {},
threadBadgeColor: themes.light.fontInfo
}}
>
}}>
<Story />
</MessageContext.Provider>
</ScrollView>
Expand Down
3 changes: 1 addition & 2 deletions app/containers/UIKit/UiKitModal.stories.tsx
Original file line number Diff line number Diff line change
Expand Up @@ -43,8 +43,7 @@ export default {
onDiscussionPress: () => {},
onReactionLongPress: () => {},
threadBadgeColor: themes.light.fontInfo
}}
>
}}>
<Story />
</MessageContext.Provider>
</ScrollView>
Expand Down
3 changes: 1 addition & 2 deletions app/containers/markdown/Table.tsx
Original file line number Diff line number Diff line change
Expand Up @@ -55,8 +55,7 @@ const Table = React.memo(({ children, numColumns, theme }: ITable) => {
style={[
styles.containerTable,
{ maxWidth: getTableWidth(), maxHeight: MAX_HEIGHT, borderColor: themes[theme].strokeLight }
]}
>
]}>
{renderRows(false)}
</ScrollView>
<Text style={[styles.textInfo, { color: themes[theme].fontSecondaryInfo }]}>{I18n.t('Full_table')}</Text>
Expand Down
3 changes: 1 addition & 2 deletions app/containers/message/CallButton.tsx
Original file line number Diff line number Diff line change
Expand Up @@ -18,8 +18,7 @@ const CallButton = React.memo(({ handleEnterCall }: IMessageCallButton) => {
onPress={handleEnterCall}
background={Touchable.Ripple(themes[theme].surfaceNeutral)}
style={[styles.button, { backgroundColor: themes[theme].badgeBackgroundLevel2 }]}
hitSlop={BUTTON_HIT_SLOP}
>
hitSlop={BUTTON_HIT_SLOP}>
<>
<CustomIcon name='camera' size={16} style={styles.buttonIcon} color={themes[theme].fontWhite} />
<Text style={[styles.buttonText, { color: themes[theme].fontWhite }]}>{I18n.t('Click_to_join')}</Text>
Expand Down
Loading

0 comments on commit 7bf0bfb

Please sign in to comment.