fix: apply resizing fix #47
Conversation
|
cc @rezkiy37 |
rezkiy37
left a comment
rezkiy37
left a comment
There was a problem hiding this comment.
Also, we need to update the example once it is deployed.
| { | ||
| "name": "react-fast-pdf", | ||
| "version": "1.0.25", | ||
| "version": "1.0.26", |
There was a problem hiding this comment.
The bot bumps it automatically.
| "version": "1.0.26", |
There was a problem hiding this comment.
strange, it should be already like that on main
There was a problem hiding this comment.
https://github.com/Expensify/react-fast-pdf/blob/main/package.json same on our fork 🤔
There was a problem hiding this comment.
Yes, I see it. What's going on? 😅
dangrous
left a comment
dangrous
left a comment
There was a problem hiding this comment.
This looks good! What's the process for getting these out? Do I just merge? What do I need to do after? Thanks!
|
|
||
| # NPM file created by GitHub actions | ||
| .npmrc | ||
| .idea/ |
There was a problem hiding this comment.
NAB - what's this change?
There was a problem hiding this comment.
it's for Webstorm users - to ignore local files that Webstorm creates
|
hello! just a friendly bump on this one 😃 |
4 participants
Details
Implement cache invalidation using react-window's resetAfterIndex API and combining it with another useLayoutEffect in this component - to prevent
topvalue miscalculation for the PageRenderer.Related Issues
Expensify/App#55941
Manual Tests
Screen.Recording.2025-02-13.at.13.48.45.mp4
Linked PRs