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.

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

Node icon does not ignore overridden output context #8442

Closed
somebody1234 opened this issue Dec 4, 2023 · 7 comments
Closed

Node icon does not ignore overridden output context #8442

somebody1234 opened this issue Dec 4, 2023 · 7 comments
Assignees
@somebody1234
Labels
--bug Type: bug -gui d-intermediate Difficulty: some prior knowledge required p-medium Should be completed in the next few sprints
Milestone
Beta Release

Comments

@somebody1234
Copy link
Contributor

somebody1234 commented Dec 4, 2023 •
edited
Loading

Context not overridden:
image

Context overridden:
image

The logo changes back to the default (fallback) logo.
@somebody1234 somebody1234 added d-hard Difficulty: significant prior knowledge required d-intermediate Difficulty: some prior knowledge required p-medium Should be completed in the next few sprints --bug Type: bug -gui labels Dec 4, 2023
@github-project-automation github-project-automation bot moved this to ❓New in Issues Board Dec 4, 2023
@farmaazon
Copy link
Contributor

I don't understand this issue. I see both nodes has same node icon, and everything seems right.

@somebody1234
Copy link
Contributor Author

sorry, my bad - something messed up... i'll re-upload a correct screenshot

@somebody1234
Copy link
Contributor Author

@farmaazon screenshot updated.

@somebody1234
Copy link
Contributor Author

note that the PR on which this is an issue isn't merged yet, so it's not yet actionable

@farmaazon farmaazon moved this from ❓New to 📤 Backlog in Issues Board Dec 5, 2023
@somebody1234 somebody1234 removed the d-hard Difficulty: significant prior knowledge required label Dec 8, 2023
@somebody1234
Copy link
Contributor Author

uh oh... this is still an issue

@somebody1234
Copy link
Contributor Author

the node color seems to also not ignore an overridden output context.

@AdRiley AdRiley added this to the Beta Release milestone Mar 7, 2024
@farmaazon
Copy link
Contributor

Fixed by #9188

@github-project-automation github-project-automation bot moved this from 📤 Backlog to 🟢 Accepted in Issues Board Mar 18, 2024
@farmaazon farmaazon moved this from 🟢 Accepted to 🗄️ Archived in Issues Board Mar 19, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@somebody1234 somebody1234

Labels
--bug Type: bug -gui d-intermediate Difficulty: some prior knowledge required p-medium Should be completed in the next few sprints
Projects
Issues Board
Archived in project
Milestone
Beta Release
Development

No branches or pull requests
3 participants
@AdRiley @farmaazon @somebody1234