Review UX of Metrics report MR widget
Everyone can contribute. Help move this issue forward while earning points, leveling up and collecting rewards.
Release notes
Problem to solve
As a developer reviewing an MR, I want a visual indicator of which custom metrics got better/worse, so I can quickly identify what to address in the MR that improves the experience for the user.
Additional Details
We should clarify the text that represents each metric in the metrics report MR widget to better reflect how the value of the metric has changed in the base report versus the head report for the MR.
[...] we see two values:
1552393 (1552382). There is barely any context and I wonder if it is worthwhile to change the wording to something like.1552393 (Previous value: 1552382)
Potential complexity: If we want to do math with these values (for something like 1552393 (+11)), it could get messy because we have no guarantee that they have consistent formatting - they are just string values read from a .txt file artifact.
Proposal
- If the data is a string/other add the label
Previous Valueto the previous value - If the data is a number do the math to show the change so New Value, change (+/-)
The following discussion from !55217 (merged) should be addressed:
-
@leipert started a discussion: (+1 comment) @mfluharty As a side note and out of scope for this MR. I think the UX is a little confusing, so maybe we could create a follow-up from this thread and involve the correct UX designer.
- Why does every line start with what looks like a huge
-Icon. Is this supposed to be a spark line over time? If yes, then all is well. - In the first line we see two values:
1552393 (1552382). There is barely any context and I wonder if it is worthwhile to change the wording to something like.1552393 (Previous value: 1552382)
- Why does every line start with what looks like a huge
