Skip to content

Fix minor issue of cubes changing color in some scenes when grabbed - #545

Merged
whebertML merged 2 commits into
MixedRealityToolkit:mainfrom
whebertML:issue544
Nov 10, 2023
Merged

Fix minor issue of cubes changing color in some scenes when grabbed#545
whebertML merged 2 commits into
MixedRealityToolkit:mainfrom
whebertML:issue544

Conversation

@whebertML

@whebertML whebertML commented Nov 8, 2023

Copy link
Copy Markdown
Contributor

Fixes: #544

Fixes a minor cosmetic / quality issue with some of the MRTKDevTemplate samples where some of the colored cubes change color once grabbed and released. The intended behavior would be for the cubes to retain their color to maintain the uniqueness of each cube during and after manipulation, even after falling off the edge and being restored.

Just a minor issue with a simple material reference fix.

@AMollis AMollis left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

@whebertML can you please add screen shots of the before and after changes

Also, please add more details to the PR description. In particular I'm curious about the need for this change.

@AMollis AMollis added the Package: Extended Assets The Project extended assets package is impacted by this issue. label Nov 8, 2023
@whebertML

Copy link
Copy Markdown
Contributor Author

Before and after screen shots placed in issue description #544

@AMollis

AMollis commented Nov 9, 2023

Copy link
Copy Markdown
Contributor

/azp run

@azure-pipelines

Copy link
Copy Markdown
Azure Pipelines successfully started running 1 pipeline(s).
@whebertML

Copy link
Copy Markdown
Contributor Author

/azp run

@azure-pipelines

Copy link
Copy Markdown
Azure Pipelines successfully started running 1 pipeline(s).
@whebertML
whebertML merged commit df12510 into MixedRealityToolkit:main Nov 10, 2023
@whebertML
whebertML deleted the issue544 branch May 8, 2024 18:15
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Package: Extended Assets The Project extended assets package is impacted by this issue.

3 participants