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

fix: onboarding carousel overflows causing navigation glitches #16514

Merged
merged 1 commit into from
Jul 7, 2023

Conversation

briansztamfater
Copy link
Member

@briansztamfater briansztamfater commented Jul 6, 2023

fixes #16512

Summary

Hides overflow of background carousel, as it leads to navigation glitches within onboarding

Platforms

  • iOS
Functional
  • onboarding

Steps to test

  • Open Status
  • Start account creation flow from onboarding
  • Navigate back to the previous onboarding steps
  • Verify correct behavior

status: ready

@briansztamfater briansztamfater self-assigned this Jul 6, 2023
@briansztamfater briansztamfater marked this pull request as draft July 6, 2023 16:02
@status-im-auto
Copy link
Member

status-im-auto commented Jul 6, 2023

Jenkins Builds

Click to see older builds (8)
Commit #️⃣ Finished (UTC) Duration Platform Result
✔️ 3fa3878 #1 2023-07-06 16:08:56 ~5 min android-e2e 🤖apk 📲
✔️ 3fa3878 #1 2023-07-06 16:09:34 ~6 min android 🤖apk 📲
✔️ 3fa3878 #1 2023-07-06 16:10:48 ~7 min ios 📱ipa 📲
✔️ 3fa3878 #1 2023-07-06 16:11:23 ~8 min tests 📄log
✔️ eec4f38 #2 2023-07-06 16:30:39 ~5 min android-e2e 🤖apk 📲
✔️ eec4f38 #2 2023-07-06 16:31:42 ~6 min android 🤖apk 📲
✔️ eec4f38 #2 2023-07-06 16:33:14 ~8 min tests 📄log
✔️ eec4f38 #2 2023-07-06 16:41:11 ~16 min ios 📱ipa 📲
Commit #️⃣ Finished (UTC) Duration Platform Result
✔️ b19a3dd #3 2023-07-07 12:42:21 ~6 min android-e2e 🤖apk 📲
✔️ b19a3dd #3 2023-07-07 12:42:48 ~6 min ios 📱ipa 📲
✔️ b19a3dd #3 2023-07-07 12:44:21 ~7 min android 🤖apk 📲
✔️ b19a3dd #3 2023-07-07 12:44:53 ~8 min tests 📄log
✔️ a058399 #4 2023-07-07 15:05:46 ~8 min android-e2e 🤖apk 📲
✔️ a058399 #4 2023-07-07 15:13:53 ~16 min android 🤖apk 📲
✔️ a058399 #4 2023-07-07 15:18:15 ~20 min ios 📱ipa 📲
✔️ a058399 #5 2023-07-07 15:33:04 ~9 min tests 📄log

@briansztamfater briansztamfater force-pushed the fix/onboarding-intro-navigation-back branch from 3fa3878 to eec4f38 Compare July 6, 2023 16:24
@briansztamfater briansztamfater changed the title [WIP] fix: onboarding carousel screen overflows affecting navigation [WIP] fix: onboarding carousel screen overflows causing navigation glitches Jul 6, 2023
@briansztamfater briansztamfater changed the title [WIP] fix: onboarding carousel screen overflows causing navigation glitches [WIP] fix: onboarding carousel overflows causing navigation glitches Jul 6, 2023
@briansztamfater briansztamfater changed the title [WIP] fix: onboarding carousel overflows causing navigation glitches [fix: onboarding carousel overflows causing navigation glitches Jul 6, 2023
@briansztamfater briansztamfater changed the title [fix: onboarding carousel overflows causing navigation glitches fix: onboarding carousel overflows causing navigation glitches Jul 6, 2023
@briansztamfater briansztamfater marked this pull request as ready for review July 6, 2023 16:57
@briansztamfater briansztamfater added this to the 1.24.0 - Alpha milestone Jul 6, 2023
@pavloburykh pavloburykh self-assigned this Jul 7, 2023
@pavloburykh
Copy link
Contributor

@briansztamfater thanx for the fix! PR will be ready for merge as soon as e2e run is finished. Will keep you updated.

@pavloburykh pavloburykh force-pushed the fix/onboarding-intro-navigation-back branch from eec4f38 to b19a3dd Compare July 7, 2023 12:36
@pavloburykh
Copy link
Contributor

@briansztamfater thanx, PR is ready for merge.

e2e were posted here #15614 (comment) by the mistake. Failed e2e are not PR related.

Signed-off-by: Brian Sztamfater <brian@status.im>
@briansztamfater briansztamfater force-pushed the fix/onboarding-intro-navigation-back branch from b19a3dd to a058399 Compare July 7, 2023 14:57
@briansztamfater briansztamfater merged commit 4414f8f into develop Jul 7, 2023
@briansztamfater briansztamfater deleted the fix/onboarding-intro-navigation-back branch July 7, 2023 15:55
codemaster115 pushed a commit that referenced this pull request Jul 7, 2023
Signed-off-by: Brian Sztamfater <brian@status.im>
andresceballosm pushed a commit to andresceballosm/status-mobile that referenced this pull request Jul 11, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
No open projects
Archived in project
Development

Successfully merging this pull request may close these issues.

Onboarding slide screen is displayed when navigating back to the previous onboarding screens (IOS only)
5 participants