Skip to content

Conversation

@KaranPradhan266
Copy link

Closes: #5704

Tooltip Filter Button Default Behavior

Changes

  • The tooltip filter button now defaults to visible by setting showFilterButton = true in
    src/components/tooltip/Marker.tsx.
  • Timeline tooltips explicitly opt out of the filter button by passing
    showFilterButton={false} in:
    • src/components/timeline/Markers.tsx
    • src/components/timeline/TrackNetwork.tsx
    • src/components/timeline/TrackCustomMarkerGraph.tsx
  • Marker charts continue to display the filter button via the existing prop in
    src/components/marker-chart/Canvas.tsx.

Tests

  • Updated Jest snapshots to reflect the new tooltip behavior and minor download-size text changes:
    • src/test/components/__snapshots__/TrackCustomMarker.test.tsx.snap
    • src/test/components/__snapshots__/TrackNetwork.test.tsx.snap
    • src/test/components/__snapshots__/MenuButtons.test.tsx.snap

@KaranPradhan266
Copy link
Author

@canova would love a review when you get a chance. Thanks!

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.

Filter button in the marker hover panel isn't useful when hovering over the marker when displayed in a track

1 participant