Open shendy-a8c opened 3 days ago
9790
or branch name fix/7230-payments-details-mobile-view
in your-test.site/wp-admin/admin.php?page=jetpack-beta&plugin=woocommerce-payments :rocket: Launch a JN site with this branch :rocket:
:information_source: Install this Tampermonkey script to get more options.
Build info:
Note: the build is updated when a new commit is pushed to this PR.
Size Change: +195 B (0%)
Total Size: 1.34 MB
Filename | Size | Change |
---|---|---|
release/woocommerce-payments/dist/index-rtl.css |
52.7 kB | +76 B (0%) |
release/woocommerce-payments/dist/index.css |
52.6 kB | +79 B (0%) |
release/woocommerce-payments/dist/index.js |
303 kB | +40 B (0%) |
Fixes https://github.com/Automattic/woocommerce-payments/issues/7230
Changes proposed in this Pull Request
In my local, the pill does not overflow like shown in the screenshot of the original issue.
Before:
After:
I have to re-arrange the elements. I group the transaction amount and currency and move the
PaymentStatusChip
andDisputeStatusChip
.I also added
className
prop forPaymentStatusChip
andDisputeStatusChip
so I can target them in CSS selector.Before:
After:
Testing instructions
Make sure desktop view does not change due to these PR changes.
Make sure in mobile view, the status chips is located at the top.
Make sure desktop view does not change due to these PR changes.
Make sure in mobile view, the status chips is located at the top.
npm run changelog
to add a changelog file, choosepatch
to leave it empty if the change is not significant. You can add multiple changelog files in one PR by running this command a few times.Post merge