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

Overview > Balances card should show balances in stacked layout on mobile #8599

Open
nagpai opened this issue Apr 9, 2024 · 3 comments · May be fixed by #10040
Open

Overview > Balances card should show balances in stacked layout on mobile #8599

nagpai opened this issue Apr 9, 2024 · 3 comments · May be fixed by #10040
Assignees
Labels
category: core WC Payments core related issues, where it’s obvious. focus: reporting priority: medium The issue/PR is medium priority—non-critical functionality loss, minimal effect on usability type: enhancement The issue is a request for an enhancement.

Comments

@nagpai
Copy link
Contributor

nagpai commented Apr 9, 2024

The Balance widget in the overview page, currently shows Total balance and Available funds side-by-side on desktop as well as mobile view.

On mobile screen, we would want this one below the other.

To Reproduce

  1. Browse to Payments section in admin
  2. Check how the Balance widget currently appears

Additional context

The responsive behavior improvement for the Balances tiles was recommended by design - @rogermattic , to make it consistent with the mobile view of the payment activity widget tiles.

p1712249591796559/1712130711.969139-slack-C06BBFTF6EM

Props - @rogermattic

Design mockups

Mobile views
@nagpai nagpai added type: enhancement The issue is a request for an enhancement. priority: low The issue/PR is low priority—not many people are affected or there’s a workaround, etc. category: core WC Payments core related issues, where it’s obvious. focus: reporting labels Apr 9, 2024
@haszari haszari changed the title Overview: Fix mobile view of the Balances widget Overview > Balances card should show balances in stacked layout on mobile Dec 18, 2024
@haszari haszari added priority: medium The issue/PR is medium priority—non-critical functionality loss, minimal effect on usability and removed priority: low The issue/PR is low priority—not many people are affected or there’s a workaround, etc. labels Dec 18, 2024
@haszari
Copy link
Contributor

haszari commented Dec 18, 2024

Upgrading this to medium and adding to current sprint. This will make WooPayments better on mobile and should be relatively straightforward – quick win!

p1712249591796559/1712130711.969139-slack-C06BBFTF6EM

@nagpai could you add anything relevant from that slack link inline? E.g. where did this enhancement originate, is it follow up to a project/issue, general design feedback or reported by a merchant/support?

@brucealdridge
Copy link
Contributor

FYI: In the mobile app these are side-by-side. Not sure if we want consistency here.

Image

Sidenote: Mobile app shows Pending/Available funds and there is an issue open for this woocommerce/woocommerce-ios#13206

@nagpai nagpai self-assigned this Dec 24, 2024
@nagpai
Copy link
Contributor Author

nagpai commented Dec 25, 2024

The design has changed slightly from the screenshots in the main message. We have moved the currency selector outside the account balances box . Hence the final outcome of the PR would be slightly different visually.

@nagpai nagpai linked a pull request Dec 25, 2024 that will close this issue
6 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
category: core WC Payments core related issues, where it’s obvious. focus: reporting priority: medium The issue/PR is medium priority—non-critical functionality loss, minimal effect on usability type: enhancement The issue is a request for an enhancement.
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants