Modify

Opened 6 months ago

Last modified 3 weeks ago

#17654 new defect

View Autoscrolling During Way Creation

Reported by: MelioraCogito Owned by: team
Priority: normal Milestone:
Component: Core Version: latest
Keywords: Cc:

Description

Can't seem to find the preference which enables/DISABLES view auto-scrolling when drawing ways — centering the screen view to the newly created way point.

This is a damn annoying feature and I want to disable it.

Attachments (2)

17654.patch (2.0 KB) - added by GerdP 6 months ago.
17654-v2.patch (1.0 KB) - added by GerdP 6 months ago.

Download all attachments as: .zip

Change History (14)

comment:1 Changed 6 months ago by GerdP

The "Viewport following" mode is toggled with Ctrl+Shift+F. You can disable that keyboard shortcut in the preferences dialog.

comment:2 Changed 6 months ago by Klumbumbus

Maybe we should remove that shortcut. I already read this issue several times here in the trac or in the forum. It semms it is too easy to hit it instead of Ctrl+F. If someone really wants to use the shortcut he can manually set it again.

comment:3 Changed 6 months ago by GerdP

+1

Changed 6 months ago by GerdP

Attachment: 17654.patch added

comment:4 Changed 6 months ago by GerdP

On the other hand I miss the ability to configure a shortcut for "Edit: Move Node" action. The attached patch implements both changes (no shortcut by default, but possibility to configure one)

comment:5 in reply to:  2 ; Changed 6 months ago by Don-vip

Replying to Klumbumbus:

Maybe we should remove that shortcut.

If we remove it, many people will complain. Maybe we could instead make it clear and display somewhere that "Viewport following is enabled, press Ctrl+Shift+F to disable it".

Changed 6 months ago by GerdP

Attachment: 17654-v2.patch added

comment:6 Changed 6 months ago by GerdP

If we remove it, many people will complain. Maybe we could instead make it clear and display somewhere that "Viewport following is enabled, press Ctrl+Shift+F to disable it".

Yes, maybe better to try this first. The new patch shows the message when the mode is enabled and another when it is disabled.
I see only one problem: When you press the shortcut and react quickly by pressing it again the messages are shown too long. It would be better to replace the message in the moment when the keys are pressed again, but I found no simple way to implement that.

comment:7 Changed 6 months ago by GerdP

In 15023/josm:

see #17654: Allow to configure shortcut for "Move Node" action

comment:8 in reply to:  6 ; Changed 6 months ago by Don-vip

Replying to GerdP:

I see only one problem: When you press the shortcut and react quickly by pressing it again the messages are shown too long. It would be better to replace the message in the moment when the keys are pressed again

This would require an enhancement of the Notification system:

  • add an optional type or id to notifications, allowing to identifying them
  • add a display mode for similar/identical notifications: append or replace
  • adapt NotificationManager accordingly

comment:9 in reply to:  4 Changed 6 months ago by ar2988-os@…

Replying to GerdP:

On the other hand I miss the ability to configure a shortcut for "Edit: Move Node" action.

Several years ago, I discovered a workaround for this problem. Add the button for "Edit: Move Node" to the toolbar. Then you can configure a shortcut. (I use Command+Shift+M on Mac.) Presumably this would work for any other action which has a button, and does not have the possibility to configure a shortcut.

comment:10 in reply to:  8 Changed 4 weeks ago by GerdP

Replying to Don-vip:

Replying to GerdP:

I see only one problem: When you press the shortcut and react quickly by pressing it again the messages are shown too long. It would be better to replace the message in the moment when the keys are pressed again

This would require an enhancement of the Notification system:

  • add an optional type or id to notifications, allowing to identifying them
  • add a display mode for similar/identical notifications: append or replace
  • adapt NotificationManager accordingly

The issue came up again in the German forum. Any idea how much work that would be? I guess we can first apply my simple patch?

comment:11 in reply to:  5 Changed 3 weeks ago by Klumbumbus

Replying to Don-vip:

Replying to Klumbumbus:

Maybe we should remove that shortcut.

If we remove it, many people will complain.

I don't think so, as long as the users can restore the shortcut in the preferences if they really need it.

Displaying a notification doesn't really solve the problem as this can easily be overlooked.

comment:12 Changed 3 weeks ago by Klumbumbus

Ticket #18156 has been marked as a duplicate of this ticket.

Modify Ticket

Change Properties
Set your email in Preferences
Action
as new The owner will remain team.
as The resolution will be set.
to The owner will be changed from team to the specified user.
The owner will change to MelioraCogito
as duplicate The resolution will be set to duplicate.The specified ticket will be cross-referenced with this ticket
The owner will be changed from team to anonymous.

Add Comment


E-mail address and name can be saved in the Preferences.

 
Note: See TracTickets for help on using tickets.