Allow for fractional Movement
ClosedPublic

Authored by rizzitello on Sep 2 2018, 8:09 PM.

Diff Detail

Repository
R232 AtCore
Branch
fractionalMovement
Lint
No Linters Available
Unit
No Unit Test Coverage
Build Status
Buildable 2445
Build 2463: arc lint + arc unit
rizzitello requested review of this revision.Sep 2 2018, 8:09 PM
rizzitello created this revision.
rizzitello edited the summary of this revision. (Show Details)Sep 2 2018, 8:18 PM
rizzitello added a project: Atelier: AtCore.
rizzitello added a subscriber: Atelier: AtCore.
rizzitello updated this revision to Diff 40873.Sep 2 2018, 8:21 PM
  • slight cleanup
rizzitello edited the summary of this revision. (Show Details)Sep 2 2018, 8:26 PM

If you want to put a scale, we should remove the intern circles and only let one xyz control without 0.1,1,10 buttons, only 1x.

If you want to put a scale, we should remove the intern circles and only let one xyz control without 0.1,1,10 buttons, only 1x.

This is a way to allow fractional movement on the pie controller. do u think i should just change the defaults to 0.1 1 10 and drop the scale completely ?

If you want to put a scale, we should remove the intern circles and only let one xyz control without 0.1,1,10 buttons, only 1x.

This is a way to allow fractional movement on the pie controller. do u think i should just change the defaults to 0.1 1 10 and drop the scale completely ?

I'm just saying that a scale for a pie (that has scales) does not make sense to me. I accept the scale control if the xy pie and z control has only one positive and negative directional button for each axis.

rizzitello updated this revision to Diff 40877.Sep 2 2018, 10:50 PM
rizzitello edited the summary of this revision. (Show Details)
  • Simplify
rizzitello edited the summary of this revision. (Show Details)Sep 2 2018, 10:51 PM
rizzitello updated this revision to Diff 41017.Sep 5 2018, 4:42 AM
  • remove axis control changes
rizzitello edited the summary of this revision. (Show Details)Sep 5 2018, 4:42 AM
rizzitello edited the summary of this revision. (Show Details)
  • rebase
laysrodrigues added inline comments.Sep 15 2018, 2:15 PM
src/core/atcore.cpp
623

What does this mean?
Can you clarify for me?

rizzitello updated this revision to Diff 41702.Sep 15 2018, 2:58 PM
  • Add comment on the number format
rizzitello marked an inline comment as done.Sep 15 2018, 2:59 PM
laysrodrigues accepted this revision.Sep 15 2018, 3:41 PM
This revision is now accepted and ready to land.Sep 15 2018, 3:41 PM
patrickelectric accepted this revision.Sep 18 2018, 2:20 PM
rizzitello closed this revision.Sep 18 2018, 7:52 PM