Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Set default node size in the shell panel to be 70 px #210

Closed
prrao87 opened this issue Aug 19, 2024 · 2 comments
Closed

Set default node size in the shell panel to be 70 px #210

prrao87 opened this issue Aug 19, 2024 · 2 comments

Comments

@prrao87
Copy link
Member

prrao87 commented Aug 19, 2024

The default node label size of 40 px is just too small, making the truncated text very hard to read. So the one thing I find myself dong every time I create a graph is manually changing the node label size for every node table one by one. I think most users who use Explorer are using it to visualize ~100 nodes/rels or fewer at a time (it's not designed for visualizing a huge number of nodes/rels anyway).

After tons of experimentation across many graphs, I find that the size of 70 px is a good compromise (doesn't look too large or small). This size also corresponds more closely with Neo4j's default node size.

@prrao87 prrao87 changed the title Set default node label size in the shell panel to be 70 Set default node size in the shell panel to be 70 px Aug 19, 2024
@prrao87
Copy link
Member Author

prrao87 commented Aug 19, 2024

Below is an example of displaying names on Person nodes using the default node size of 40. It only shows the first 3 characters, and the rest is truncated.
image

With a node size of 70, I can at least see the first 7-10 characters and it becomes easier to "guess" what the full word might be saying, improving user experience.
image

@prrao87
Copy link
Member Author

prrao87 commented Aug 19, 2024

I did notice that the force-directed layout algorithm behaves a bit differently when the nodes are larger (the clusters don't go as far apart as they did when the nodes were smaller). I presume there's a setting on the force-directed algorithm to help keep the nodes a certain minimum distance apart that's dependent on the size, and that it could be updated if the default node size is also updated?

mewim added a commit that referenced this issue Aug 19, 2024
@mewim mewim closed this as completed Aug 19, 2024
mewim added a commit that referenced this issue Aug 19, 2024
* Fix grammar generation

* Publish next branch as dev build (#151)

* Update Dockerfile to install libatomic1

* Implemented query progress (#152)

* Implemented query progress

* Run linters

* Fix grammar generation

* Publish next branch as dev build (#151)

* Update Dockerfile to install libatomic1

* Implemented query progress

* Run linters

* Publish next branch as dev build (#151)

* Implemented query progress

* Run linters

* Implemented feedback

---------

Co-authored-by: Lint Bot <MSebanc@users.noreply.github.com>
Co-authored-by: Chang Liu <liuc223@gmail.com>

* Update kuzu

* Fix deduplication issue (#172)

* Update kuzu

* Data import UI (#174)

* X

* Update kuzu

* Update kuzu

* Update grammar

* X

* X

* X

* X

* X

* X

* X

* X

* X

* X

* X

* Add dialog when files are initially selected

* X

* X

* Improve parquet format detection

* X

* X

* X

* Minor fix

* Fix custom CSV format issues for previewing and type detection (#175)

* X

* X

* X

* X

* X

* X

* Additional import ui fixes I (#176)

* Refactor ImporterViewFileProcessingModal with ImporterViewProcessingModal

* Fix column ignore functionality

* Additional validation rules for import ui (#177)

* Fix preview header display

* X

* Address column name change for from and to keys

* X

* X

* X

* X

* X

* Refactor ImporterViewValidationModal component for error display

* Refactor planning to use DDL module

* Import UI fix (#178)

* Handle simple import cases independently

* Refactor ImporterViewValidationModal and ImporterViewProcessingModal components

* Improve uploading efficiency with multer dependency (#179)

* Improve uploading efficiency with multer dependency

* X

* Lint

* X

* X

* Fix several issues in planning

* Lint

* Additional fixes (#180)

* Fix table name handling with special chars

* Delete temporary files in Import.js router

* X

* Refactor

* X

* Lint

* Import UI fix (#181)

* Improve wording

* Improve UI consistency

* Update kuzu

* Update grammar

* Update server API routes for read-only mode

* Bump kuzu to 0.5.0

* Address #183 (#194)

* Fix rel validation (#195)

* Enable src and dst key validation for existing rel tables

* Add file remove confirmation hack

* Address 188 (#196)

* Address #188

* Improve error message

* Improve style for very long table name (#199)

* Make column name case insensitive (#200)

* Add in-memory mode support

* Update kuzu

* Update submodule

* Address comments in #201 (#206)

* Fix #203

* Fix #202

* Fix #209 (#211)

* Handle #210

* Update grammar

---------

Co-authored-by: Mattias <81063080+MSebanc@users.noreply.github.com>
Co-authored-by: Lint Bot <MSebanc@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants