Closed Jinksi closed 1 day ago
9763
or branch name fix/7014-documents-list-view-table-download-label
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: +45 B (0%)
Total Size: 1.33 MB
Filename | Size | Change |
---|---|---|
release/woocommerce-payments/dist/index-rtl.css |
52.6 kB | +21 B (0%) |
release/woocommerce-payments/dist/index.css |
52.5 kB | +20 B (0%) |
release/woocommerce-payments/dist/index.js |
302 kB | +4 B (0%) |
Requesting a review from @jessy-p as a fellow reporting quality squad member this sprint. 🙏
✅ - Ensure you can see the "Download" column label
✅ - Open the three dots menu and ensure the menu includes a "Download" toggle
I am able to see this, however the menu itself seems clipped on the right side. The issue seems to be with an upstream component though since I am seeing the same issue with Woo core analytics pages. ( I will test later and create a core issue if needed )
I am able to see this, however the menu itself seems clipped on the right side. The issue seems to be with an upstream component though since I am seeing the same issue with Woo core analytics pages. ( I will test later and create a core issue if needed )
@nagpai +1 and there's an open issue for it https://github.com/woocommerce/woocommerce/issues/43256, so I've added some context to it, mentioning that this impacts downstream consumers of TableCard
.
I made a fix for TableCard in https://github.com/woocommerce/woocommerce/pull/53066
FWIW, if you face a similar issue with EllipsisMenu
elsewhere, you can use <EllipsisMenu placement="bottom-end">
to fix it (once the PR gets merged).
Fixes #7014
Changes proposed in this Pull Request
Adds the missing "Download" column heading label and toggle menu option to the Payments → Documents list view table.
I've also:
isLink
Button component prop withvariant="link"
on the download buttonBefore/After gifs
Document list view table:
Document list view table toggle menu:
Testing instructions
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