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.

Sign up for GitHub

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

Jump to bottom

Jump to overlaps last asset item in the list on Wallet screen #19980

Closed
pavloburykh opened this issue May 10, 2024 · 2 comments · Fixed by #19990
Closed

Jump to overlaps last asset item in the list on Wallet screen #19980

pavloburykh opened this issue May 10, 2024 · 2 comments · Fixed by #19990
Assignees
@OmarBasem
Labels
E:Mobile Bug MVP E:Mobile Wallet Main Page MVP all issues for the main page of the wallet E:Mobile Wallet MVP Design system components, initial research and page wireframes

Comments

@pavloburykh
Copy link
Contributor

Looks like leftover from #19732 cc @OmarBasem

Steps:

  1. Login account with long list of assets
  2. Open wallet tab
  3. Scroll assets list to the bottom
  4. Pay attention at Jump to button location

Actual result:

telegram-cloud-document-2-5327995973950000350.mp4

Additional Information

  • Status version: nightly
  • Operating System: Android, iOS
@pavloburykh pavloburykh added E:Mobile Bug MVP E:Mobile Wallet MVP Design system components, initial research and page wireframes E:Mobile Wallet Main Page MVP all issues for the main page of the wallet labels May 10, 2024
@pavloburykh pavloburykh changed the title Jump to overlaps lats asset item in the list on Wallet screen Jump to overlaps last asset item in the list on Wallet screen May 10, 2024
@OmarBasem
Copy link
Member

OmarBasem commented May 10, 2024 •
edited

@pavloburykh thanks for opening the issue. It is a regression from #19831

I can take it anyways 👍

@OmarBasem OmarBasem self-assigned this May 10, 2024
@pavloburykh
Copy link
Contributor Author

@pavloburykh thanks for opening the issue. It is a regression from #19831

I can take it anyways 👍

Got it! Thanks a lot @OmarBasem!

@OmarBasem OmarBasem mentioned this issue May 13, 2024
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@OmarBasem OmarBasem

Labels
E:Mobile Bug MVP E:Mobile Wallet Main Page MVP all issues for the main page of the wallet E:Mobile Wallet MVP Design system components, initial research and page wireframes
Projects
Do not use - tasks moved to unified Desktop / Mobile Board
Status: Done
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

fix: jump-to button positioning status-im/status-mobile
2 participants
@OmarBasem @pavloburykh