ux(dashboard): meaningful container names + readable node cards#412
Open
maximbelyakov wants to merge 2 commits into
Open
ux(dashboard): meaningful container names + readable node cards#412maximbelyakov wants to merge 2 commits into
maximbelyakov wants to merge 2 commits into
Conversation
1. Community clusters are now labeled by their member files
("sagas · service · actions +2") instead of "Cluster A/B/C".
2. A single folder covering the whole filtered set (e.g. a Redux slice
folder like src/store/meetingTypes) is kept as ONE container named
after the folder instead of being split into anonymous Louvain
communities. Tests updated + new coverage for both behaviors.
- Node cards: 310px max width (was 220), titles wrap to 2 lines and truncate at 60 chars (was 24), summaries clamp at 3 lines (was 2). NODE_WIDTH/HEIGHT layout constants bumped to match. - Portal edges: stroke opacity 0.2 -> 0.55, width 1.6 - they were nearly invisible on the dark background, making layers look disconnected even when edges existed.
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.
Two UX improvements for the layer-detail view
1. Meaningful container names (
deriveContainers)sagas · service · actions +2.meetingTypes).Tests updated; new test covers member-derived community names. 43/43 dashboard tests pass.
2. Readable node cards and visible portal edges
UpsertInboxMeetingTypeHandler.java) were unreadable. Cards widened to 310px, titles wrap to 2 lines / 60 chars, summaries clamp at 3 lines.NODE_WIDTH/NODE_HEIGHTlayout constants bumped to match.rgba(...,0.2)are nearly invisible on the dark theme — bumped to 0.55 / 1.6px. (Pairs with fix(dashboard): edges attached to containers are never rendered (missing Handles) + portal edge re-routing on expand #411: even after container edges render, users couldn't see them.)The style commit is opinionated — happy to gate it behind a setting if preferred.
Found while building a graph for a real Java + TypeScript codebase (49 files, 171 nodes).