Skip to content

Introduce ScaleMaintainType - #976

Merged
whitetigerhelix merged 13 commits into
MixedRealityToolkit:mainfrom
nayanatharap:user/npalanivel/BoundsHandleScaleMaintainTypes
Mar 5, 2025
Merged

Introduce ScaleMaintainType#976
whitetigerhelix merged 13 commits into
MixedRealityToolkit:mainfrom
nayanatharap:user/npalanivel/BoundsHandleScaleMaintainTypes

Conversation

@nayanatharap

@nayanatharap nayanatharap commented Jan 29, 2025

Copy link
Copy Markdown
Contributor

Fixes #979 - Introduce different options for scaling bounds handles. The options include:

  • GlobalSize - Maintains global size (previously stored as a separate bool property):
    GlobalSize

  • FixedScale - Sets scale based on initial parent scale:
    FixedScale

  • Advanced - Adjusts scale to retain same size
    Advanced

Introduce different options for scaling bounds handles. The options include:
- Maintaining global size (previously stored as a separate bool property)
- Setting scale based on initial parent scale
- Adjusting scale to retain same size
@nayanatharap
nayanatharap requested a review from a team as a code owner January 29, 2025 22:18
@nayanatharap
nayanatharap marked this pull request as draft January 29, 2025 23:47
@nayanatharap
nayanatharap force-pushed the user/npalanivel/BoundsHandleScaleMaintainTypes branch from bd3abcf to eb48694 Compare January 31, 2025 21:57
@keveleigh
keveleigh self-requested a review January 31, 2025 23:39
@nayanatharap
nayanatharap marked this pull request as ready for review February 1, 2025 01:00
@nayanatharap
nayanatharap force-pushed the user/npalanivel/BoundsHandleScaleMaintainTypes branch from 471b186 to 1f360c6 Compare February 7, 2025 23:25
Comment thread org.mixedrealitytoolkit.spatialmanipulation/package.json Outdated

@keveleigh keveleigh 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.

Approved, pending my last question!

@whitetigerhelix
whitetigerhelix merged commit 8bd3432 into MixedRealityToolkit:main Mar 5, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

3 participants