DiscussionsIssue archiveOCCT:Visualization

Archived issue #0031644

Visualization - flat skin for AIS_Manipulator presentation

Open CASCADEOCCT:Visualizationfeedback17 public notes

Search issues

Description

Currently AIS_Manipulator supports only one presentation with thick arrows.
In some applications another skin might be preferred - with thin arrow-less axes.

Steps to reproduce

Not required

Public activity

17 archived notes

Participants are labeled by their role within this record.

01Commenter 1
Branch [archived branch] has been created by Commenter 2.

[revision removed]


Detailed log of new commits:

Author: Commenter 2
Date: Fri Sep 25 17:53:09 2020 +0300

    0031644: Visualization - flat skin for AIS_Manipulator presentation
    
    Created flat skin for AIS_Manipulator presentation
    Added "-flat" option to vmanipulator command
    
    v3d/manipulator/flat: test case added
02Commenter 1
Branch [archived branch] has been updated by Commenter 2.

[revision removed]


Detailed log of new commits:

Author: Commenter 2
Date: Tue Sep 29 15:36:04 2020 +0300

    remarks from kgv

03Commenter 1
Branch [archived branch] has been created by Commenter 2.

[revision removed]


Detailed log of new commits:

Author: Commenter 2
Date: Fri Sep 25 17:53:09 2020 +0300

    0031644: Visualization - flat skin for AIS_Manipulator presentation
    
    Created flat skin for AIS_Manipulator presentation
    Added "-flat" option to vmanipulator command
    
    v3d/manipulator/flat: test case added
04Commenter 2
05Commenter 2
The patch CR31644_1 is ready to review
06Author
 
+  enum AIS_SkinMode
+  {
+    AIS_SM_Shaded,
+    AIS_SM_Flat
+  };

Please don't shortcut names in enum values and remove AIS_ prefix for enumeration declared in scope of a class.
ManipulatorSkin_Shaded, ManipulatorSkin_Flat.

+               const AIS_SkinMode theSkinMode = AIS_SM_Shaded,

I'm not sure it makes sense defining a default value for an internal class method.

+++ b/tests/v3d/manipulator/flat

Please attach screenshots to the bug.
07Commenter 1
Branch [archived branch] has been updated by Commenter 2.

[revision removed]


Detailed log of new commits:

Author: Commenter 2
Date: Wed Sep 30 10:21:41 2020 +0300

    remarks from kgv

08Commenter 1
Branch [archived branch] has been updated forcibly by Commenter 2.

[revision removed]
09Author
I also don't see a "flat" skin for rotation mode of manipulator on screenshots.
10Commenter 1
Branch [archived branch] has been updated forcibly by Commenter 2.

[revision removed]
11Commenter 1
Branch [archived branch] has been updated forcibly by Commenter 2.

[revision removed]
12Author
Marina, could you please put a description of this issue to Bugtracker?

- A textual description of new skin - presentation and interaction aspects implemented by the patch;
- A list of the features present in referred CADRays application but not (yet) implemented;
- Mechanics implemented in the patch for handling camera rotation-dependent updates.
13Commenter 1
Branch [archived branch] has been created by Commenter 2.

[revision removed]


Detailed log of new commits:

Author: Commenter 2
Date: Fri Sep 25 15:53:09 2020 +0100

    0031644: Visualization - flat skin for AIS_Manipulator presentation
    
    Created flat skin for AIS_Manipulator presentation
    Added "-flat" option to vmanipulator command
    
    Added functionality to transform the manipulator depending on the camera rotation.
    
    Added local transformation for Graphic3d_Group.
    
    Added RecomputeTransformation method to PrsMgr_PresentableObject.
    When overloading this method, you can set a transformation for an object depending on the rotation of the camera.
    
    v3d/manipulator/flat: test case added
14Commenter 1
Branch [archived branch] has been updated forcibly by Commenter 2.

[revision removed]
15Commenter 1
Branch [archived branch] has been updated by Commenter 2.

[revision removed]


Detailed log of new commits:

Author: Commenter 2
Date: Fri Jan 31 17:59:53 2025 +0000

    Fix formatting

Author: Commenter 2
Date: Fri Jan 31 17:57:42 2025 +0000

    Fix warnings
16Commenter 1
Branch [archived branch] has been updated by Commenter 2.

[revision removed]


Detailed log of new commits:

Author: Commenter 2
Date: Fri Jan 31 23:19:19 2025 +0000

    Fix sensitivity

Author: Commenter 2
Date: Fri Jan 31 23:08:17 2025 +0000

    Fix color
17Commenter 1
Branch [archived branch] has been updated forcibly by Commenter 2.

[revision removed]

Related records