Skip to content

Migrate to shadcn/ui chart components#3

Draft
Copilot wants to merge 3 commits intomainfrom
copilot/switch-to-shadcn-charts
Draft

Migrate to shadcn/ui chart components#3
Copilot wants to merge 3 commits intomainfrom
copilot/switch-to-shadcn-charts

Conversation

Copy link

Copilot AI commented Dec 14, 2025

Replaces direct recharts usage with shadcn/ui chart wrappers for consistent theming and styling across dashboard visualizations.

Changes

  • Added components/ui/chart.tsx: Implements shadcn chart primitives (ChartContainer, ChartTooltip, ChartTooltipContent, ChartLegend) with CSS variable-based theming and type-safe value rendering

  • Updated workload-heatmap.tsx: Wrapped BarChart in ChartContainer, replaced Recharts Tooltip with ChartTooltip, migrated colors from inline vars to hsl(var(--chart-1)) format

  • Updated course-strategy-sidebar.tsx: Wrapped PieChart in ChartContainer, replaced Recharts Tooltip with ChartTooltipContent, refactored data structure to use fill property with proper opacity handling

Example

Before:

<ResponsiveContainer width="100%" height="100%">
  <BarChart data={chartData}>
    <Tooltip content={...} />
  </BarChart>
</ResponsiveContainer>

After:

<ChartContainer config={chartConfig} className="h-[200px]">
  <BarChart data={chartData}>
    <ChartTooltip content={...} />
  </BarChart>
</ChartContainer>

The chart component layer provides unified styling hooks while preserving existing recharts functionality and data structures.

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:

  • fonts.googleapis.com
    • Triggering command: /usr/local/bin/node node /home/REDACTED/work/student-os/student-os/node_modules/.bin/next build (dns block)
  • ui.shadcn.com
    • Triggering command: /usr/local/bin/node node /home/REDACTED/.npm/_npx/d66c5096c7023bfb/node_modules/.bin/shadcn add chart (dns block)

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

Original prompt

This section details on the original issue you should resolve

<issue_title>Switch to shadcn charts</issue_title>
<issue_description>https://ui.shadcn.com/charts/bar#charts</issue_description>

Comments on the Issue (you are @copilot in this section)


✨ Let Copilot coding agent set things up for you — coding agent works faster and does higher quality work when set up for your repo.

@vercel
Copy link

vercel bot commented Dec 14, 2025

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Review Updated (UTC)
student-os Error Error Dec 14, 2025 10:55am

@coderabbitai
Copy link
Contributor

coderabbitai bot commented Dec 14, 2025

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.

Copilot AI changed the title [WIP] Switch to shadcn charts for data visualization Migrate to shadcn/ui chart components Dec 14, 2025
Copilot AI requested a review from Lemirq December 14, 2025 10:56
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.

Switch to shadcn charts

2 participants