Skip to content

Draft: Draft_Layer new Material handling#13949

Merged
Roy-043 merged 1 commit intoFreeCAD:mainfrom
Roy-043:Draft-Draft_Layer-new-Material-handling
May 28, 2024
Merged

Draft: Draft_Layer new Material handling#13949
Roy-043 merged 1 commit intoFreeCAD:mainfrom
Roy-043:Draft-Draft_Layer-new-Material-handling

Conversation

@Roy-043
Copy link
Contributor

@Roy-043 Roy-043 commented May 11, 2024

Discussion: #13657.

This PR adds the ShapeAppearance view property to Draft Layers.

The DefaultShapeColor is used for the DiffuseColor of the ShapeAppearance material. Other material properties are not based on the preferences when a layer is created.

The old ShapeColor and Transparency properties remain (ShapeColor as a hidden property) and are kept in synch with the ShapeAppearance. This is consistent with how ShapeAppearance is implemented in Core.

The gui_layers.py and make_layer.py files do not have to be changed. They manipulate the vobj via the mentioned old properties.

Discussion: FreeCAD#13657.

This PR adds  the ShapeAppearance view property to Draft Layers.

The DefaultShapeColor is used for the DiffuseColor of the ShapeAppearance material. Other material properties are not based on the preferences when a layer is created.

The old ShapeColor and Transparency properties remain (ShapeColor as a hidden property) and are kept in synch with the ShapeAppearance. This is consistent with how ShapeAppearance is implemented in Core.

The gui_layers.py and make_layer.py files do not have to be changed. They manipulate the vobj via the mentioned old properties.
@Roy-043 Roy-043 added the Mod: Draft Related to the Draft Workbench label May 11, 2024
@Roy-043 Roy-043 marked this pull request as draft May 12, 2024 07:50
@maxwxyz
Copy link
Collaborator

maxwxyz commented May 15, 2024

@Roy-043 just a heads up for the upcoming feature freeze:
If you want this draft PR to be considered for a version 1.0 release, it should be marked ready for review by Monday, 2024-06-03 at the latest. (further details in the 1.0 feature freeze forum announcement).

@Roy-043 Roy-043 marked this pull request as ready for review May 21, 2024 14:21
@chennes chennes requested a review from davesrocketshop May 27, 2024 16:05
@Roy-043 Roy-043 merged commit 8bcbeae into FreeCAD:main May 28, 2024
@Roy-043 Roy-043 deleted the Draft-Draft_Layer-new-Material-handling branch May 28, 2024 07:45
@Roy-043
Copy link
Contributor Author

Roy-043 commented May 28, 2024

Oops I missed the review request.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Mod: Draft Related to the Draft Workbench

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants