Modify

Opened 13 years ago

Closed 12 years ago

#8470 closed enhancement (fixed)

Extrude mode: allow dragginng nodes along adjacent segments

Reported by: akks Owned by: team
Priority: minor Milestone:
Component: Core Version: latest
Keywords: extrude Cc:

Description (last modified by akks)

Sorry for committing in stabilization phase, the refactoring of extrude action was unfinished (and potentially buggy) and while finishing it some new behavior was added.
Now nodes in extrude mode can be dragged along the adjacent segments (see the picture)

I leave the ticket to detect the bugs or usability issues before tested is released.

before:
after:

Attachments (2)

p1.png (10.5 KB ) - added by akks 13 years ago.
p2.png (4.3 KB ) - added by akks 13 years ago.

Download all attachments as: .zip

Change History (17)

by akks, 13 years ago

Attachment: p1.png added

by akks, 13 years ago

Attachment: p2.png added

comment:1 by akks, 13 years ago

Description: modified (diff)

comment:2 by akks, 13 years ago

In 5746/josm:

see #8470: finish refactoring of Extrude mode, allow single node dragging along adjacent segments

in reply to:  2 ; comment:3 by skyper, 13 years ago

Replying to akks:

In 5746/josm:

see #8470: finish refactoring of Extrude mode, allow single node dragging along adjacent segments

Thanks for this feature. Think it would be even more obvious if the mouse pointer would change before action to indicate the different mode similar to selection mode. Maybe simply highlighting the node will work, too.

in reply to:  3 comment:4 by akks, 13 years ago

Replying to skyper:

Thanks for this feature. Think it would be even more obvious if the mouse pointer would change before action to indicate the different mode similar to selection mode. Maybe simply highlighting the node will work, too.

I agree, but currently the extrude mode does not have mouse move listening. It is very good for speed and CPU usage, but no highlighting or cursor changes are possible when dragging was not started. I do not know a good way to add it without risking bugs and performance loss...

comment:5 by anonymous, 13 years ago

I find annoying, that now I must zoom in real close because otherwise I risk dragging nodes, when in extrude mode I mostly want to drag segments. Moving one node is not really "extruding", so could you make node-along-segment need a modifier key, please?

comment:6 by akks, 13 years ago

If you would ask 1.5 months ago, we would easily add modifier.
But now, after 3 tested releases, people get used to the functionality, so only some configuration option or menu is possible (like with angle snapping).

Last edited 13 years ago by akks (previous) (diff)

comment:7 by stoecker, 13 years ago

Hmm, I doubt Extrude mode is that popular...

So changes are still possible (when better than previous solution).

in reply to:  5 ; comment:8 by akks, 12 years ago

Replying to anonymous:

I find annoying, that now I must zoom in real close because otherwise I risk dragging nodes, when in extrude mode I mostly want to drag segments. Moving one node is not really "extruding", so could you make node-along-segment need a modifier key, please?

Do we need at least an option for that?

in reply to:  8 comment:9 by skyper, 12 years ago

Replying to stoecker:

Hmm, I doubt Extrude mode is that popular...

So changes are still possible (when better than previous solution).

Yeah, it seems to get popular ( were some discussion on talk-de). So better change fast.

Replying to akks:

Replying to anonymous:

I find annoying, that now I must zoom in real close because otherwise I risk dragging nodes, when in extrude mode I mostly want to drag segments. Moving one node is not really "extruding", so could you make node-along-segment need a modifier key, please?

Do we need at least an option for that?

+1

comment:10 by akks, 12 years ago

In 6037/josm:

see #8470: extrude mode option - extrude.drag-nodes-without-ctrl

comment:11 by akks, 12 years ago

Ctrl-only-dragging option is introduced, but disabled by default. Extruding "random" segment near the node was not restored intentionally (do we need it?) We possibly need a checkbox in preferences for this too

Let us decide, should this be enabled by default?
For me it is convenient to drag nodes without ctrl (extruding is in fact somerthing like moving one segment). Your opinion?

comment:12 by anonymous, 12 years ago

Yes, it should. Node-dragging is really annoying, because (in my workflow at least) extrude tool is only used for extruding ways: e.g. parts of building. --Zverik

comment:13 by Felis Pimeja, 12 years ago

+1 to --Zverik

comment:14 by akks, 12 years ago

Let it be default then...

comment:15 by akks, 12 years ago

Resolution: fixed
Status: newclosed

In 6050/josm:

fix #8470: drag nodes along segments in Extrude mode only when Ctrl pressed
use extrude.drag-nodes-without-ctrl=true if you prefer dragging without Ctrl

Modify Ticket

Change Properties
Set your email in Preferences
Action
as closed The owner will remain team.
as The resolution will be set.
The resolution will be deleted. Next status will be 'reopened'.

Add Comment


E-mail address and name can be saved in the Preferences .
 
Note: See TracTickets for help on using tickets.