Skip to content

feat: better dashboard chart#378

Merged
KMKoushik merged 3 commits intomainfrom
better-ui
Mar 15, 2026
Merged

feat: better dashboard chart#378
KMKoushik merged 3 commits intomainfrom
better-ui

Conversation

@KMKoushik
Copy link
Member

@KMKoushik KMKoushik commented Mar 15, 2026


Summary by cubic

Improves the dashboard email chart with interactive, accessible metric filtering and cleaner visuals. The dashboard now defaults to a 30‑day view and includes refreshed sidebar colors.

  • New Features

    • Click metric cards to show/hide series; multi-select supported; “Total” is read-only.
    • Cards are buttons with focus/ARIA states; chart and tooltip only show selected, non-zero metrics.
    • Rounded bar tops adjust to the current visible stack order.
  • UI Improvements

    • Default time range is 30 days across filters, chart, and reputation metrics.
    • Cleaner X-axis (no axis/tick lines, muted ticks) and larger bars (20px).
    • Updated sidebar colors for light and dark themes.

Written for commit 2655459. Summary will update on new commits.

Summary by CodeRabbit

  • New Features

    • Email chart metrics are now toggleable for custom views; chart and tooltip update to reflect selected metrics and "total" behavior.
    • Dashboard default time window changed to 30 days.
  • Style

    • Updated sidebar color tokens for light and dark themes.
    • Minor visual and axis/tooltip refinements for the email chart.

@vercel
Copy link

vercel bot commented Mar 15, 2026

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

Project Deployment Actions Updated (UTC)
unsend-marketing Ready Ready Preview, Comment Mar 15, 2026 7:28am

Copy link

@greptile-apps greptile-apps bot left a comment

Choose a reason for hiding this comment

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

Your free trial has ended. If you'd like to continue receiving code reviews, you can add a payment method here.

@coderabbitai
Copy link
Contributor

coderabbitai bot commented Mar 15, 2026

Caution

Review failed

The pull request is closed.

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: Organization UI

Review profile: CHILL

Plan: Pro

Run ID: 99b363b8-d0e4-4239-978e-36180dd12686

📥 Commits

Reviewing files that changed from the base of the PR and between bc627d7 and 2655459.

📒 Files selected for processing (1)
  • packages/ui/styles/globals.css

Walkthrough

Default dashboard time window defaults changed from 7 to 30 days across page-level state and child components. The email chart now supports interactive metric toggling: selectedMetrics state controls which metric bars render, tooltip entries and per-date rows are generated from the visible metrics, createRoundedTopShape accepts visible stack order, and EmailChartItem gains isActive/isClickable/onClick props. CSS theme tokens for sidebar colors were adjusted for light and dark modes. Minor formatting-only edits were made in the dashboard service.

Possibly related PRs

  • usesend/useSend PR 320: Modifies email-chart.tsx tooltip and chart rendering logic, including payload handling and metric visibility/aggregation.
🚥 Pre-merge checks | ✅ 1 | ❌ 2

❌ Failed checks (1 warning, 1 inconclusive)

Check name Status Explanation Resolution
Docstring Coverage ⚠️ Warning Docstring coverage is 0.00% which is insufficient. The required threshold is 80.00%. Write docstrings for the functions missing them to satisfy the coverage threshold.
Title check ❓ Inconclusive The title 'feat: better dashboard chart' is vague and overly broad. While it references the dashboard, it doesn't clearly convey the specific changes: defaulting to 30-day view, adding dynamic metric visibility to the email chart, updating theme colors, and adjusting UI styling. The term 'better' is non-descriptive and doesn't communicate the actual improvements made. Consider using a more specific title that highlights primary changes, such as: 'feat: change dashboard default to 30-day view and add interactive email chart metrics' or 'feat: improve dashboard with 30-day default and dynamic metric filtering'.
✅ Passed checks (1 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.

📝 Coding Plan
  • Generate coding plan for human review comments

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

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

@cloudflare-workers-and-pages
Copy link

Deploying usesend with  Cloudflare Pages  Cloudflare Pages

Latest commit: 2655459
Status:⚡️  Build in progress...

View logs

@KMKoushik KMKoushik merged commit 4307670 into main Mar 15, 2026
3 of 5 checks passed
@KMKoushik KMKoushik deleted the better-ui branch March 15, 2026 07:28
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.

1 participant