Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

fix: Add y-axis label support to chart rendering #113

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

askhogan
Copy link

@askhogan askhogan commented Oct 25, 2024

Add y-axis label support to chart rendering for financial-data-analyst quickstart

Overview:

This PR introduces y-axis label support to the charting library for the financial-data-analyst quickstart. Previously, users could only view data values by hovering over elements like lines, as there was no y-axis label to provide immediate context. The absence of a y-axis label often caused confusion, particularly when comparing multiple data points, and was inconsistent with the y-axis labeling standard on claude.ai, as shown below in Figure 1.

Key Changes:

  • Added y-axis label functionality for all relevant chart types.
  • Ensured that the labels dynamically adjust based on the scale and chart data.
  • Hovering over data points still displays values for detailed examination, but now with corresponding y-axis context.

Why this is important:

Charts with unlabeled axes can mislead users or make interpretation harder, especially when users aren't familiar with the specific dataset. This update makes charts more intuitive and user-friendly by ensuring all axes are clearly labeled.

Financial analyst chart displayed on the Claude homepage October 2024

Figure 1: Screenshot from the https://claude.ai homepage October 2024 displaying a chart with y-axis support.

- Implemented y-axis label functionality to ensure charts display corresponding labels.
- Previously, charts had hover values but no y-axis label, causing confusion when interpreting data.
- This update improves clarity and readability by providing labeled y-axes alongside interactive hover values.
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