Replace homebrew list virtualization with react-window#415
Conversation
|
The latest updates on your projects. Learn more about Vercel for GitHub.
|
- Install react-window package - Create reusable VirtualizedList component - Replace ProgressiveRender in block-panel-generic list-view - Replace ProgressiveRender in lists/list-view - Replace manual rendering in BlockListsView - Preserve ad slot insertion functionality Co-authored-by: thieflord06 <25804734+thieflord06@users.noreply.github.com>
- Update RenderSearchResults to use VirtualizedList - Simplify HistoryScrollableList by removing progressive rendering - Remove unused ProgressiveRender component - All list views now use react-window virtualization Co-authored-by: thieflord06 <25804734+thieflord06@users.noreply.github.com>
|
@copilot Where in the code is staging.api.clearsky.services referenced? |
The reference to The staging URL is used when:
This is defined in the |
|
@copilot staging.api.clearsky.services isn't in the current codebase and only one domain is being used, can you merge the latest commits and verify that this addresses the issue? |
Verified: The codebase is already using the unified Key findings:
The domain unification was already done in the base commit (2cf880d "Update to use same domain") before my virtualization changes. The current code correctly uses only |
Replaces the custom
ProgressiveRendercomponent with thereact-windowlibrary for proper list virtualization across all list views.Changes
New VirtualizedList component (
src/common-components/virtualized-list.jsx)react-window'sListwith viewport height calculation, ad insertion (every 35 items), and infinite scroll detectiononReachEndcallback for paginationUpdated list views to use VirtualizedList:
Removed:
ProgressiveRendercomponentHistoryScrollableList(now handled by virtualization)Example
Before (homebrew progressive rendering):
After (react-window virtualization):
The virtualized list only renders visible items in the DOM, significantly reducing memory usage and improving scroll performance for large lists while preserving ad placement and pagination behavior.
Warning
Firewall rules blocked me from connecting to one or more addresses (expand for details)
I tried to connect to the following addresses, but was blocked by firewall rules:
staging.api.clearsky.servicesnode /home/REDACTED/work/ClearskyUI/ClearskyUI/node_modules/.bin/vite --open(dns block)If you need me to access, download, or install something from one of these locations, you can either:
Original prompt
💡 You can make Copilot smarter by setting up custom instructions, customizing its development environment and configuring Model Context Protocol (MCP) servers. Learn more Copilot coding agent tips in the docs.