Skip to content
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(react-native): add landscape mode support to Lobby, RingingCallContent and DF app #1123

Merged
merged 8 commits into from
Oct 12, 2023

Conversation

khushal87
Copy link
Member

@khushal87 khushal87 commented Oct 3, 2023

Add support for landscape mode temporarily for Lobby and RingingCall components so that it doesn't overflow when rendered alongside other components.

SDK components:

  • Lobby
    Screenshot_1696869145
  • Incoming Call
    Simulator Screenshot - iPhone SE (3rd generation) - 2023-10-10 at 10 58 34
    Screenshot_1696878441
  • Outgoing Call
    Screenshot_1696915730

DF app screens:
Screenshot_1696868884
Screenshot_1696869005
Screenshot_1696869029
Screenshot_1696878061

@khushal87 khushal87 changed the title feat(react-native): add landscape mode support to Lobby and RingingCallContent feat(react-native): add landscape mode support to Lobby, RingingCallContent and DF app Oct 3, 2023
}: IncomingCallProps) => {
const { t } = useI18n();
const {
theme: { colors, incomingCall, typefaces },
} = useTheme();

const landScapeContentStyles: ViewStyle = {
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggested change
const landScapeContentStyles: ViewStyle = {
const landscapeContentStyles: ViewStyle = {

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Fixed

@@ -71,12 +74,16 @@ const JoinCallScreen = () => {
}
};

const landScapeStyles: ViewStyle = {
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggested change
const landScapeStyles: ViewStyle = {
const landscapeStyles: ViewStyle = {

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Fixed

@khushal87 khushal87 merged commit cc247f0 into main Oct 12, 2023
8 of 11 checks passed
@khushal87 khushal87 deleted the khushal87/landscape-mode-lobby-ringing branch October 12, 2023 13:30
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants