clawlter/hermes-brain-web
1
0
Fork 0
Code Issues Pull requests Projects Releases Packages 1 Wiki Activity Actions

fix: refresh mobile neighborhood preview edges #8

Merged
clawlter merged 1 commit from fix/mobile-neighborhood-preview-edges into main 2026-04-15 19:33:30 -04:00
Conversation 1 Commits 1 Files changed 6 +62 -9
clawlter commented 2026-04-15 19:29:51 -04:00
Owner
Copy link

Summary

  • refresh the neighborhood graph on the first mobile tap so the explored node's full neighborhood becomes visible before opening detail
  • strengthen compact graph edge styling with explicit stroke colors, wider strokes, and brighter arrow markers
  • extend mobile regression coverage for preview graph expansion and edge visibility defaults

Testing

  • npm run lint
  • npm run typecheck
  • npm test
  • npm run build
## Summary - refresh the neighborhood graph on the first mobile tap so the explored node's full neighborhood becomes visible before opening detail - strengthen compact graph edge styling with explicit stroke colors, wider strokes, and brighter arrow markers - extend mobile regression coverage for preview graph expansion and edge visibility defaults ## Testing - npm run lint - npm run typecheck - npm test - npm run build
fix(graph): refresh mobile neighborhood preview
All checks were successful
Build and publish container image / Build Docker image and publish package tags (pull_request) Successful in 1m14s
Details
CI / Validate lint, types, tests, and production build (pull_request) Successful in 1m56s
Details
Build and publish container image / Build Docker image and publish package tags (push) Successful in 43s
Details
CI / Validate lint, types, tests, and production build (push) Successful in 1m59s
Details
8ebe9c8618
clawlter commented 2026-04-15 19:33:18 -04:00
Author
Owner
Copy link

Self-review:

  • Reproduced the reported mobile behavior gap with a failing regression test: first tap stayed in graph view but did not rebuild the neighborhood around the tapped node.
  • Fixed it by threading a graph-only preview selection through BrowserShell so the first mobile tap in neighborhood mode refreshes the graph without switching the detail pane.
  • Kept full-graph behavior unchanged: the preview refresh only applies in neighborhood mode.
  • Strengthened compact/mobile edge rendering with explicit stroke colors, wider strokes, and brighter arrow markers so edges remain visible against the dark canvas.
  • Added regression coverage for preview expansion plus stronger mobile edge visibility expectations.

Validation:

  • npm run lint
  • npm run typecheck
  • npm test
  • npm run build
  • PR actions: build-package.yml and ci.yml both green
Self-review: - Reproduced the reported mobile behavior gap with a failing regression test: first tap stayed in graph view but did not rebuild the neighborhood around the tapped node. - Fixed it by threading a graph-only preview selection through BrowserShell so the first mobile tap in neighborhood mode refreshes the graph without switching the detail pane. - Kept full-graph behavior unchanged: the preview refresh only applies in neighborhood mode. - Strengthened compact/mobile edge rendering with explicit stroke colors, wider strokes, and brighter arrow markers so edges remain visible against the dark canvas. - Added regression coverage for preview expansion plus stronger mobile edge visibility expectations. Validation: - npm run lint - npm run typecheck - npm test - npm run build - PR actions: build-package.yml and ci.yml both green
clawlter deleted branch fix/mobile-neighborhood-preview-edges 2026-04-15 19:33:30 -04:00
Sign in to join this conversation.
Reviewers
No reviewers
Labels
Clear labels
No items
No labels
Milestone
Clear milestone
No items
No milestone
Projects
Clear projects
No items
No project
Assignees
Clear assignees
No assignees
1 participant
Notifications
Due date
The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set.

Reference
clawlter/hermes-brain-web!8
No description provided.