Skip to content

Implement Initial Atlas#352

Open
PauloMFJ wants to merge 22 commits into
datacommonsorg:mainfrom
madebypxlp:paulo/atlas
Open

Implement Initial Atlas#352
PauloMFJ wants to merge 22 commits into
datacommonsorg:mainfrom
madebypxlp:paulo/atlas

Conversation

@PauloMFJ
Copy link
Copy Markdown
Contributor

@PauloMFJ PauloMFJ commented Jun 1, 2026

Overview

Implements the initial Atlas — an infinite, tldraw-backed canvas that renders data "cards" laid out on a grid. This establishes the rendering foundation and the component architecture for the app, ahead of wiring up a real data source.

@PauloMFJ PauloMFJ self-assigned this Jun 1, 2026
Copy link
Copy Markdown

@gemini-code-assist gemini-code-assist Bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Code Review

This pull request introduces several foundational UI components such as Button, CardBase, CardChart, and CardText, along with custom hooks and utility functions. The review feedback highlights critical issues that need to be addressed: action buttons in CardBase must stop pointerDown propagation to prevent interfering with tldraw canvas gestures; several accessibility violations of WCAG 2.2 AA standards exist in the tab and button components due to missing ARIA attributes; and a layout bug in CardText allows the loading skeleton and body to render simultaneously.

Comment thread dataweaver/apps/web/src/components/elements/card/base.tsx
Comment thread dataweaver/apps/web/src/components/elements/card/chart/tabs.tsx Outdated
Comment thread dataweaver/apps/web/src/components/elements/card/chart/chart.tsx Outdated
Comment thread dataweaver/apps/web/src/components/elements/card/text/text.tsx Outdated
Comment thread dataweaver/apps/web/src/components/elements/button.tsx
@PauloMFJ PauloMFJ requested review from beets and nick-nlb June 3, 2026 10:28
@datacommonsorg datacommonsorg deleted a comment from google-cla Bot Jun 3, 2026
@PauloMFJ PauloMFJ marked this pull request as ready for review June 3, 2026 10:29
@PauloMFJ PauloMFJ requested a review from a team as a code owner June 3, 2026 10:29
nick-nlb
nick-nlb previously approved these changes Jun 4, 2026
Copy link
Copy Markdown

@nick-nlb nick-nlb left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks for this Paulo!

A couple of comments (one is a nit) but otherwise looks good to go.

Comment thread dataweaver/apps/web/src/components/elements/card/chart/data_line_chart.tsx Outdated
Comment thread dataweaver/apps/web/src/components/scopes/atlas/atlas.tsx Outdated
beets
beets previously approved these changes Jun 4, 2026
@PauloMFJ PauloMFJ dismissed stale reviews from beets and nick-nlb via 89e07ed June 5, 2026 08:30
@nick-nlb nick-nlb self-requested a review June 5, 2026 16:21
@PauloMFJ PauloMFJ requested a review from miss-o-soup as a code owner June 5, 2026 16:28
@nick-nlb nick-nlb self-requested a review June 5, 2026 16:29
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants