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

Adding a simple API to Nodify #36

Draft
wants to merge 35 commits into
base: master
Choose a base branch
from
Draft

Adding a simple API to Nodify #36

wants to merge 35 commits into from

Conversation

miroiu
Copy link
Owner

@miroiu miroiu commented Jul 15, 2022

📝 Description of the Change

This PR aims to reduce the boilerplate and simplify the way node editors are built. There will be a new library called Nodifier which comes with view model abstractions for the existing controls and binds those to the views using Stylet.

There will also be an example built on top of Nodifier and possibly a reusable toolkit.

🐛 Possible Drawbacks

Dependency on Stylet for the new library.
The new library may be too restrictive (not being able to customize Nodify the same way you would by using it directly).
The codebase is getting larger.

Notes

A separate NuGet package will be generated for Nodifier.
I may consider moving the new library into its own repo.

Please let me know if you have any ideas for a better library name.

Things to consider:

  • exposing editor API
  • graph serialization
  • graph auto layout
  • collaborative editing
  • undo/redo

@miroiu miroiu self-assigned this Jul 15, 2022
@miroiu miroiu added enhancement New feature or enhancement help wanted Extra attention is needed application Example application labels Jul 15, 2022
@miroiu miroiu force-pushed the feature/new-blueprint branch 5 times, most recently from 8ab3e90 to 4e7d407 Compare September 5, 2022 21:21
@miroiu miroiu force-pushed the feature/new-blueprint branch from 1e7bdfc to 8e37ac5 Compare September 29, 2022 19:43
@miroiu miroiu changed the title Adding a simple API interface to Nodify Adding a simple API to Nodify Oct 3, 2022
@ankitsatapara07
Copy link

Hi @miroiu
Is there a plan to merge this PR?
Actually I am looking for this to start my journey in node based application.
You are doing a awesome job, I really really appreciate it. :)

@miroiu
Copy link
Owner Author

miroiu commented Aug 23, 2023

Hi @ankitsatapara07 , I'm not actively working on this anymore, although, I may reconsider it in the future. I know it would be really useful to have something like this implemented, and I want it to be generic enough to cover a wide range of use cases. That turns out to be difficult, and there's also a tradeoff on the ease of use. I guess you can use it as an inspiration for now.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
application Example application enhancement New feature or enhancement help wanted Extra attention is needed
Projects
None yet
2 participants