-
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
Android tabbar replace elevation with border #10832
Conversation
Jenkins BuildsClick to see older builds (11)
|
fb5c851
to
f34759a
Compare
97% of end-end tests have passed
Failed tests (3)Click to expand
Passed tests (92)Click to expand |
100% of end-end tests have passed
Passed tests (1) |
Looks good on Androids |
f34759a
to
1afa0c5
Compare
@Ferossgp can we merge this one ? |
@flexsurfer I'm experimenting with other approaches to change shadow in android, let's skip this PR for 1.5 |
97% of end-end tests have passed
Failed tests (3)Click to expand
Passed tests (92)Click to expand |
Small request - could you please rebase to latest master so this commit b5fda12 gets included? This means we aren't running E2E tests on eth.prod cluster anymore, and since this happens on every push, the impact is quite big. This ensures we have more accurate metrics going forward from Jul 1 onward, which would be awesome. See https://discuss.status.im/t/user-growth-and-retention/1782 for more |
@Ferossgp any updates on this ? |
acca83a
to
b007520
Compare
99% of end-end tests have passed
Failed tests (1)Click to expand
Passed tests (90)Click to expand
|
Checked with Android 6, 8, 9, 10 - no gradient anymore with dark and light mode. And it shows on iOS. |
Signed-off-by: Gheorghe Pinzaru <feross95@gmail.com>
b007520
to
8342972
Compare
Fixes #10604