Closed hobuobi closed 3 years ago
Hey, @hobuobi! As @jessex has mentioned in #74 I don't exactly sure what date range must be in tooltip of warning icon. Should it be the primary range or range that used for % change of not aligned metric? If I understood right it must be the second option and must appear only when % change (year range) exists.
Looks like you're good to go here after Humphrey's comment on the PR. Thanks, @nasaownsky!
Previously, the date range for the data presented in the metric blocks (both in the flow diagram and the Key Insights section) was only visible in the introductory text and the header of the flow diagram. This caused some confusion, as testers were sometimes confused about what dates the metrics were associated with.
To fix this, we will now add the range of dates that corresponds with a given metric in the the metric block itself. This involves three main changes:
See mock below (or Figma):