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

Add node widget embeddable option #374

Open
wants to merge 5 commits into
base: master
Choose a base branch
from

Conversation

zmoth
Copy link
Contributor

@zmoth zmoth commented Feb 20, 2023

close #373

examples/resizable_images

image

@kundezui
Copy link

关闭 #373

examples/resizable_images

图像

Hello, after I successfully installed QtNodes, none of the projects are fully operational in the build/bin folder! Where ". /resizable_images" runs, it cannot load or even compress the images, accompanied by the following error. Do you know what is the reason for this, please?
image

@zmoth
Copy link
Contributor Author

zmoth commented Mar 29, 2023

@kundezui Hi,I have no problem with Windows 10, ubuntu20.04 and Qt>6. Can you share your environment? If the main branch doesn't work either, you may need to open an issue.

@kundezui
Copy link

@zmoth
My environment is Ubuntu 18.04 and Qt5.14.2. When I run . /resizable_images, the following window pops up when I use the right mouse button, and nothing happens when I click on either ImageLoaderModel or ImageShowModeld with the left mouse button. I have also run other projects inside the examples folder, such as calculatord, etc., and again there is no response. I don't know if I'm doing it wrong. You run . /resizable_imagesg project how does it work?
image

@nolankramer
Copy link
Contributor

nolankramer commented Aug 16, 2024

To be clear - it looks like this PR adds a boolean toggle to whether a node's widget is displayed in the node or pops-out to a separate window? @zmoth

If so, it may make sense to add a pop-out indicator in the latter case - to let the user know that interactive controls will pop-out if the node is clicked.

examples/resizable_images/main.cpp Outdated Show resolved Hide resolved
examples/resizable_images/main.cpp Outdated Show resolved Hide resolved
src/DefaultHorizontalNodeGeometry.cpp Outdated Show resolved Hide resolved
src/DefaultHorizontalNodeGeometry.cpp Outdated Show resolved Hide resolved
src/DefaultHorizontalNodeGeometry.cpp Outdated Show resolved Hide resolved
src/DefaultHorizontalNodeGeometry.cpp Outdated Show resolved Hide resolved
src/DefaultHorizontalNodeGeometry.cpp Outdated Show resolved Hide resolved
src/DefaultVerticalNodeGeometry.cpp Outdated Show resolved Hide resolved
src/DefaultVerticalNodeGeometry.cpp Outdated Show resolved Hide resolved
@zmoth
Copy link
Contributor Author

zmoth commented Aug 16, 2024

To be clear - it looks like this PR adds a boolean toggle to whether a node's widget is displayed in the node or pops-out to a separate window? @zmoth

If so, it may make sense to add a pop-out indicator in the latter case - to let the user know that interactive controls will pop-out if the node is clicked.

I'm all for it. Users can use DataFlowGraphicsScene: : nodeHovered themselves to achieve a custom prompt dialog box.

I took a quick look at the DataFlowGraphModel and it doesn't seem to have a setToolTip method to unify this behavior. So I think this tip is still in the future PR to do.

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.

floating widget
4 participants