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

feat #1217: Open side panel when new node is added #1563

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

ibek
Copy link
Contributor

@ibek ibek commented Oct 11, 2024

No description provided.

Copy link

sonarcloud bot commented Oct 11, 2024

Copy link

codecov bot commented Oct 11, 2024

Codecov Report

Attention: Patch coverage is 13.33333% with 13 lines in your changes missing coverage. Please review.

Please upload report for BASE (main@f0ac182). Learn more about missing BASE report.

Files with missing lines Patch % Lines
...s/Visualization/Custom/ContextMenu/ItemAddStep.tsx 13.33% 10 Missing and 3 partials ⚠️
Additional details and impacted files
@@           Coverage Diff           @@
##             main    #1563   +/-   ##
=======================================
  Coverage        ?   79.87%           
  Complexity      ?      272           
=======================================
  Files           ?      276           
  Lines           ?     7845           
  Branches        ?     1539           
=======================================
  Hits            ?     6266           
  Misses          ?     1467           
  Partials        ?      112           

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

const result = props.vizNode.data.path
? findVisualizationNode(controller.getGraph(), props.vizNode.data.path)
: null;
controller.fireEvent(SELECTION_EVENT, [result?.getNextNode()?.id]);
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

There are a few cases in which getNextNode doesn't yield the desired node:

  1. When adding a step in from
  2. Preprending a step from any given node

I'll try to check if we can get the new node id directly.

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks, I will wait for your changes in lordrip@3d668cd before adding more capabilities

Copy link
Member

@apupier apupier left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

  • several cases are not working:
    • When adding a step after the first element (it is called "add step" in this case), the config panel is closing
      NotSelectingNewNodeWithAddStep
    • When prepending a step, the select node is not the new node but the one on which we prepended
      selectingCurrentInsteadOfCreatedWhenPrepending

which I guess are linked to comment here #1563 (comment)

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

Successfully merging this pull request may close these issues.

3 participants