Skip to content

feat(attribute-breakdown): Open in expanded modal#106559

Merged
nsdeschenes merged 16 commits intomasterfrom
nd/exp-714/feat-attribute-breakdown-open-in-modal
Jan 22, 2026
Merged

feat(attribute-breakdown): Open in expanded modal#106559
nsdeschenes merged 16 commits intomasterfrom
nd/exp-714/feat-attribute-breakdown-open-in-modal

Conversation

@nsdeschenes
Copy link
Contributor

@nsdeschenes nsdeschenes commented Jan 20, 2026

This PR adds the functionality for users to be able to expand their attribute breakdown charts (single, and comparison mode) into a modal with the chart and table data so they can inspect the data more closely.

Ticket: EXP-714

Screenshots:

Single mode:
Screenshot 2026-01-20 at 10 42 34

Comparison mode:
Screenshot 2026-01-20 at 10 40 30

@linear
Copy link

linear bot commented Jan 20, 2026

@nsdeschenes
Copy link
Contributor Author

@sentry review

@nsdeschenes
Copy link
Contributor Author

@cursor review

@github-actions github-actions bot added the Scope: Frontend Automatically applied to PRs that change frontend components label Jan 20, 2026
cursor[bot]

This comment was marked as outdated.

@nsdeschenes nsdeschenes force-pushed the nd/exp-714/feat-attribute-breakdown-open-in-modal branch from 0125def to 76d3c2e Compare January 20, 2026 12:11
@nsdeschenes
Copy link
Contributor Author

@sentry review

@nsdeschenes
Copy link
Contributor Author

@cursor review

cursor[bot]

This comment was marked as outdated.

@nsdeschenes nsdeschenes force-pushed the nd/exp-714/feat-attribute-breakdown-open-in-modal branch from 57aebcb to 0b6af0e Compare January 22, 2026 17:50
Copy link
Contributor

@cursor cursor bot left a comment

Choose a reason for hiding this comment

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

Cursor Bugbot has reviewed your changes and found 2 potential issues.

Bugbot Autofix is OFF. To automatically fix reported issues with Cloud Agents, enable Autofix in the Cursor dashboard.

const singleTableData = distributionToTableData(
attributeDistribution.values,
cohortCount
);
Copy link
Contributor

Choose a reason for hiding this comment

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

Chart and table show inconsistent data in modal

Medium Severity

The modal chart and table display different data when attributes have more than 40 unique values. distributionToSeriesData and cohortsToSeriesData apply .slice(0, CHART_MAX_SERIES_LENGTH) to limit chart data to 40 items, but distributionToTableData and cohortsToTableData do not apply this limit. This causes the chart to show only the top 40 items while the table shows all items, which is confusing for users trying to inspect their data.

Additional Locations (1)

Fix in Cursor Fix in Web

@nsdeschenes nsdeschenes merged commit e7846a5 into master Jan 22, 2026
53 checks passed
@nsdeschenes nsdeschenes deleted the nd/exp-714/feat-attribute-breakdown-open-in-modal branch January 22, 2026 18:17
@github-actions github-actions bot locked and limited conversation to collaborators Feb 7, 2026
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

Scope: Frontend Automatically applied to PRs that change frontend components

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants