feat(virtual-table): Declarative VirtualTable — Table-matched default styling (dev-v3)#4747
Draft
gethinwebster wants to merge 3 commits into
Draft
feat(virtual-table): Declarative VirtualTable — Table-matched default styling (dev-v3)#4747gethinwebster wants to merge 3 commits into
gethinwebster wants to merge 3 commits into
Conversation
Add a new additive component that coexists with Table, aimed at large and streaming datasets: - First-class row virtualization with a single owned scroll container, opt-in per-row measurement, and incremental scroll-offset correction. - Arbitrary non-tabular row expansion (render slot) with measured variable expanded-row heights and a valid grid-child disclosure a11y model. - Full-dataset ARIA grid contract (aria-rowcount/rowindex, aria-colcount/ colindex incl. a materialised disclosure column) under windowing. - Config-driven props API mirroring Table where it transfers. - CloudWatch Logs Insights demo pages (standard, patterns, raw/file views), unit + a11y tests, usage docs, and a test-utils wrapper.
…e Table design tokens; native scrollbar)
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Draft PR for the Declarative (F1-A1) VirtualTable thread, reworked on the
dev-v3-gethinw-virtualtable-declarativebranch (thedev-v3-*prefix mirrors into Brazil AwsUi-v3).What changed vs the P4 thread
21ee74d) so all thread work (config-driven core, CloudWatch standard/patterns/raw dev pages, test-utils, unit + a11y tests, USAGE.md) is carried forward verbatim.Tableby reusing Table's own design tokens: header thead surface + divider, header-cell color/font/line-height/padding, per-row divider, selected-row surface (byte-identical to Table), sticky-header surface, default single-line truncate (= TablewrapLines=false) with an opt-in.cell-wraphook (= TablewrapLines), and an expanded-region detail-panel treatment. No bespoke/invented values.::-webkit-scrollbar, no overlay/synthetic scrollbar; the virtualization viewport scrolls natively.Verification (green)
npm run build— exit 0 (only pre-existing webpack asset-size warnings)npm run lint(eslint + stylelint + prettier) — 0 errorsSupersedes
This dev-v3 branch supersedes draft PR #4738 (head
virtualtable-F1-A1) for the Declarative thread. A PR's head branch cannot be repointed, so this is a fresh draft; #4738 can be closed as superseded.Human-gated (out of scope)
Official Cloudscape API review + design/release sign-off; the
aria-livenode -> internalLiveRegiondocumenter follow-up.