Ondsel-Development / FreeCAD

This is the official source code of FreeCAD, a free and opensource multiplatform 3D parametric modeler.
https://www.freecad.org
Other
180 stars 7 forks source link

[1.0 Bughunt] If we pad multiple unconnected shapes in a sketch, they cannot be selected properly in the tree. #79

Open raindropsfromsky opened 4 months ago

raindropsfromsky commented 4 months ago

Is there an existing issue for this?

Problem description

  1. Add two unconnected shapes in a single sketch, and close the sketch.
  2. Click on the Pad button.
  3. The task panel offers both shapes. Select one of them.
  4. Click on the Pad button again.
  5. This time the remaining shape is padded directly, without "select a shape" dialog (this is smart!)

But here comes the bug:

  1. The tree shows the two shapes as two different sketches, under different pads! (Ondsel does not retain the original sketch with two unconnected shapes: It splits them in two sketches.)

  2. If I select the first Pad in the tree, nothing is selected in the canvas

  3. If I select the second (tip) pad in the tree, both pads are selected together. (The tree does not let us select the first shape or the second shape individually at all: It is either "both" or "none"!)

Bug- The tree cannot select padded unconnected shapes.zip Bug in selecting padded shapes from common sketch

Full version info

OS: Windows 11 build 22631
Word size of Ondsel: 64-bit
Version: 2024.2.0dev.37608 (Git)
Build type: Release
Branch: flavor
Hash: cbb758c5dc79fd4c016a9cd45a007d3c7ce1a689
Python 3.11.9, Qt 5.15.13, Coin 4.0.2, Vtk 9.2.6, OCC 7.7.2
Locale: English/India (en_IN)
Installed mods: 
  * Ondsel-Lens 2024.3.26.01

Subproject(s) affected?

None

Anything else?

No response

Code of Conduct

raindropsfromsky commented 4 months ago

BTW, I had added the shapes in two different planes within the same sketch. If that is not intended, Ondsel should stop me from doing it with a warning.

Since that did not happen, I am assuming that Ondsel considers that to be a valid step.