[Graph-] Added appearance panel (Text fade distance & Node font size) #1555
Conversation
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
Contributor
There was a problem hiding this comment.
Pull request overview
This PR adds an "Appearance" panel to the graph visualization with two new controls: "Text Fade" (distance) and "Node Font Size" (multiplier). The Text Fade slider adjusts when node labels become visible based on zoom level, while the Node Font Size slider controls the font size multiplier for node labels. The PR also includes VS Code configuration support for the font size multiplier to persist across sessions.
Key changes:
- Added an "Appearance" folder to the dat.GUI with sliders for text fade distance (0-5) and node font size multiplier (0.5-3)
- Implemented VS Code configuration persistence for
nodeFontSizeMultiplier - Removed restrictive CSS width constraints on dat.GUI controls
Reviewed changes
Copilot reviewed 4 out of 4 changed files in this pull request and generated 4 comments.
| File | Description |
|---|---|
| packages/foam-vscode/static/dataviz/graph.js | Added appearance panel with text fade and font size controls; updated font size calculation and opacity domain; added message handler for font size multiplier updates |
| packages/foam-vscode/static/dataviz/graph.css | Removed fixed width constraints on dat.GUI controls to accommodate new sliders |
| packages/foam-vscode/src/features/panels/dataviz.ts | Added configuration change listener and initialization for nodeFontSizeMultiplier setting |
| packages/foam-vscode/package.json | Added foam.graph.nodeFontSizeMultiplier configuration property with default value of 1 |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
…sociated calls in datavis.ts
…dler by storing a reference to the controller and then calling its update method
riccardoferretti
approved these changes
Nov 28, 2025
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
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.
Adds 2 sliders to change the text fade distance so that nodes can be seen when zoomed out, this is useful especially in conjunction with the forces panel which might result in changing of the graph size.
Text Fade Distance
Font Size
