[pull requested] Sketcher: New Features: Symmetry, Copy and 2D Array transformations

[jmaustpc]

... I still miss one for the mirroring of a sketch...

How about this one based on the existing "merge sketch" icon...

Sketcher_MirrorSketch.svg

(11.97 KiB) Downloaded 136 times

Sketcher_MirrorSketch_22px.png (1.04 KiB) Viewed 1362 times

[triplus]

Then probably you discovered a bug in normal symmetry that was not discovered yet... for the next coding slot...

Yes. You did discover a bug, but neither on the code from yesterday, nor in the normal symmetry code, but in the addition of a list of geometries.

I introduced this bug while improving the Sketcher geometry creation earlier this year (speed up creation of complex objects and construction toggle as a modal tool). This is the message from the commit fixing the bug, if you want to know more:

Code: Select all

commit d9596f896165bd3a3e293294f259f30c810dded7                                                                                                                                                                    
Author: Abdullah Tahiri <abdullah.tahiri.yo@gmail.com>                                                                                                                                                             
Date:   Sat Aug 15 12:15:46 2015 +0200                                                                                                                                                                             
                                                                                                                                                                                                                   
    Sketcher: Bug fix addGeometry list creation and construction lines
    
    ==================================================================
    
    A recent improvement was to allow to create an object (several geometric elements) as construction lines
    by providing an additional parameter as true (that defaults to false).
    
    This introduced a bug, that when a list of geometries mixing construction and normal lines was added (with the parameter as false),
    construction lines would be converted to normal lines.
    
    This fixes the bug.

The fix is in the branch now.

Am I finished or are there outstanding issues?

Great.

P.S. "Copy" probably is the tool user can use for "cloning" sketcher geometry.

[abdullah]

Pull requested:
viewtopic.php?f=17&t=12178

[cox]

How about this one based on the existing "merge sketch" icon...

Realy nice icon

[abdullah]

Assuming that somebody does a nice job and not thanking him each time is not right.

My apologies: Thanks Jim for the icons, they are really nice