-
Notifications
You must be signed in to change notification settings - Fork 984
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
First carrousel image onboarding is cut off #10427
Comments
@hesterbruikman, which device is this? I've tested on a Pixel 2 and cannot replicate. |
Ah interesting. This is on Galaxy S10e. Let me check iPhone 👀 |
Fixes #10427 ### Summary Narrow devices were causing text in the intro carousel to grow to two lines on *the first* slide but not others, causing the height to be recalculated incorrectly on the last slide (that only had one line of text). Create a vector of atoms that can individually track the heights required for each slide independently. ### Testing notes #### Areas that maybe impacted - Intro screen status: ready
@hesterbruikman -- seemed to happen on narrow devices (where the title was pushed to two lines). See the comments in #10442 as there is some UI guidance needed. |
Good catch, looks like that's it. Just checked iPhone 8 which is fine too. @errorists any advice on how to handle text on small screens in onboarding carousel? |
@hesterbruikman I already commented in the issue |
Fixes #10427 Narrow devices were causing text in the intro carousel to grow to two lines on *the first* slide but not others, causing the height to be recalculated incorrectly on the last slide (that only had one line of text). Create a vector of atoms that can individually track the heights required for each slide independently. - Intro screen status: ready
…ow to two lines on *the first* slide but not others, causing the height to be recalculated incorrectly on the last slide (that only had one line of text).
…ow to two lines on *the first* slide but not others, causing the height to be recalculated incorrectly on the last slide (that only had one line of text).
…ow to two lines on *the first* slide but not others, causing the height to be recalculated incorrectly on the last slide (that only had one line of text).
…ow to two lines on *the first* slide but not others, causing the height to be recalculated incorrectly on the last slide (that only had one line of text).
…ow to two lines on *the first* slide but not others, causing the height to be recalculated incorrectly on the last slide (that only had one line of text). Signed-off-by: Andrey Shovkoplyas <motor4ik@gmail.com>
Fixed in #10442 |
Problem
In dark mode it becomes visible that the walkie-talkie is cut off at the bottom.
Implementation
Requires investigation as to why the image is cut off.
Specification of image size, position and padding in Figma
Acceptance Criteria
Full image is shown
Notes
cc @errorists for review
Future Steps
The text was updated successfully, but these errors were encountered: