Skip to content

Commit b517372

Browse files
committed
self review cleanup fixes
1 parent 9785589 commit b517372

File tree

12 files changed

+36
-24
lines changed

12 files changed

+36
-24
lines changed

packages/react-native-sdk/src/components/Call/Lobby/Lobby.tsx

Lines changed: 11 additions & 9 deletions
Original file line numberDiff line numberDiff line change
@@ -84,15 +84,11 @@ export const Lobby = ({
8484
<View style={[styles.container, lobby.container]}>
8585
{connectedUser && (
8686
<>
87-
<Text
88-
style={[
89-
styles.heading,
90-
{ color: colors.textPrimary },
91-
typefaces.heading5,
92-
lobby.heading,
93-
]}
94-
>
95-
{t('Setup your test call')}
87+
<Text style={[styles.heading, typefaces.heading5, lobby.heading]}>
88+
{t('Before joining')}
89+
</Text>
90+
<Text style={[styles.subHeading, lobby.subHeading]}>
91+
{t('Setup your audio and video')}
9692
</Text>
9793
{isVideoEnabledInCall && (
9894
<View
@@ -167,8 +163,14 @@ const useStyles = (landscape = false) => {
167163
() =>
168164
StyleSheet.create({
169165
heading: {
166+
textAlign: 'center',
167+
color: theme.colors.textPrimary,
168+
paddingBottom: theme.variants.spacingSizes.xs,
169+
},
170+
subHeading: {
170171
textAlign: 'center',
171172
paddingBottom: theme.variants.spacingSizes.md,
173+
color: theme.colors.textSecondary,
172174
},
173175
container: {
174176
flex: 1,

packages/react-native-sdk/src/components/Call/Lobby/LobbyFooter.tsx

Lines changed: 10 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -27,17 +27,21 @@ export const LobbyFooter = ({
2727
const { useCallSession } = useCallStateHooks();
2828
const { t } = useI18n();
2929
const session = useCallSession();
30-
const participantsCount = session?.participants.length;
30+
const numberOfParticipants = session?.participants.length;
3131

3232
const participantsText = useMemo(() => {
33-
if (!participantsCount) {
33+
if (!numberOfParticipants) {
3434
return t('Currently there are no other participants in the call.');
3535
}
36-
if (participantsCount === 1) {
37-
return t(`There is ${participantsCount} more person in the call.`);
36+
if (numberOfParticipants === 1) {
37+
return t('There is {{numberOfParticipants}} more person in the call.', {
38+
numberOfParticipants,
39+
});
3840
}
39-
return t(`There are ${participantsCount} more people in the call.`);
40-
}, [participantsCount, t]);
41+
return t('There are {{numberOfParticipants}} more people in the call.', {
42+
numberOfParticipants,
43+
});
44+
}, [numberOfParticipants, t]);
4145

4246
return (
4347
<View style={[styles.mainContainer, lobby.infoContainer]}>

packages/react-native-sdk/src/constants/TestIds.ts

Lines changed: 0 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -15,8 +15,6 @@ export enum ComponentTestIds {
1515
PARTICIPANTS_INFO = 'participants-info',
1616
PARTICIPANT_SCREEN_SHARING = 'participant-screen-sharing',
1717
REACTIONS_PICKER = 'reactions-picker',
18-
CHAT_UNREAD_BADGE_COUNT_INDICATOR = 'chat-unread-badge-count-indicator',
19-
BADGE_COUNT_INDICATOR = 'badge-count-indicator',
2018
}
2119

2220
export enum ButtonTestIds {

packages/react-native-sdk/src/translations/en.json

Lines changed: 6 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -11,7 +11,7 @@
1111
"End Stream": "End Stream",
1212
"Leave Stream": "Leave Stream",
1313
"Live": "Live",
14-
"Setup your test call": "Setup your test call",
14+
"Before joining": "Before joining",
1515
"Setup your audio and video": "Setup your audio and video",
1616
"Setup your audio": "Setup your audio",
1717
"You are first to Join the call.": "You are first to Join the call.",
@@ -23,5 +23,9 @@
2323
"{{ numberOfParticipants }} participant(s) are in the call.": "{{ numberOfParticipants }} participant(s) are in the call.",
2424
"You are about to join a call with id {{ callId }}.": "You are about to join a call with id {{ callId }}.",
2525
"Preparing call": "Preparing call",
26-
"You have left the call": "You have left the call"
26+
"You have left the call": "You have left the call",
27+
"You are about to join a call.": "You are about to join a call.",
28+
"Currently there are no other participants in the call.": "Currently there are no other participants in the call.",
29+
"There is {{numberOfParticipants}} more person in the call.": "There is {{numberOfParticipants}} more person in the call.",
30+
"There are {{numberOfParticipants}} more people in the call.": "There are {{numberOfParticipants}} more people in the call."
2731
}

sample-apps/react-native/dogfood/src/components/CallControlls/BadgeCountIndicator.tsx

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1,8 +1,8 @@
11
import React, { useMemo } from 'react';
22
import { StyleSheet, Text, View } from 'react-native';
33
import { useTheme } from '@stream-io/video-react-native-sdk';
4-
import { ComponentTestIds } from '@stream-io/video-react-native-sdk/src/constants/TestIds';
54
import { Z_INDEX } from '../../constants';
5+
import { ComponentTestIds } from '../../constants/TestIds';
66

77
/**
88
* A badge that displays a number.

sample-apps/react-native/dogfood/src/components/CallControlls/CallRecordingModal.tsx

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1,6 +1,6 @@
11
import { useTheme } from '@stream-io/video-react-native-sdk';
22
import { IconWrapper } from '@stream-io/video-react-native-sdk/src/icons';
3-
import { RecordCall } from '@stream-io/video-react-native-sdk/src/icons/RecordCall';
3+
import { RecordCall } from '../../assets/RecordCall';
44
import React, { useMemo } from 'react';
55
import {
66
Modal,

sample-apps/react-native/dogfood/src/components/CallControlls/CallStatusBadge.tsx

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1,7 +1,7 @@
11
import React, { useEffect, useMemo, useState } from 'react';
22
import { View, Text, StyleSheet } from 'react-native';
33
import { CallDuration } from '../../assets/CallDuration';
4-
import { RecordCall } from '@stream-io/video-react-native-sdk/src/icons/RecordCall';
4+
import { RecordCall } from '../../assets/RecordCall';
55
import { IconWrapper } from '@stream-io/video-react-native-sdk/src/icons';
66
import { useTheme } from '@stream-io/video-react-native-sdk';
77
import { useCallStateHooks } from '@stream-io/video-react-bindings';

sample-apps/react-native/dogfood/src/components/CallControlls/RecordCallButton.tsx

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -3,7 +3,7 @@ import {
33
CallControlsButton,
44
useTheme,
55
} from '@stream-io/video-react-native-sdk';
6-
import { RecordCall } from '@stream-io/video-react-native-sdk/src/icons/RecordCall';
6+
import { RecordCall } from '../../assets/RecordCall';
77
import { IconWrapper } from '@stream-io/video-react-native-sdk/src/icons';
88
import { CallRecordingModal } from './CallRecordingModal';
99

0 commit comments

Comments
 (0)