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

P2022-1506 Prepare landspace layout/view #40

Merged
merged 2 commits into from
Mar 14, 2022

Conversation

Anna-Surma
Copy link
Contributor

Creates landspace for: activity_details, activity_login, activity_reset_password, fragment_sign_in, fragment_sign_up.

Removes repeating color.

Changing the button padding to incompatible with the design. Padding of the size given in the design cut off a piece of the text for smaller phones.

Adds highlighting edit text in login files.

Creates landscape for: activity_details, activity_login, activity_reset_password, fragment_sign_in, fragment_sign_up.

Removes repeating color.

Changing the button padding to incompatible with the design. Padding of the size given in the design cut off a piece of the text for smaller phones.

Adds highlighting edit text in login files.
@Anna-Surma
Copy link
Contributor Author

Anna-Surma commented Mar 13, 2022

obraz
obraz
obraz
obraz
obraz
obraz

Copy link
Contributor

@Lucas-Fresko Lucas-Fresko left a comment

Choose a reason for hiding this comment

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

Everything works.

Interesting use of nested scroll view and collapsing toolbar in details activity but I doubt that the user would care for scrolling before accessing the description and play button. (Ideally the play button would be at the top, over the image, or even a trailer would start playing automatically.)

The esthetics could be discussed (button not taking up the entire width of the screen), and we could consider putting tablayout in line with the app logo to save space and avoid scrolling, but without official designs everything goes and this successfully makes landscape views functional.

No access to reset password screen, but that's unrelated.

@Anna-Surma
Copy link
Contributor Author

I was also thinking about the changes you mentioned, but as you wrote, there is no design, so I tried to make it as similar to portrait mode as possible. We can arrange the details at the meeting.

@kosiara kosiara merged commit 5a3011d into master Mar 14, 2022
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.

3 participants