kibana icon indicating copy to clipboard operation
kibana copied to clipboard

[APM] Display the previous period timestamp on the chart tooltip when comparison is enabled

Open gbamparop opened this issue 2 years ago • 6 comments

Summary

Currently, only the current period timestamp is displayed on the chart tooltip. It would be great to include the previous period as well as it is not always clear, especially when Period before is selected.

image

gbamparop avatar Mar 09 '22 16:03 gbamparop

Pinging @elastic/datavis (Team:DataVis)

elasticmachine avatar Mar 22 '22 09:03 elasticmachine

@gbamparop we are currently working on a tooltip redesign that will cover this requirement. Can you please explain a bit more the desired outcome, in particular, I'd like to know:

  • how are you offsetting the data (by calendar (start of the week vs start of the week before) or by fixed date (7 days vs 14 days ago).
  • Do you need to show the exact date/time of the previous period, or just what "changed" between the current and the previous( for example just the different date if the hour is the same)

markov00 avatar Mar 22 '22 09:03 markov00

@markov00 that's great to hear!

The offset depends on the selection of the comparison option.

It can be a day / week before or a custom period which is based on the time difference between the current start and end dates (we subtract this difference from both dates to calculate the previous period). In the last case, the year / day / time might be different than the current one so in my opinion it would be good to display the full date of the current / previous period.

cc @formgeist

gbamparop avatar Mar 22 '22 11:03 gbamparop

cc @gvnmagni

markov00 avatar Mar 22 '22 14:03 markov00

@markov00 Should this issue be transferred to https://github.com/elastic/elastic-charts or is there already a related issue we can follow and link to from here?

sorenlouv avatar Jun 21 '22 14:06 sorenlouv

This is the issue to follow is actually this one https://github.com/elastic/elastic-charts/issues/919

markov00 avatar Jun 21 '22 14:06 markov00

Pinging @elastic/kibana-visualizations @elastic/kibana-visualizations-external (Team:Visualizations)

elasticmachine avatar Nov 04 '22 07:11 elasticmachine

This is no longer blocked by Elastic Charts (https://github.com/elastic/elastic-charts/issues/615#issuecomment-1410089411).

@gbamparop It's probably too late to include this in 8.8 but we could add it as a stretch goal or include it in 8.9. WDYT?

sorenlouv avatar Jan 31 '23 15:01 sorenlouv

That's good news! Let's include it in 8.9.

gbamparop avatar Feb 02 '23 10:02 gbamparop