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

Icon colours, rendering optimizations, folders #88

Merged
merged 8 commits into from
Jun 20, 2024

Conversation

MoffKalast
Copy link
Owner

@MoffKalast MoffKalast commented Jun 11, 2024

Icon colours

Certain some widgets can be set to different drawing colours, it should be practical to reflect those colours in the icons as well. The current approach of using hue-rotate doesn't seem to be completely accurate so some colours don't look consistent with the actual render colour, but it's close enough.

colours

Rendering

Turns out that now that most visualizers transform the messages they get to the fixed frame, we can skip redrawing them on every tf frame change (since fixed_frame rarely changes) for a massive rendering speed boost.

Folders

A prototype implementation for folders that have to be created separately via the add widget menu for now, but that has a neat side effect that any widgets inside them get loaded later than the folder, which makes infinite folder recursion possible without any specific handling. Currently showing the first two widget icons inside.

image

image

@MoffKalast MoffKalast changed the title Icon colours Icon colours, rendering optimizations Jun 15, 2024
@MoffKalast MoffKalast changed the title Icon colours, rendering optimizations Icon colours, rendering optimizations, folders Jun 15, 2024
@MoffKalast MoffKalast marked this pull request as ready for review June 15, 2024 21:03
@MoffKalast MoffKalast merged commit 332684b into noetic-devel Jun 20, 2024
@MoffKalast MoffKalast deleted the noetic-feature branch July 14, 2024 09:25
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.

1 participant