Modify

Opened 13 years ago

Closed 10 years ago

Last modified 10 years ago

#6367 closed defect (fixed)

[PATCH]Select last node of way when undoing Follow Line

Reported by: Manchito Owned by: team
Priority: normal Milestone: 14.05
Component: Core Version: latest
Keywords: undo follow line Cc:

Description (last modified by stoecker)

  1. Extend a line with Follow Line
  2. Undo the last action
  3. Try to use Follow Line again
  4. Crash

Attachments (2)

undo_crash.patch (1.7 KB ) - added by Manchito 13 years ago.
JOSM.patch (5.6 KB ) - added by landwirt@… 13 years ago.
patch for undoing Follow Line

Download all attachments as: .zip

Change History (15)

by Manchito, 13 years ago

Attachment: undo_crash.patch added

comment:1 by stoecker, 13 years ago

Summary: Crash when using Follow Line action after undo[patch] Crash when using Follow Line action after undo

I think this patch fixes symptoms instead of real cause.

comment:2 by bastiK, 13 years ago

Undo is a little strange for follow line:

  • Press F a few times
  • Ctrl-Z --> rubber line vanishes
  • Ctrl-Z repeatedly until you reach the first node --> rubber line comes back and is attached to the (still selected) last node

I think it would be best, if undo moved the rubber line back node by node.

comment:3 by stoecker, 13 years ago

Resolution: fixed
Status: newclosed

In [4111/josm]:

fix #6367 - patch by Manchito - fix NPE

comment:4 by stoecker, 13 years ago

@Manchito:
Please no formatting changes in patches.

@bastiK:
That is an normal DrawAction issue, as the last node of a ChangeCommand stays selected. But currently we do not do real selection handling in Undo/Redo. I do not know if it is a good idea to introduce this. It could take a lot of memory and bring little benefits. But maybe we can link the existing SelectionHistory to Undo/Redo, so in case we have a selection history we can use it.

comment:5 by skyper, 13 years ago

Resolution: fixed
Status: closedreopened
Summary: [patch] Crash when using Follow Line action after undoSelect last node of way when undoing Follow Line (WAS:Crash when using Follow Line action after undo)
Type: defectenhancement

Would be nice if the selected node would change to the last node of the way.

Right now I often only miss a "crossing" by one or two nodes and hit undo. If have to manually select the last node to continue drawing which means changing modes twice.

Thanks

comment:6 by landwirt@…, 13 years ago

Summary: Select last node of way when undoing Follow Line (WAS:Crash when using Follow Line action after undo)Select last node of way when undoing Follow Line
Type: enhancementdefect

Letting the undo command select the last node is an expected behavior, doing not so is clearly a defect.

by landwirt@…, 13 years ago

Attachment: JOSM.patch added

patch for undoing Follow Line

comment:7 by landwirt@…, 13 years ago

Summary: Select last node of way when undoing Follow Line[PATCH]Select last node of way when undoing Follow Line

With the new patch josm sets the selection to the last node after undoing Follow Line. Feel free to test it.

comment:8 by stoecker, 12 years ago

Description: modified (diff)

This patch looks like it has been overlooked. Anyone tested it?

comment:9 by skyper, 12 years ago

Any news ?

comment:10 by skyper, 11 years ago

Another six month have passed.

comment:11 by skyper, 11 years ago

ping

This is really annoying and the patch is waiting over 21 month for review now.

comment:12 by simon04, 10 years ago

Resolution: fixed
Status: reopenedclosed

In 7122/josm:

fix #6367 - Select last node of way when undoing Follow line (patch by Landwirt, updated, modified)

comment:13 by Don-vip, 10 years ago

Milestone: 14.05

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.