Skip to content

Conversation

Copy link
Contributor

Copilot AI commented Jan 6, 2026

New Pull Request Checklist

Issue Description

PR #2867 contains chart visualization feature but has rebase conflicts and includes unrelated changes (line endings, etc.). This PR extracts only the chart visualization changes.

Implements interactive chart visualization for selected cells in the data browser, supporting time series and numeric data with bar, line, and pie charts in a draggable/resizable panel.

Approach

Dependencies

  • Added chart.js (^4.5.0), react-chartjs-2 (^5.3.0), chartjs-adapter-date-fns (^3.0.0)
  • Fixed webpack config to handle react-chartjs-2's ESM imports by adding fullySpecified: false to module resolution

New Components

  • ChartVisualization: Renders charts with automatic type detection (time series vs numeric), supports Bar/Line/Pie switching
  • DraggableResizablePanel: Reusable window component with drag/resize handles

DataBrowser Integration

  • Enhanced handleCellClick for Shift+Click multi-cell selection across Number, Date, and String columns
  • Added state: isChartPanelVisible, selectedData, numericSelectedData, hasDateInSelection
  • Auto-closes chart panel when underlying data changes (sorting, filtering)
  • Separates numeric values for Sum button when dates are in selection

Toolbar Updates

  • Added chart toggle button with analytics icon
  • Responsive styling with mobile breakpoints
  • Shows only when 2+ cells selected

Chart Detection Logic

// Scans columns for dates, validates 60% threshold
if (dateCount >= totalRows * 0.6) {
  isTimeSeries = true;
  // Use date column for X-axis, numeric columns as datasets
}

TODOs before merging

  • Add tests
  • Add changes to documentation (guides, repository pages, in-code descriptions)

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:

  • googlechromelabs.github.io
    • Triggering command: /usr/local/bin/node node install.mjs (dns block)
  • https://storage.googleapis.com/chrome-for-testing-public/138.0.7204.94/linux64/chrome-linux64.zip
    • Triggering command: /usr/local/bin/node node install.mjs (http block)

If you need me to access, download, or install something from one of these locations, you can either:

Original prompt

Create a new PR based on #2867 with only the changes needed for the chart visualization feature. The old PR has rebase issues and therefore contains a lot of unrelated changes.


💬 We'd love your input! Share your thoughts on Copilot coding agent in our 2 minute survey.

@coderabbitai
Copy link

coderabbitai bot commented Jan 6, 2026

Important

Review skipped

Bot user detected.

To trigger a single review, invoke the @coderabbitai review command.

You can disable this status message by setting the reviews.review_status to false in the CodeRabbit configuration file.


Comment @coderabbitai help to get the list of available commands and usage tips.

@parseplatformorg
Copy link
Contributor

parseplatformorg commented Jan 6, 2026

Snyk checks have passed. No issues have been found so far.

Status Scanner Critical High Medium Low Total (0)
Open Source Security 0 0 0 0 0 issues

💻 Catch issues earlier using the plugins for VS Code, JetBrains IDEs, Visual Studio, and Eclipse.

Copilot AI and others added 3 commits January 6, 2026 14:45
Co-authored-by: mtrezza <5673677+mtrezza@users.noreply.github.com>
Co-authored-by: mtrezza <5673677+mtrezza@users.noreply.github.com>
Co-authored-by: mtrezza <5673677+mtrezza@users.noreply.github.com>
Copilot AI changed the title [WIP] Add chart visualization feature to parse-dashboard feat: Add chart visualization to data browser Jan 6, 2026
Copilot AI requested a review from mtrezza January 6, 2026 15:03
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