Conversation
The basic functionality works, but currently data isn't being displayed
Wait, I think I understand the problem now. The row in the table view doesn't display the correct data for multiple transactions. |
Yes. An edge (and link) can consist of multiple transactions and therefore the edge/link could contain multiple values for a given attribute and so this should probably be reflected in the table view. I'm not exactly sure of the best approach though given as highlighted, too many values may blow out the table more than desired |
I'm thinking the table view should display |
CoreTableView/src/au/gov/asd/tac/constellation/views/tableview/state/TableViewState.java
Show resolved
Hide resolved
…om/constellation-app/constellation into enhancement/table-view-edges-links
|
|
|
This pull request is stale because it has been open for 6 months with no activity. Consider reviewing and taking an action on this pull request. |
|



Prerequisites
Reviewed the checklist
Reviewed feedback from the "Sonar Cloud" bot. Note that you have to wait
for the "CI / Unit Tests") to complete first. Failed Unit tests can be
debugged by adding the label "verbose logging" to the GitHub PR.
Description of the Change
Added functionality to view edges and links in the table view, as well as the existing transactions are vertices.
Alternate Designs
For selecting edges/links on the graph and have that selection be reflected in the table view:
This was achieved iterating over the selected transactions on the graph and selecting edges/links in the table view that those transactions represent. An alternate design would be to add functionality to the selection coed that handles edges/links and use that information directly.
Why Should This Be In Core?
Benefits
Users can now view information about edges and links in table view.
Possible Drawbacks
Aside from the usual risk of changing code, none.
Verification Process
Verified visually and through automated testing.
Applicable Issues
#1859