Skip to content

Reinstate stylebox overrides #144

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

Merged
merged 1 commit into from
Jul 18, 2024
Merged

Reinstate stylebox overrides #144

merged 1 commit into from
Jul 18, 2024

Conversation

wjt
Copy link
Member

@wjt wjt commented Jul 18, 2024

This partially reverts commit 1b6d64b (“Don't recreate resources when editing plugin scenes”).

I incorrectly believed that these were being fully recreated every time the scene is instantiated based on another element's stylebox, but in fact they are being used as a template and it is only the colour that is being adjusted.

See discussion at #138 (comment).

This partially reverts commit 1b6d64b (“Don't
recreate resources when editing plugin scenes”).

I incorrectly believed that these were being fully recreated every time the
scene is instantiated based on another element's stylebox, but in fact they are
being used as a template and it is only the colour that is being adjusted.

See discussion at #138 (comment).
@wjt wjt requested a review from wnbaum July 18, 2024 12:57
Copy link
Contributor

@wnbaum wnbaum left a comment

Choose a reason for hiding this comment

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

Thanks for fixing! Looks good to me.

@wnbaum wnbaum merged commit 5ce3180 into main Jul 18, 2024
2 checks passed
@wnbaum wnbaum deleted the push-wlkynvlkxowm branch July 18, 2024 14:55
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.

2 participants