chore: Merge 4.51.0 into master (#5860)

RocketChat · Sep 9, 2024 · 92ebf5e · 92ebf5e
1 parent deb8ea4
commit 92ebf5e
Show file tree

Hide file tree

Showing 158 changed files with 1,978 additions and 2,783 deletions.
diff --git a/android/app/build.gradle b/android/app/build.gradle
@@ -93,7 +93,7 @@ android {
         minSdkVersion rootProject.ext.minSdkVersion
         targetSdkVersion rootProject.ext.targetSdkVersion
         versionCode VERSIONCODE as Integer
-        versionName "4.50.1"
+        versionName "4.51.0"
         vectorDrawables.useSupportLibrary = true
         if (!isFoss) {
             manifestPlaceholders = [BugsnagAPIKey: BugsnagAPIKey as String]

diff --git a/android/app/src/main/java/chat/rocket/reactnative/MainActivity.kt b/android/app/src/main/java/chat/rocket/reactnative/MainActivity.kt
@@ -33,12 +33,4 @@ class MainActivity : ReactActivity() {
   override fun invokeDefaultOnBackPressed() {
     moveTaskToBack(true)
   }
-
-	// from react-native-orientation
-	override fun onConfigurationChanged(newConfig: Configuration) {
-		super.onConfigurationChanged(newConfig)
-		val intent = Intent("onConfigurationChanged")
-		intent.putExtra("newConfig", newConfig)
-		sendBroadcast(intent)
-	}
 }
diff --git a/app/actions/actionsTypes.ts b/app/actions/actionsTypes.ts
@@ -16,8 +16,6 @@ export const ROOMS = createRequestTypes('ROOMS', [
 	...defaultTypes,
 	'REFRESH',
 	'SET_SEARCH',
-	'CLOSE_SERVER_DROPDOWN',
-	'TOGGLE_SERVER_DROPDOWN',
 	'OPEN_SEARCH_HEADER',
 	'CLOSE_SEARCH_HEADER'
 ]);

diff --git a/app/actions/rooms.ts b/app/actions/rooms.ts
@@ -53,18 +53,6 @@ export function setSearch(searchText: string): ISetSearch {
 	};
 }
 
-export function closeServerDropdown(): Action {
-	return {
-		type: ROOMS.CLOSE_SERVER_DROPDOWN
-	};
-}
-
-export function toggleServerDropdown(): Action {
-	return {
-		type: ROOMS.TOGGLE_SERVER_DROPDOWN
-	};
-}
-
 export function openSearchHeader(): Action {
 	return {
 		type: ROOMS.OPEN_SEARCH_HEADER

diff --git a/app/containers/ActionSheet/Item.tsx b/app/containers/ActionSheet/Item.tsx
@@ -38,7 +38,7 @@ export const Item = React.memo(({ item, hide }: IActionSheetItem) => {
 	return (
 		<View accessible accessibilityLabel={item.title}>
 			<Touch onPress={onPress} style={[styles.item, { backgroundColor: colors.surfaceLight }]} testID={item.testID}>
-				{item.icon ? <CustomIcon name={item.icon} size={20} color={color} /> : null}
+				{item.icon ? <CustomIcon name={item.icon} size={24} color={color} /> : null}
 				<View style={styles.titleContainer}>
 					<Text numberOfLines={1} style={[styles.title, { color, marginLeft: item.icon ? 16 : 0 }]}>
 						{item.title}

diff --git a/app/containers/ActionSheet/styles.ts b/app/containers/ActionSheet/styles.ts
@@ -24,7 +24,7 @@ export default StyleSheet.create({
 	},
 	title: {
 		fontSize: 16,
-		...sharedStyles.textRegular
+		...sharedStyles.textMedium
 	},
 	handle: {
 		justifyContent: 'center',

diff --git a/app/containers/Button/index.tsx b/app/containers/Button/index.tsx
@@ -67,7 +67,13 @@ const Button: React.FC<IButtonProps> = ({
 	const textStyle = [styles.text, { color: isDisabled ? colors.fontDisabled : resolvedTextColor, fontSize }, styleText];
 
 	return (
-		<Touchable onPress={onPress} disabled={isDisabled} style={containerStyle} accessibilityLabel={title} {...otherProps}>
+		<Touchable
+			onPress={onPress}
+			disabled={isDisabled}
+			style={containerStyle}
+			accessibilityLabel={title}
+			accessibilityRole='button'
+			{...otherProps}>
 			{loading ? <ActivityIndicator color={resolvedTextColor} /> : <Text style={textStyle}>{title}</Text>}
 		</Touchable>
 	);

diff --git a/app/containers/DirectoryItem/index.tsx b/app/containers/DirectoryItem/index.tsx
@@ -50,27 +50,29 @@ const DirectoryItem = ({
 }: IDirectoryItem): React.ReactElement => {
 	const { theme } = useTheme();
 	return (
-		<Touch onPress={onPress} style={{ backgroundColor: themes[theme].surfaceRoom }} testID={testID}>
-			<View style={[styles.directoryItemContainer, styles.directoryItemButton, style]}>
-				<Avatar text={avatar} size={30} type={type} rid={rid} style={styles.directoryItemAvatar} />
-				<View style={styles.directoryItemTextContainer}>
-					<View style={styles.directoryItemTextTitle}>
-						{type !== 'd' ? <RoomTypeIcon type={type} teamMain={teamMain} /> : null}
-						<Text style={[styles.directoryItemName, { color: themes[theme].fontTitlesLabels }]} numberOfLines={1}>
-							{title}
-						</Text>
+		<View accessible accessibilityLabel={`${title} ${rightLabel}`}>
+			<Touch onPress={onPress} style={{ backgroundColor: themes[theme].surfaceRoom }} testID={testID}>
+				<View style={[styles.directoryItemContainer, styles.directoryItemButton, style]}>
+					<Avatar text={avatar} size={30} type={type} rid={rid} style={styles.directoryItemAvatar} />
+					<View style={styles.directoryItemTextContainer}>
+						<View style={styles.directoryItemTextTitle}>
+							{type !== 'd' ? <RoomTypeIcon type={type} teamMain={teamMain} /> : null}
+							<Text style={[styles.directoryItemName, { color: themes[theme].fontTitlesLabels }]} numberOfLines={1}>
+								{title}
+							</Text>
+						</View>
+						{description ? (
+							<MarkdownPreview
+								msg={description}
+								style={[styles.directoryItemUsername, { color: themes[theme].fontSecondaryInfo }]}
+								numberOfLines={1}
+							/>
+						) : null}
 					</View>
-					{description ? (
-						<MarkdownPreview
-							msg={description}
-							style={[styles.directoryItemUsername, { color: themes[theme].fontSecondaryInfo }]}
-							numberOfLines={1}
-						/>
-					) : null}
+					<DirectoryItemLabel text={rightLabel} theme={theme} />
 				</View>
-				<DirectoryItemLabel text={rightLabel} theme={theme} />
-			</View>
-		</Touch>
+			</Touch>
+		</View>
 	);
 };
 

diff --git a/app/containers/HeaderButton/HeaderButtonItem.tsx b/app/containers/HeaderButton/HeaderButtonItem.tsx
@@ -14,6 +14,7 @@ export interface IHeaderButtonItem {
 	badge?(): void;
 	color?: string;
 	disabled?: boolean;
+	accessibilityLabel?: string;
 }
 
 export const BUTTON_HIT_SLOP = {
@@ -40,7 +41,17 @@ const styles = StyleSheet.create({
 	}
 });
 
-const Item = ({ title, iconName, onPress, testID, badge, color, disabled, ...props }: IHeaderButtonItem): React.ReactElement => {
+const Item = ({
+	title,
+	iconName,
+	onPress,
+	testID,
+	badge,
+	color,
+	disabled,
+	accessibilityLabel,
+	...props
+}: IHeaderButtonItem): React.ReactElement => {
 	const { colors } = useTheme();
 	return (
 		<PlatformPressable
@@ -53,7 +64,8 @@ const Item = ({ title, iconName, onPress, testID, badge, color, disabled, ...pro
 				{
 					opacity: disabled ? 0.5 : 1
 				}
-			]}>
+			]}
+			accessibilityLabel={accessibilityLabel}>
 			<>
 				{iconName ? (
 					<CustomIcon name={iconName} size={24} color={color} {...props} />

diff --git a/app/containers/List/ListHeader.tsx b/app/containers/List/ListHeader.tsx
@@ -21,14 +21,15 @@ const styles = StyleSheet.create({
 interface IListHeader {
 	title: string;
 	translateTitle?: boolean;
+	numberOfLines?: number;
 }
 
-const ListHeader = React.memo(({ title, translateTitle = true }: IListHeader) => {
+const ListHeader = React.memo(({ title, translateTitle = true, numberOfLines = 1 }: IListHeader) => {
 	const { theme } = useTheme();
 
 	return (
 		<View style={styles.container}>
-			<Text style={[styles.title, { color: themes[theme].fontHint }]} numberOfLines={1}>
+			<Text style={[styles.title, { color: themes[theme].fontHint }]} numberOfLines={numberOfLines}>
 				{translateTitle ? I18n.t(title) : title}
 			</Text>
 		</View>

diff --git a/app/containers/List/ListIcon.tsx b/app/containers/List/ListIcon.tsx
@@ -3,6 +3,7 @@ import { StyleProp, StyleSheet, View, ViewStyle } from 'react-native';
 
 import { CustomIcon, TIconsName } from '../CustomIcon';
 import { ICON_SIZE } from './constants';
+import { useTheme } from '../../theme';
 
 interface IListIcon {
 	name: TIconsName;
@@ -19,11 +20,14 @@ const styles = StyleSheet.create({
 	}
 });
 
-const ListIcon = ({ name, color, style, testID, size }: IListIcon): React.ReactElement => (
-	<View style={[styles.icon, style]}>
-		<CustomIcon name={name} color={color} size={size ?? ICON_SIZE} testID={testID} />
-	</View>
-);
+const ListIcon = ({ name, color, style, testID, size }: IListIcon): React.ReactElement => {
+	const { colors } = useTheme();
+	return (
+		<View style={[styles.icon, style]}>
+			<CustomIcon name={name} color={color || colors.fontDefault} size={size ?? ICON_SIZE} testID={testID} />
+		</View>
+	);
+};
 
 ListIcon.displayName = 'List.Icon';
 

diff --git a/app/containers/List/ListItem.tsx b/app/containers/List/ListItem.tsx
@@ -1,4 +1,4 @@
-import React from 'react';
+import React, { useMemo } from 'react';
 import { I18nManager, StyleProp, StyleSheet, Text, TextStyle, View } from 'react-native';
 
 import Touch from '../Touch';
@@ -42,7 +42,7 @@ const styles = StyleSheet.create({
 	title: {
 		flexShrink: 1,
 		fontSize: 16,
-		...sharedStyles.textRegular
+		...sharedStyles.textMedium
 	},
 	subtitle: {
 		fontSize: 14,
@@ -68,6 +68,8 @@ interface IListItemContent {
 	alert?: boolean;
 	heightContainer?: number;
 	styleTitle?: StyleProp<TextStyle>;
+	additionalAcessibilityLabel?: string | boolean;
+	additionalAcessibilityLabelCheck?: boolean;
 }
 
 const Content = React.memo(
@@ -85,18 +87,44 @@ const Content = React.memo(
 		showActionIndicator = false,
 		theme,
 		heightContainer,
-		styleTitle
+		styleTitle,
+		additionalAcessibilityLabel,
+		additionalAcessibilityLabelCheck
 	}: IListItemContent) => {
 		const { fontScale } = useDimensions();
 
+		const handleAcessibilityLabel = useMemo(() => {
+			let label = '';
+			if (title) {
+				label = translateTitle ? I18n.t(title) : title;
+			}
+			if (subtitle) {
+				label = translateSubtitle ? `${label} ${I18n.t(subtitle)}` : `${label} ${subtitle}`;
+			}
+			if (typeof additionalAcessibilityLabel === 'string') {
+				label = `${label} ${additionalAcessibilityLabel}`;
+			}
+			if (typeof additionalAcessibilityLabel === 'boolean') {
+				if (additionalAcessibilityLabelCheck) {
+					label = `${label} ${additionalAcessibilityLabel ? I18n.t('Checked') : I18n.t('Unchecked')}`;
+				} else {
+					label = `${label} ${additionalAcessibilityLabel ? I18n.t('Enabled') : I18n.t('Disabled')}`;
+				}
+			}
+			return label;
+		}, [title, subtitle, translateTitle, translateSubtitle, additionalAcessibilityLabel, additionalAcessibilityLabelCheck]);
+
 		return (
 			<View
 				style={[styles.container, disabled && styles.disabled, { height: (heightContainer || BASE_HEIGHT) * fontScale }]}
-				testID={testID}>
+				testID={testID}
+				accessible
+				accessibilityLabel={handleAcessibilityLabel}
+				accessibilityRole='button'>
 				{left ? <View style={styles.leftContainer}>{left()}</View> : null}
 				<View style={styles.textContainer}>
 					<View style={styles.textAlertContainer}>
-						<Text style={[styles.title, styleTitle, { color: color || themes[theme].fontTitlesLabels }]} numberOfLines={1}>
+						<Text style={[styles.title, styleTitle, { color: color || themes[theme].fontDefault }]} numberOfLines={1}>
 							{translateTitle && title ? I18n.t(title) : title}
 						</Text>
 						{alert ? (

diff --git a/app/containers/List/constants.ts b/app/containers/List/constants.ts
@@ -1,3 +1,3 @@
 export const PADDING_HORIZONTAL = 12;
 export const BASE_HEIGHT = 46;
-export const ICON_SIZE = 20;
+export const ICON_SIZE = 24;
diff --git a/app/containers/LoginServices/ButtonService.tsx b/app/containers/LoginServices/ButtonService.tsx
@@ -7,7 +7,7 @@ import { CustomIcon } from '../CustomIcon';
 import { IButtonService } from './interfaces';
 import styles from './styles';
 
-const ButtonService = ({ name, authType, onPress, backgroundColor, buttonText, icon }: IButtonService) => {
+const ButtonService = ({ name, authType, onPress, backgroundColor, buttonText, icon, accessibilityLabel }: IButtonService) => {
 	const { colors } = useTheme();
 
 	return (
@@ -17,6 +17,8 @@ const ButtonService = ({ name, authType, onPress, backgroundColor, buttonText, i
 			style={[styles.serviceButton, { backgroundColor }]}
 			activeOpacity={0.5}
 			underlayColor={colors.fontWhite}
+			accessible
+			accessibilityLabel={accessibilityLabel}
 		>
 			<View style={styles.serviceButtonContainer}>
 				{authType === 'oauth' || authType === 'apple' ? <CustomIcon name={icon} size={24} style={styles.serviceIcon} /> : null}

diff --git a/app/containers/LoginServices/Service.tsx b/app/containers/LoginServices/Service.tsx
@@ -96,6 +96,7 @@ const Service = React.memo(
 				icon={icon}
 				name={service.name}
 				authType={service.authType}
+				accessibilityLabel={`${I18n.t('Continue_with')} ${modifiedName.current}`}
 			/>
 		);
 	}

diff --git a/app/containers/LoginServices/interfaces.ts b/app/containers/LoginServices/interfaces.ts
@@ -66,4 +66,5 @@ export interface IButtonService {
 	backgroundColor: string;
 	buttonText: ReactElement;
 	icon: TIconsName;
+	accessibilityLabel?: string;
 }
diff --git a/app/containers/MessageComposer/__snapshots__/MessageComposer.test.tsx.snap b/app/containers/MessageComposer/__snapshots__/MessageComposer.test.tsx.snap
@@ -626,10 +626,12 @@ exports[`MessageComposer Quote Add quote \`abc\` 1`] = `
                     "fontFamily": "Inter",
                     "fontSize": 16,
                     "fontWeight": "400",
+                    "lineHeight": 22,
                     "textAlign": "left",
                   },
                   {
                     "color": "#2F343D",
+                    "lineHeight": undefined,
                   },
                   {
                     "backgroundColor": "transparent",
@@ -1044,10 +1046,12 @@ exports[`MessageComposer Quote Add quote \`def\` 1`] = `
                     "fontFamily": "Inter",
                     "fontSize": 16,
                     "fontWeight": "400",
+                    "lineHeight": 22,
                     "textAlign": "left",
                   },
                   {
                     "color": "#2F343D",
+                    "lineHeight": undefined,
                   },
                   {
                     "backgroundColor": "transparent",
@@ -1219,10 +1223,12 @@ exports[`MessageComposer Quote Add quote \`def\` 1`] = `
                     "fontFamily": "Inter",
                     "fontSize": 16,
                     "fontWeight": "400",
+                    "lineHeight": 22,
                     "textAlign": "left",
                   },
                   {
                     "color": "#2F343D",
+                    "lineHeight": undefined,
                   },
                   {
                     "backgroundColor": "transparent",
@@ -1637,10 +1643,12 @@ exports[`MessageComposer Quote Remove a quote 1`] = `
                     "fontFamily": "Inter",
                     "fontSize": 16,
                     "fontWeight": "400",
+                    "lineHeight": 22,
                     "textAlign": "left",
                   },
                   {
                     "color": "#2F343D",
+                    "lineHeight": undefined,
                   },
                   {
                     "backgroundColor": "transparent",
@@ -1812,10 +1820,12 @@ exports[`MessageComposer Quote Remove a quote 1`] = `
                     "fontFamily": "Inter",
                     "fontSize": 16,
                     "fontWeight": "400",
+                    "lineHeight": 22,
                     "textAlign": "left",
                   },
                   {
                     "color": "#2F343D",
+                    "lineHeight": undefined,
                   },
                   {
                     "backgroundColor": "transparent",