feat(calm-hub): fix missed /api/calm refactor in tests#5
Closed
willosborne wants to merge 1 commit into
Closed
Conversation
willosborne
pushed a commit
that referenced
this pull request
Jul 6, 2026
…n timeline (finos#2763) * feat(calm-hub-ui): refine diagram minimap, node legibility and version timeline Item-detail diagram refinements (redesign problems #4/#5/finos#6). - MiniMap: fixed 132x84 panel with a light mask, faint node chips and a #2563EB viewport rect, padded clear of the timeline and toggleable for dense graphs (fixes the oversized solid-blue overflow) - Node legibility: fitView gains a minZoom floor so fit-to-view no longer shrinks node labels below readability on dense graphs - Version timeline: relabelled "Browse versions" with explanatory text and a mono version pill; active moment re-tokenised to the brand blue; single- version items collapse the strip. Cross-view re-render + compare preserved * fix(calm-hub-ui): address Copilot feedback on the diagram phase - Restore window.matchMedia after the mobile ArchitectureGraph test so the mobile viewport can't leak into later tests (matches the SegmentedTypeTabs convention). - Correct the minimap mask comment (the alpha suffix is 40, not AA). - Add ml-auto to the timeline version pill so it is right-aligned as its docstring describes. * fix(calm-hub-ui): align timeline dots and drop no-op pill margin - Move the sparkline track line + progress fill to the dots' vertical centre (top 18->15) so the dots sit on the track for both active and inactive sizes, restoring the alignment the removed per-dot compensation used to provide. - Drop the no-op ml-auto on the version pill (TimelineHeader is content-sized in both callers, so it had no slack) and correct the docstring to describe the actual inline layout.
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
1 participant
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.
Description
Type of Change
Affected Components
cli/)calm/)calm-ai/)calm-hub/)calm-hub-ui/)calm-server/)calm-widgets/)docs/)shared/)calm-plugins/vscode/)Commit Message Format ✅
Testing
Checklist