Skip to content

[MS-226] LiveFeedbackFragment now utilizes different layouts for horizontal and vertical screen orientations#735

Merged
alexandr-simprints merged 4 commits into
MS-2-epic-landscape-supportfrom
MS-226-face-live-feedback-screen-support-configuration-changes
May 30, 2024
Merged

[MS-226] LiveFeedbackFragment now utilizes different layouts for horizontal and vertical screen orientations#735
alexandr-simprints merged 4 commits into
MS-2-epic-landscape-supportfrom
MS-226-face-live-feedback-screen-support-configuration-changes

Conversation

@alexandr-simprints
Copy link
Copy Markdown
Contributor

Due to historical reasons, the preview is formed around the guideline in the layout, whose height is defined as percentage of the screen height. It is hardcoded to 0.3 (30% of sceren height), and it doesn't work for the smaller horizontal dimensions. Thus, the new horizontal layout is introduced, that has a guideline set at the 0.5 (50% of screen height).

face-capture-1
face-capture-2
face-capture-3
face-capture-4
image

…zontal and vertical screen orientations. Due to historical reasons, the preview is formed around the guideline in the layout, whose height is defined as percentage of the screen height. It is hardcoded to 0.3 (30% of sceren height), and it doesn't work for the smaller horizontal dimensions. Thus, the new horizontal layout is introduced, that has a guideline set at the 0.5 (50% of screen height).
@sonarqubecloud
Copy link
Copy Markdown

Quality Gate Failed Quality Gate failed

Failed conditions
71.4% Coverage on New Code (required ≥ 80%)

See analysis details on SonarCloud

…blic getter for the 'cropRect' property for testing purposes
@alexandr-simprints alexandr-simprints merged commit 277d976 into MS-2-epic-landscape-support May 30, 2024
@alexandr-simprints alexandr-simprints deleted the MS-226-face-live-feedback-screen-support-configuration-changes branch May 30, 2024 13:41
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants