-
-
Notifications
You must be signed in to change notification settings - Fork 3.8k
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
Revert: "Sketcher: Joint Line and Polyline in a command group." #14118
Conversation
This reverts commit cb0a2d2.
I'm not in favor of a plain revert of this PR. If the current implementation is not in your personal favor, we should discuss a preference otherwise (although I'm not supporting a preference for every individual tool). |
I’m also against this plain revert. There was no agreement on this (quite opposite, most users want to keep the change) so the PR is out of place. |
The changes made to #13509 are not about personal preferences but are an obvious regression in usability because in certain situations it requires a lot of extra work to achieve what the user wants. Just watch the videos and read the explanation created by berberic2 and you will see why. And in the past whenever it was about personal preferences things have always been kept flexible by allowing them to change a switch in the parameter editor. Not so with the mentioned PR where this possibility is not offered. |
That's why I'd propose a preference instead of reverting everything, same as with the dimension constraint tool. |
Yes, a preference is definitely a good idea here. Maybe in the future preferences like this will be replaced by customizable toolbars but for now, one more preference shouldn’t hurt. FreeCAD doesn’t have many settings compared with other CAD software. If only it was possible to search in preferences… |
I worry "It has been decided by the DWG" is an argument for everything. It isn't according current Contribution Process. DWG should have created a GH issue about the original change so a broader discussion may happen. Merged regressions may have been caught there before bothering users. It also tends to prove that DWG as it is today doesn't include enough real FC users to take only sensible decisions.
There was no agreement on the original PR, so not a valid argument.
I'd be in favor of plainly reverting and informing the original PR author that it should be resubmitted as a new PR including a user preference to enable the icon merge. |
The obvious regression isn't clearly outlined by anyone here. Merely stated as an 'obvious regression'. Clear delineation of the nature of the regression should be defined. If the argument is that it takes an extra click to create a single line with the polyline tool, then that is not a particularly great point being made. Previous points made which were relatively valid critiques of this change: PolyLine doesn't have OVP implemented. Too many discrete draw modes (six). Otherwise, while drawing lines, a single end to end line involves three mouse clicks instead of two. Since it has been so controversial (with 3 or 4 vocal users), I agree that adding a simple preference to disable this change seems to be the best course of action until the PolyLine tool can have its deficiencies addressed. |
Be serious please. What is unclear here: #13509 (comment) ? |
That's no change that was introduced with the original PR, it's the current behavior of the polyline tool. Both tools are still available. |
Guys, really looks like you're not using FC at all... 😕 The original PR is a PITA for one frequently using both Line and Polyline tools for what they each are targeting. As it is now, it is like if Line and Arc would have been merged in same icon. This is a major user experience crap. Is it what DWG is aiming at ? |
I'd also say that this is problematic and changes that affect user experience so much should not be pushed. Let's back off, preserve this as optional behavior, personally I'd lean towards it being opt-in not opt-out. When polyline would be improved to a point where the line tool is not needed we can think about revisiting it. Also, no body is deciding about anything, things are opinionated nothing more. |
I'm just not interested in petty infighting and circular arguments over this change. Revert it and once polyline has improvements similar to what I proposed in the original PR's comments we need to revisit this change with a closer eye towards just removing the line tool completely and renaming polyline. |
#13509 (comment)
Only true in the case of meaningful change. Which it isn't.
Someone that would both frequently use Polyline and Line -- with automatic coincidence on both endpoints --. Looks pretty basic, but not covered in DWG yet. Maybe professional there only machine/print/... cylinders. 😕
Same thing as assessing risks only based on likelihood completely discarding severity. Will fail soon.
Sounds wise. |
If it prevents 99% of human beings from dying of thirst, yes. Your provided anecdote is a poor example, neither your example or my response really apply here.
PolyLine does add automatic coincidence/point-on-object on both endpoints, just like the line tool. This is why I keep asking for an itemized list of lost functionality/regressions. From my perspective, I've seen the following complaints filed:
Do you have more points to add to the list so I can track them as well? |
I don't understand how the hell the DWG can be pushing hours to defend a change that will save 1 click a day, and then take decisions that add so much ones...
Did you look at the video I pointed above ? The main problem is that when clicking arc endpoint with polyline, it goes into tangent mode and you have to enter 'M' and again to get the correct mode. This is highly inefficient. |
As with most things in life a balance has to be made between clicks, options, efficiency, what we expect users to mentally track, etc etc. Sometimes a click should be conserved, other times concessions need to be made for greater impact in other facets of UX.
I missed the link to the video, I had looked at the other two links you provided. This also goes back to my problem with the many discrete modes of polyline. Forced tangential constraint from the endpoint of an arc is indeed a serious problem, switching tools modes for a user like this is yet another problem. That is a bit different that 'end to end lines don't get auto constraints' that was a broader statement for a specific identifiable issue which I was already aware of, but now better understand what you were trying to communicate. |
Doing a plain revert as it is done in this PR is not necessary. If you want to revert you can just modify the toolbar instead of removing the command group and all. All you need is
As said before I am not against the reversal for now as the polyline tool has apparantly some way to go before being good enough to warrant the removal of the line tool. However I will say that the position of those in favor of reversal are very conservative, anti-change and not constructive. Because they fail to see that there is value in the proposal of merging these tools : It promotes the use of polyline which reduce the probability of open wires. The best course of action imo is to improve the polyline tool to solve the annoying case described in the video. The tangent mode after an arc is not necessary. It could just be an auto-constraint when close enough to tangent. |
No one has said something like this – ever. Read the comments. The problem is not the merging of the tools, the problem is that there is no progress on the tools at all, only the UI has changed with a degregation in usability. Just a quote:
|
Indeed I stand corrected then. I read in diagonal and stopped at openBrain comment
But he indeed says 'As it is now' |
Surely not. Even absolutely opposite. This is probably the most essential rule of the merging process. Being easy to revert is what allows to easier merge PR too. The process is fully lamous if not all rules apply at same level. 😉 |
As a long time CAD professional I really like the change, I never use normal line and really like how it works now. |
What change?! There is absolutely no change in the way the tools work, yet. |
@berberic2 Instead of this reversal (which again I'm not against at this time), I can offer the following :
This would solve the situation described in your video. |
You can just exit the feature if you want a single line. You just have to get used to it. |
I wonder whether you have ever watched the video. This discussion is not about to quit the polyline command but about the extra work that must be done because of automatically added but unwanted constraints.
A single line + a semi-circle. |
Your proposals seems OK but:
To me, the sensible decision is to revert the PR (or at least its effects in the toolbar, as you proposed earlier). |
It seems that this PR can be closed. |
Superseded by #14299 |
This reverts commit cb0a2d2.
the commit 6bb7775, groups the line- and polyline-tool in a submenu, saving a tiny bit a space in the toolbar but clearly reduces the usability of the most-used sketcher tools considerably.
The main argument for this commit is, that sometimes in the future, the polyline-tool will be improved and replace the line-tool. None of this improvement can be seen now and there does not seem to be a clear understanding of how this new polyline-tool should work at all.
So, at the moment it is a degredation. The UI-change should be made when the polyline/line-improvement has happend.
For details see the discussion on
#13509
and
https://forum.freecad.org/viewtopic.php?p=759200 [german]