Skip to content

New checkbox setting to allow short list in tooltip#7133

Open
cdavalos7 wants to merge 3 commits into
tensorflow:masterfrom
cdavalos7:feature/setting-option-tooltip
Open

New checkbox setting to allow short list in tooltip#7133
cdavalos7 wants to merge 3 commits into
tensorflow:masterfrom
cdavalos7:feature/setting-option-tooltip

Conversation

@cdavalos7

@cdavalos7 cdavalos7 commented Jun 23, 2026

Copy link
Copy Markdown
Contributor

Motivation for features / changes

We wrapped a previous functionality to reduce redundancy of data in the tooltip for Scalar Chart into a checkbox so users can choose either to display the tooltip with the full list or short list for exploring data.

Technical description of changes

  • Added a new limitTooltipRows setting. As redux setting like the other controls in panel settings: action → reducer → selector → container → presentational component input.
  • New checkbox added to the metrics settings panel; defaults to unchecked box (full tooltip shown).
  • When on, the scalar card tooltip caps at MAX_TOOLTIP_ITEMS rows and shows a "N additional items" legend for the overflow.
  • Setting persists to localStorage, so it sticks across reloads, same pattern as the other metrics settings.
  • Added unit tests covering the on/off behavior of the tooltip and persistance.

Detailed steps to verify changes work correctly (as executed by you)

  • Ran the full frontend Karma suite bazel test ... tensorboard/webapp:karma_test_chromium-local
  • Verified the new functionality tests
  • Verified the new persistence tests

Screenshots of UI changes (or N/A)

Screenshot 2026-06-23 at 2 59 19 p m Screenshot 2026-06-23 at 2 59 52 p m

@cdavalos7 cdavalos7 marked this pull request as ready for review June 23, 2026 21:01
@cdavalos7 cdavalos7 requested a review from arcra June 23, 2026 21:11
@cdavalos7 cdavalos7 changed the title New checkbox setting to allow short list of items in tooltip New checkbox setting to allow short list in tooltip Jun 23, 2026
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