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

[TD] show balloon dialog on double-clicking #4272

Merged
merged 1 commit into from Jan 11, 2021

Conversation

donovaly
Copy link
Member

When adding GD&T statements as described in
https://wiki.freecadweb.org/TechDraw_Geometric_dimensioning_and_tolerancing#Geometrical_Tolerancing
you will quickly get a lot of balloons and it is then very tiring to be forced to first select a balloon with a single-click in the drawing, then go to the model tree and there double-click to get the dialog.

With this PR you can directly double-click in the drawing to open the dialog directly.

This PR also removes unused includes from the dialog code.

when adding GD&T statements as described in https://wiki.freecadweb.org/TechDraw_Geometric_dimensioning_and_tolerancing
You will quickly get a lot of balloons and it is then very tiring to be forced to first select a balloon with a single-click in the drawing, then go to the model tree and there double-click to get the dialog.

This PR also removes unused includes from the dialog code.
Copy link
Member

@chennes chennes left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This compiles, runs, and implements the feature as described. It's a very nice usability addition to TD balloons. I do note, however, that it only works when double-clicking on the bubble, and not on the leader line or end element. I don't think that's a deal-breaker for this PR, just an idea for future refinement.

@donovaly
Copy link
Member Author

I do note, however, that it only works when double-clicking on the bubble, and not on the leader line or end element.

This is a general design decision by TechDraw. A balloon can only be selected in the drawing by clicking on its body. I guess this is because a balloon needn't to have a leader line, symbol etc. You see the same behavior also for other TD elements, or example a dimensions can only be selected when you click on its value, not onto its lines.

I think the TD behavior is reasonable since selecting by clicking onto a thin line is not useful at all since you need several clicks until you hit it.

@yorikvanhavre yorikvanhavre merged commit c331669 into FreeCAD:master Jan 11, 2021
@yorikvanhavre
Copy link
Member

IIRC I am the one who started this "partial" select thing with Draft dimensions, and it was then copied to TD, but indeed today I don't really see why we couldn't activate an element by clicking just anywhere on it. At some point we could try to mend that. But yeah, no need to solve all of earth's problems at once ;)

@donovaly
Copy link
Member Author

but indeed today I don't really see why we couldn't activate an element by clicking just anywhere on it.

For TD one often has dimension and leader line crossing each other. Therefore it is in my opinion better only to select via the body of the element (labels, text or whatever). Then it is clear what element should be addressed.

@yorikvanhavre
Copy link
Member

Good point, but this is the same problem anywhere in the CAD world where lines cross... Indeed it's easy to select wrongly.

@donovaly donovaly deleted the TD-show-balloon branch January 13, 2021 02:59
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants