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 android padding calc #5877

Merged
merged 1 commit into from
Jun 21, 2024
Merged

fix android padding calc #5877

merged 1 commit into from
Jun 21, 2024

Conversation

brunobar79
Copy link
Member

Fixes APP-####

What changed (plus any additional context for devs)

We were asumming all android phones have the softMenu enabled and that's not the case for my Pixel 4.
I've added a check and adjusted the padding accordingly.

Screen recordings / screenshots

BEFORE
Screenshot 2024-06-20 at 5 22 19 PM

AFTER
Screenshot 2024-06-20 at 5 19 27 PM

What to test

Copy link
Contributor

@ibrahimtaveras00 ibrahimtaveras00 left a comment

Choose a reason for hiding this comment

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

Fix looks good on different sized screens ✅

@brunobar79 brunobar79 merged commit 37c16c1 into develop Jun 21, 2024
6 checks passed
@brunobar79 brunobar79 deleted the @bruno/android-keyboard-fix branch June 21, 2024 04:12
BrodyHughes added a commit that referenced this pull request Jun 26, 2024
…able-unit-tests

* 'develop' of github.com:rainbow-me/rainbow:
  rerun only failed tests (#5878)
  bump versions to v1.9.28 (#5893)
  input asset regression fix on v1 (#5891)
  bump to version 1.9.27 (#5890)
  take into account l1 op  gas fee for claimBridge (#5889)
  Points UI (#5866)
  Remove input value typeof checks (#5886)
  Fix swap button presses, adjust button state logic (#5884)
  Swaps: Gas fee range to use for native asset maxSwappableAmount buffer (#5881)
  clear custom gas settings (#5875)
  Review button states (#5854)
  fix handler (#5880)
  Use graphql ens key everywhere (#5882)
  Fix chain badges on android (#5867)
  fix android padding calc (#5877)
  add unknown type as failed (#5871)
  Swaps: gas icons, review sheet cleanup (#5872)
  Swap flashbots min priority fee (#5869)
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.

None yet

2 participants