Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Sketcher/Polyline create null-length line if double-clicking end of tangent arc #13924

Open
2 tasks done
0penBrain opened this issue May 9, 2024 · 9 comments
Open
2 tasks done
Labels
DWG Input or discussion from Design Working Group needed Feature FR for improvements or new features UI/UX WB Sketcher Related to the Sketcher Workbench

Comments

@0penBrain
Copy link
Contributor

Is there an existing issue for this?

  • I have searched the existing issues

Problem description

Steps to reproduce:

  • Open an empty sketch
  • Select Polyline tool
  • Draw a first line then press M key till "tangent arc"
  • Move mouse a bit to get an arc and double-click => ⚠️ In better case a null-length (actually something in the 10^-6 mm range) is created and a coincident constraint added, in worst case the line is really null and it breaks solver "Fail to converge"

Would be good to prevent double-click acceptation in this case

Full version info

OS: Ubuntu 22.04.4 LTS (KDE/plasma)
Word size of FreeCAD: 64-bit
Version: 0.22.0dev.37243 (Git)
Build type: Debug
Branch: master
Hash: 0095e52466c79c2135fce860a341554f3f51156c
Python 3.10.12, Qt 5.15.3, Coin 4.0.0, Vtk 7.1.1, OCC 7.6.3
Locale: English/United States (en_US)

Subproject(s) affected?

Sketcher

Anything else?

No response

Code of Conduct

  • I agree to follow this project's Code of Conduct
@maxwxyz
Copy link
Collaborator

maxwxyz commented May 9, 2024

It's not just with arc, with all modes.
What should happen on double click? End the tool?

@maxwxyz maxwxyz added UI/UX Feature FR for improvements or new features WB Sketcher Related to the Sketcher Workbench DWG Input or discussion from Design Working Group needed labels May 9, 2024
@pierreporte
Copy link

I would say end the polyline and exit tool. Currently, exiting the tool needs double right click so there would be some sort of consistency.

I don’t remember how other programs handle this though. I should check Alibre but I won’t have access to Catia until Monday.

@0penBrain
Copy link
Contributor Author

It's not just with arc, with all modes.

I don't have it if for example I keep "broken line" mode. In this case an error is generated.

What should happen on double click? End the tool?

No, just discard creation of a line. Eventually generate same error as in default "broken line" mode.

@maxwxyz
Copy link
Collaborator

maxwxyz commented May 9, 2024

IMO double click should end the tool as well.

@0penBrain
Copy link
Contributor Author

IMO double click should end the tool as well.

This isn't how other tools behave, and would be a PITA if you didn't do it on purpose. :)

@maxwxyz
Copy link
Collaborator

maxwxyz commented May 9, 2024

how do other apps handle this? In Inkscape, GIMP, PS when creating paths or drawing pixels a double click is just used as input like currently in FreeCAD.

@0penBrain
Copy link
Contributor Author

how do other apps handle this? In Inkscape, GIMP, PS when creating paths or drawing pixels a double click is just used as input like currently in FreeCAD.

I don't care about other apps. I care about consistency among the Sketcher tools. 😉
Just try with the 2nd point of a Slot and you'll see.

@maxwxyz
Copy link
Collaborator

maxwxyz commented May 9, 2024

so you could go on: it creates a new slot if you double click on the third click of a slot or the second click on a polygon.

@0penBrain
Copy link
Contributor Author

Yes, and it starts a new one (both cases). Which is OK because in this case, the 2 clicks can be interpreted as valid points and don't create malformed geometry. 😉

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
DWG Input or discussion from Design Working Group needed Feature FR for improvements or new features UI/UX WB Sketcher Related to the Sketcher Workbench
Projects
None yet
Development

No branches or pull requests

3 participants