fix(hud): show axis labels next to metric values in Point Analysis panel - #30
Merged
Conversation
Point Analysis panel previously showed a header line ("Metrics (x, y, z)")
followed by a 3-column grid of bare values — the axis name only
appeared in that header or a hover tooltip, making it easy to misread
which value belonged to which axis.
Each metric now gets its own label-left/value-right row, matching the
existing UID and Classification row style. Rows are generated by
mapping over [x, y, z] rather than three hand-written blocks, with
border logic based on array length rather than a hardcoded index.
Contributor
There was a problem hiding this comment.
Pull request overview
Updates the Point Analysis HUD so axis names are shown directly next to their corresponding metric values, reducing the chance of misreading x/y/z values and keeping labels consistent with the active dataset’s axis mappings.
Changes:
- Replace the 3-column “bare values” metrics grid with three label-left/value-right rows.
- Render metric rows by mapping over
["x", "y", "z"]and reading bothaxisLabels[axis]andhoveredPoint[axis].
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
| correct even if this array's size ever changes, | ||
| rather than depending on someone remembering to | ||
| update a hardcoded index alongside it. */} | ||
| <span className="text-white/40 text-[10px] block mb-1"> |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Closes #21
The Point Analysis panel's metric section previously showed a header line ("Metrics (x, y, z)") followed by a 3-column grid of bare values — the axis name only appeared in that header or a hover tooltip, making it easy to misread which value belonged to which axis.
Each metric now gets its own label-left/value-right row, matching the existing UID and Classification row style already used elsewhere in the same panel.
Testing:
Screenshot: