[PATCH] [ATLAS] Multi-Selection object rotation

[michael]

Multi-Selection object rotation. Right-click rotation (as with single objects). Just rough calculate a center point of the selection and use that as a center for rotation.

[trompetin17]

this ticket depend on #1163 (Buildind Constraint)

Here is my video about implementation

​http://youtu.be/2MQoelad0Is

[Niek]

As said through IRC, it would be nice when (as a third option maybe) the selected entities would keep their relative rotation to each other (as an extension to the rotation where the entities would move around)

[Stan]

CTRL + Rotation or something ?

[trompetin17]

Some code style fix

[trompetin17]

this is the video about how I implemented ​http://youtu.be/Z3ynovnPwhs

• shift for translate without rotation itself
• ctrl(command in osx) for translate with rotation itself

Replying to niektb:

As said through IRC, it would be nice when (as a third option maybe) the selected entities would keep their relative rotation to each other (as an extension to the rotation where the entities would move around)

[Niek]

Very good! That's indeed how I had it in mind!

[historic_bruno]

In RotateObjectsFromCenterPoint, m_centerPoint should be camel case m_CenterPoint and you don't need to initialize it (the default constructor is 0,0,0 but this line actually causes a build error in MSVC). Similarly m_angleInitialRotation should be capitalized.

I get these other build errors in VS2010:

1>  TransformObject.cpp
1>..\..\..\source\tools\atlas\AtlasUI\ScenarioEditor\Tools\TransformObject.cpp(555): error C2864: 'TransformObject::sRotating::rotateFromCenterPoint' : only static const integral data members can be initialized within a class
1>..\..\..\source\tools\atlas\AtlasUI\ScenarioEditor\Tools\TransformObject.cpp(560): error C2039: 'mUpdateVisualSelectedObjects' : is not a member of 'AtlasMessage'
1>..\..\..\source\tools\atlas\AtlasUI\ScenarioEditor\Tools\TransformObject.cpp(560): error C2065: 'mUpdateVisualSelectedObjects' : undeclared identifier
1>..\..\..\source\tools\atlas\AtlasUI\ScenarioEditor\Tools\TransformObject.cpp(560): error C2059: syntax error : ')'
1>..\..\..\source\tools\atlas\AtlasUI\ScenarioEditor\Tools\TransformObject.cpp(560): error C2039: 'mUpdateVisualSelectedObjects' : is not a member of 'AtlasMessage'
1>..\..\..\source\tools\atlas\AtlasUI\ScenarioEditor\Tools\TransformObject.cpp(560): error C2039: 'mUpdateVisualSelectedObjects' : is not a member of 'AtlasMessage'

[trompetin17]

Done. This tickets was depending #1163, but I didnt update this one with last changes over these ticket.

Replying to historic_bruno:

In RotateObjectsFromCenterPoint, m_centerPoint should be camel case m_CenterPoint and you don't need to initialize it (the default constructor is 0,0,0 but this line actually causes a build error in MSVC). Similarly m_angleInitialRotation should be capitalized.

I get these other build errors in VS2010:

1>  TransformObject.cpp
1>..\..\..\source\tools\atlas\AtlasUI\ScenarioEditor\Tools\TransformObject.cpp(555): error C2864: 'TransformObject::sRotating::rotateFromCenterPoint' : only static const integral data members can be initialized within a class
1>..\..\..\source\tools\atlas\AtlasUI\ScenarioEditor\Tools\TransformObject.cpp(560): error C2039: 'mUpdateVisualSelectedObjects' : is not a member of 'AtlasMessage'
1>..\..\..\source\tools\atlas\AtlasUI\ScenarioEditor\Tools\TransformObject.cpp(560): error C2065: 'mUpdateVisualSelectedObjects' : undeclared identifier
1>..\..\..\source\tools\atlas\AtlasUI\ScenarioEditor\Tools\TransformObject.cpp(560): error C2059: syntax error : ')'
1>..\..\..\source\tools\atlas\AtlasUI\ScenarioEditor\Tools\TransformObject.cpp(560): error C2039: 'mUpdateVisualSelectedObjects' : is not a member of 'AtlasMessage'
1>..\..\..\source\tools\atlas\AtlasUI\ScenarioEditor\Tools\TransformObject.cpp(560): error C2039: 'mUpdateVisualSelectedObjects' : is not a member of 'AtlasMessage'

[historic_bruno]

Seems to work well, but I would make the current Ctrl behavior the default, that should be the most common and useful type of rotation. Then make the other two functions use the modifier keys.

[trompetin17]

Change Ctrl behavior

[trompetin17]

Done. Replying to historic_bruno:

Seems to work well, but I would make the current Ctrl behavior the default, that should be the most common and useful type of rotation. Then make the other two functions use the modifier keys.

[Itms]

In 16275:

Add various rotation modes for multiple object selections (individual rotation, global rotation with or without individual rotation).

Patch by trompetin17, fixes #1213

[Itms]

Thanks a lot for your work! :D