Skip to content

Add setBackgroundColor support #503

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.

Sign up for GitHub

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

Jump to bottom
Closed
g-abilio wants to merge 30 commits into from
Closed

Add setBackgroundColor support #503

g-abilio wants to merge 30 commits into from

Conversation

@g-abilio
Copy link
Contributor

@g-abilio g-abilio commented Dec 4, 2025 •
edited by tatatupi
Loading

Type of change

  • New feature

Description

  • add background color helper to NodeStyle
  • expose NodeDelegateModel::setBackgroundColor

Testing

  • Qt version tested:
  • Existing tests still pass

tatatupi and others added 30 commits June 20, 2025 15:26
@tatatupi
Updates fork from upstream
2edfb0b 
Integrate upstream
@tatatupi
wip: adds NodeValidationState info to NodeDelegateModel
988ce45
@tatatupi
makes the nodeObject red in case of invalid state and adds a tooltip …
1c84750 
…for error msg
@tatatupi
adds warning state and adapts calculator example
3007c23
@tatatupi
adds validation icon and adapts calculation example
129d414
@g-abilio
core improvements to develop node processing status
e0b0c4a
@g-abilio
first commit on the creation of a processing status example
e002f5b
@tatatupi
fixes nodeprocessingstatus cast
985b638
@g-abilio
creation of random gen example, and fix of icon color
3f55e35
@tatatupi
Connect delegate UI update signal
50c6bec
@g-abilio
fix random number node dynamic
1c09ecf
@g-abilio
Merge pull request #5 from fabns-nano/codex/add-signal-for-ui-updates…
50ce760 
…-from-nodedelegatemodel

Add UI update signal
@g-abilio
clean up test code in multiplication node
f2c120f
@hudsonmiranda291
Add per-node background color API
965eb15
@tatatupi
Merge branch 'paceholder:master' into master
97d48d5
@tatatupi
Merge branch 'master' of https://github.com/paceholder/nodeeditor
89ac7f2
@tatatupi
solves conflict
37c3bf8
@tatatupi
revert unnecessary changes in multiplication model
b23ecc3
@tatatupi
revert unnecessary changes
0a35f76
@g-abilio
solve icon size and refactor NodeProcessingStatus code
a9be092
@g-abilio
update and merge new code
5530b72
@g-abilio
remove duplicate code
03e311a
@g-abilio
add space to better organize processing status in node display
a87ceb2
@g-abilio
remove processing value default value
ae3715b
@g-abilio
fix bugs in node processing status
eedc666
@g-abilio
add Q_DECLARE_METATYPE to solve linux build problems
1298605
@g-abilio
declaring metatype in the correct place
47d948f
@g-abilio
Merge pull request #6 from fabns-nano/node_processing_status
21b55ac 
Adds NodeProcessingStatus
@g-abilio
fix undocommands module inclusion
4f1b23c
@g-abilio
solve conflicts
87e219b
tatatupi
tatatupi requested changes Dec 5, 2025
View reviewed changes
Copy link
Collaborator

@tatatupi tatatupi left a comment

Choose a reason for hiding this comment

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

There are changes from other branches that should not be here.

@g-abilio
Copy link
Contributor Author

g-abilio commented Dec 8, 2025

This PR is being closed and a new version of the PR containing only the setter changes is being created.

@g-abilio g-abilio closed this Dec 8, 2025
@g-abilio
Copy link
Contributor Author

g-abilio commented Dec 8, 2025

New PR: #504

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@tatatupi tatatupi tatatupi requested changes

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

3 participants

@g-abilio @tatatupi @hudsonmiranda291