-
Notifications
You must be signed in to change notification settings - Fork 72
Fix: Gradient overlay on tables does not disappear on resize #10970
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
base: develop
Are you sure you want to change the base?
Fix: Gradient overlay on tables does not disappear on resize #10970
Conversation
Test the buildOption 1. Jetpack Beta
Option 2. Jurassic Ninja - available for logged-in A12s🚀 Launch a JN site with this branch 🚀 ℹ️ 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: +222 B (0%) Total Size: 869 kB
ℹ️ View Unchanged
|
…does-not-disappear-on-resize
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I think the solution works for us, but since this is a WC Core component, shouldn't the fix lie in the Table
component we consume from WC Core? It seems that the calculation that adds the is-scrollable-right
class isn't working correctly.
What seems to be happening to me is that this useEffect
in WC Core ( https://github.com/woocommerce/woocommerce/blob/trunk/packages/js/components/src/table/table.tsx#L158-L174 ) doesn't account for whether the table is scrollable based on the new window dimensions. The updateTableShadow
seems to be applying only scroll position changes, but doesn't handle whether the table is scrollable at all.
…does-not-disappear-on-resize
…does-not-disappear-on-resize
FYI @frosso I just opened this pull request that addresses the root cause in the |
…does-not-disappear-on-resize
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
thank you for fixing this in WooCommerce
Fixes #WOOPMNT-5217
This PR fixes an issue where the linear gradient overlay on WooCommerce Payments tables remains visible after resizing the viewport back to a larger size.
Changes proposed in this Pull Request
client/style.scss
to hide the gradient overlay on larger screensTesting instructions
Payments > Transactions
Payments > Payouts
Payments > Disputes
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