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 click behavior not working for pivot table header cells and tables pivoted (pivoted column cells) #39569

Conversation

JesseSDevaney
Copy link
Contributor

Before

2024-03-04.16-06-31.mp4

After

2024-03-04.16-02-54.mp4

@JesseSDevaney JesseSDevaney added Visualization/Tables Raw, summarized, and pivoted tables backport Automatically create PR on current release branch on merge Reporting/Dashboards/Click Behavior .Team/DashViz Dashboard and Viz team labels Mar 4, 2024
@JesseSDevaney JesseSDevaney requested review from kulyk and a team March 4, 2024 21:09
@JesseSDevaney JesseSDevaney self-assigned this Mar 4, 2024
@metabase-bot metabase-bot bot added the visual Run Percy visual testing label Mar 4, 2024
Copy link

github-actions bot commented Mar 4, 2024

Codenotify: Notifying subscribers in CODENOTIFY files for diff 20211d0...93da304.

Notify File(s)
@alxnddr frontend/src/metabase/visualizations/lib/table.js

Copy link

replay-io bot commented Mar 4, 2024

Status Complete ↗︎
Commit 93da304
Results
⚠️ 4 Flaky
2344 Passed

Copy link
Member

@kulyk kulyk left a comment

Choose a reason for hiding this comment

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

The fix looks good! Could we please add a repro test for this?

@JesseSDevaney JesseSDevaney enabled auto-merge (squash) March 5, 2024 21:43
@JesseSDevaney JesseSDevaney merged commit 5d27229 into master Mar 5, 2024
132 of 133 checks passed
@JesseSDevaney JesseSDevaney deleted the 25203/fix-click-behavior-not-working-for-pivot-tables-and-tables-pivoted branch March 5, 2024 21:59
Copy link

github-actions bot commented Mar 5, 2024

@JesseSDevaney Did you forget to add a milestone to the issue for this PR? When and where should I add a milestone?

github-actions bot pushed a commit that referenced this pull request Mar 5, 2024
…s pivoted (pivoted column cells) (#39569)

* update pivot table header cells clicked format to match the expected format

* fix clicked table pivoted click behavior for the chosen pivot column

* fix failing tests

* add tests
metabase-bot bot added a commit that referenced this pull request Mar 6, 2024
…s pivoted (pivoted column cells) (#39569) (#39663)

* update pivot table header cells clicked format to match the expected format

* fix clicked table pivoted click behavior for the chosen pivot column

* fix failing tests

* add tests

Co-authored-by: Jesse Devaney <22608765+JesseSDevaney@users.noreply.github.com>
@ignacio-mb
Copy link
Contributor

@JesseSDevaney How hard is it to backport this to 48?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
backport Automatically create PR on current release branch on merge Reporting/Dashboards/Click Behavior support/Repro .Team/DashViz Dashboard and Viz team visual Run Percy visual testing Visualization/Tables Raw, summarized, and pivoted tables
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Click Behavior not working for Pivot Table (and Table pivoted) for the non-value columns
3 participants