Dev #162
Merged
Dev #162
Conversation
This file contains 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
…table, and if it is treat it as '' so that it won't crash.
Note, since a blank label is being used to delete nodes, I didn't want to mess with the validation yet, so this should be improved later. But for the short term, a missing label is now saved as '' and I also check for missing labels in the node table sorting so that if one got through it'll be fixed. Not ideal, but safer for short-term.
…hould not be valid.
…or other) extension to save the table as a csv, it has what it needs to be easily converted to Gephi or other.
…in the NodeTable every time that edit was clicked.
We need a better solution so that Edit is hidden in the NodeTable but keeping track of the state was outside my capabilities.
… Gephi or similar
… panel. Note, also changed edge color to be easier to read in nodes (more of a yellow now). Also added a fix to id's not showing up hidden in tables (that was also added to the prior commit).
…f the template, and use .2 for default if it is missing.
…filters I am not sure why degrees are re-calculated within the node and edge table, so this might be wrong, or that might just be an oversight. Worth reviewing.
…them without having to re-open the filters panel.
…e those degrees instead of constantly re-calculating them in the node table ... not sure why it was doing that so make sure nothing broke.
…es) to template at top level. Both use the old default if they are missing for backwards compatibility, but ... ToDo: update template default to include these and defaultTransparency
… will load it so that you can follow the links. Todo: get this to scroll up so that it is obvious.
… move up to avoid confusion.
…ted, the edge editor closes as we navigate to the new node.
… yet (just enums)
… defunct edgetable.
… default colors for selected / searched nodes, and c) have default transparencies
…fo to match the logged in approach (using a div instead of form group) to fix that.
jdanish
approved these changes
Feb 4, 2021
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.
No description provided.