Skip to content

[BUGFIX] TimeSeriesChart: fix tooltip when timezone changes#619

Merged
jgbernalp merged 1 commit intoperses:mainfrom
tremes:timeseries-tooltip-fix
Apr 8, 2026
Merged

[BUGFIX] TimeSeriesChart: fix tooltip when timezone changes#619
jgbernalp merged 1 commit intoperses:mainfrom
tremes:timeseries-tooltip-fix

Conversation

@tremes
Copy link
Copy Markdown
Contributor

@tremes tremes commented Apr 2, 2026

Description

This is a suggestion fix for perses/perses#4001

Screenshots

Checklist

  • Pull request has a descriptive title and context useful to a reviewer.
  • Pull request title follows the [<catalog_entry>] <commit message> naming convention using one of the
    following catalog_entry values: FEATURE, ENHANCEMENT, BUGFIX, BREAKINGCHANGE, DOC,IGNORE.
  • All commits have DCO signoffs.

UI Changes

  • Changes that impact the UI include screenshots and/or screencasts of the relevant changes.
  • Code follows the UI guidelines.

@tremes tremes requested a review from a team as a code owner April 2, 2026 12:06
@tremes tremes requested review from Gladorme and removed request for a team April 2, 2026 12:06
@tremes tremes force-pushed the timeseries-tooltip-fix branch from 40e3419 to 5d552c5 Compare April 2, 2026 12:10
@jgbernalp
Copy link
Copy Markdown
Contributor

LGTM @tremes, it makes more sense to change the formatting rather than the actual timestamp. Can you fix the lint issues?

@AntoineThebaud AntoineThebaud changed the title [BUGFIX] fix tooltip of timeseries chart when timezone changed [BUGFIX] TimeSeriesChart: fix tooltip when timezone changes Apr 7, 2026
@jgbernalp jgbernalp self-assigned this Apr 8, 2026
@tremes tremes force-pushed the timeseries-tooltip-fix branch from 5e30c6e to fcc3ff1 Compare April 8, 2026 06:45
Signed-off-by: Tomáš Remeš <tremes@redhat.com>
@tremes tremes force-pushed the timeseries-tooltip-fix branch from fcc3ff1 to f62ff21 Compare April 8, 2026 06:54
@jgbernalp jgbernalp added this pull request to the merge queue Apr 8, 2026
Merged via the queue into perses:main with commit 538dc38 Apr 8, 2026
16 checks passed
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.

2 participants