Repository navigation
Viewer fixes from the Stage 1 review: finding counts, connection hover, direct-only node hover - #14
Merged
Merged
Conversation
…r, direct-only node hover The owner reported two defects while reading pilot diagrams in VS Code and asked for a narrower node hover: - Phase and group badges and Outline rows count each finding once, so they agree with the toolbar and status bar; a collapsed group also counts the findings of the connections it hides. - Hovering a connection or its severity marker lists the connection's findings in the hover card; the whole marker opens it at any zoom, and the connection Inspector lists the findings too. A group's card lists the findings its badge counts. - Hovering a card lights and animates only its direct connections; focus mode (F) keeps the full lineage. Trunks a hover or focus mode opened fold again when it ends, cards a hover dims stay clickable, and turning focus mode off under the pointer restores the hover. Checked by local jsdom tests (webview 122, extension 266) and scripts/e2e.sh; not yet checked in live VS Code. Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
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.
Viewer fixes the owner asked for while reviewing the Stage 1 pilot diagrams in VS Code. This PR holds no pilot records, so any merge method is fine.
What changes
webview/srcchanges are rebuilt intowebview/dist/andvscode-extension/media/. The CHANGELOG has an Unreleased section, anddocs/STATUS.mdhas a paragraph. The version is unchanged.docs/FEATURES_FLOW_AND_SCOPE.mdstill describes the lineage hover, but it has the historical-record banner, so it is left as written.Checks run locally
Node v26.4.0, Python 3.13.15
sh scripts/e2e.sh --skip-npm-install:E2E OK: 15 exercised gatesThese are jsdom and unit tests only. The viewer has not been re-checked in live VS Code.
🤖 Generated with Claude Code