Modify

Opened 13 years ago

Closed 13 years ago

Last modified 13 years ago

#6840 closed defect (fixed)

shortcut for add node changed

Reported by: dieterdreist Owned by: tilusnet
Priority: blocker Milestone:
Component: Plugin alignways Version: latest
Keywords: shortcut add node Cc:

Description

Somehow the default shortcut for add node was recently changed from "a" to "d". The delete shortcut changed from "d" to shift+alt+d and "a" now is Tool:all inside.

Please revert this, it does not make sense and is an important usability issue for all previous users.

Maybe this is a plugin issue?

Attachments (0)

Change History (19)

comment:1 by dieterdreist, 13 years ago

Priority: blockercritical

comment:2 by rickmastfan67, 13 years ago

What plugins do you have installed? I'm running r4419 and the problem doesn't happen.

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

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

Replying to rickmastfan67:

Plugin: ImportImagePlugin (26606)
Plugin: OpeningHoursEditor (26606)
Plugin: PicLayer (26606)
Plugin: alignways (26606)
Plugin: buildings_tools (26606)
Plugin: dataimport (26606)
Plugin: osmarender (26606)
Plugin: pdfimport (26606)
Plugin: photo_geotagging (26606)
Plugin: proj4j (26606)
Plugin: public_transport (26606)
Plugin: restart (26606)
Plugin: reverter (26606)
Plugin: tagging-preset-tester (26606)
Plugin: undelete (26606)
Plugin: utilsplugin2 (26641)
Plugin: waypoint_search (26606)
Last edited 13 years ago by dieterdreist (previous) (diff)

comment:4 by skyper, 13 years ago

Owner: changed from team to dieterdreist
Status: newneedinfo

Please try it with a new/empty josm directory.

This seems to be a problem with a plugin. Please try to figure out which one(s).

Some more infomation like operating system, java version and josm version would also be helpful.

Thanks

comment:5 by dieterdreist, 13 years ago

It now became worse ;-). I don't see any "mode"-shortcuts in the preferences menu any more (some of them still do work). IMHO there could also be a box: use all defaults (as it is currently you have to check all entries manually if I see it right).

comment:6 by dieterdreist, 13 years ago

After restarting, the modes came back to the shortcut preferences. All shortcuts are set to default. IMHO a plugin should not be able to change a core shortcut (maybe except user confirms explicitly, but then the shortcut is not "default" any more). I am still investigating, which plugin creates the problem. The current silent shortcut conflicts are:

Silent shortcut conflict: 'menu:PicLayer' moved by 'menu:Imagery' to 'Alt+B'.
Silent shortcut conflict: 'tools:taggingresettester' moved by 'tools:multipoly' to 'Alt+Umschalt+A'.
Silent shortcut conflict: 'menu:Public Transport' moved by 'menu:Presets' to 'Alt+C'.
Silent shortcut conflict: 'tools:intway' moved by 'tools:alignways' to 'Alt+Umschalt+I'.
Silent shortcut conflict: 'tools:selinside' moved by 'tools:intway' to 'A'.
Silent shortcut conflict: 'mapmode:draw' moved by 'tools:selinside' to 'D'.
Silent shortcut conflict: 'mapmode:delete' moved by 'mapmode:draw' to 'Alt+Umschalt+D'.
Silent shortcut conflict: 'mapmode:parallel' moved by 'tools:pdfimport' to 'Alt+Umschalt+P'.
Silent shortcut conflict: 'subwindow:properties' moved by 'mapmode:parallel' to 'Alt+D'.
Silent shortcut conflict: 'properties:add' moved by 'menu:PicLayer' to 'Alt+G'.
Silent shortcut conflict: 'subwindow:authors' moved by 'tools:taggingresettester' to 'Alt+J'.
Keystroke alt pressed J is already assigned to utilsplugin2.ExtractPointAction@e634bf, will be overridden by org.openstreetmap.josm.gui.dialogs.ToggleDialog$ToggleDialogAction@bd4dc2
Silent shortcut conflict: 'subwindow:conflict' moved by 'menu:Public Transport' to 'Alt+Umschalt+C'.

and there is also some Toolbar actions without name (not sure if they belong to this thread):

Registered toolbar action without name: org.openstreetmap.josm.plugins.osmarender.OsmarenderPlugin$Action
Toolbar action without name: org.openstreetmap.josm.plugins.ImportImagePlugin.LoadImageAction
Toolbar action without name: org.openstreetmap.josm.plugins.osmarender.OsmarenderPlugin$Action
Toolbar action without name: org.openstreetmap.josm.plugins.piclayer.NewLayerFromFileAction
Toolbar action without name: org.openstreetmap.josm.plugins.piclayer.NewLayerFromClipboardAction

comment:7 by dieterdreist, 13 years ago

It is the utilsplugin2. Happy to have found this, JOSM was unusuable with this bug. After I deactivated the plugin everything turned back normal. Btw.: would it be possible to have error and warning messages like the ones above somehow easily correlateable (the same) with the plugin name? In this case the plugin is called utilsplugin2 and the errors are about "tool".

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

comment:8 by dieterdreist, 13 years ago

Owner: changed from dieterdreist to team
Status: needinfonew

comment:9 by dieterdreist, 13 years ago

Component: CorePlugin utilsplugin2
Priority: criticalblocker

comment:10 by skyper, 13 years ago

Component: Plugin utilsplugin2Plugin

Can not reproduce with only utilsplugin2 installed with new preference directory.

Using /usr/lib/jvm/java-6-openjdk/bin/java to execute josm.
Repository Root: http://josm.openstreetmap.de/svn
Build-Date: 2011-09-14 01:32:24
Last Changed Author: stoecker
Revision: 4419
Repository UUID: 0c6e7542-c601-0410-84e7-c038aed88b3b
URL: http://josm.openstreetmap.de/svn/trunk
Last Changed Date: 2011-09-13 12:16:47 +0200 (Tue, 13 Sep 2011)
Last Changed Rev: 4419

GET http://api.openstreetmap.org/api/capabilities... OK
Communications with http://api.openstreetmap.org/api established using protocol version 0.6.
loading plugin 'utilsplugin2' (version 26646)

There is either at least one more plugin involved or your settings are outdated.
JOSM should not change core shortcuts thought but this is already requested in #6341.

comment:11 by rickmastfan67, 13 years ago

Component: PluginPlugin alignways
Owner: changed from team to tilusnet
Silent shortcut conflict: 'tools:selinside' moved by 'tools:intway' to 'A'.

The conflict with "A" is caused by "alignways" v26606. Thus, I've updated the component to reflect that.

comment:12 by akks, 13 years ago

I have just moved AlignWays to Ctrl-Alt-A to eliminate conflicts with Utilsplugin2 and chain reaction described above.

I key is used with almost all modifiers for Info (core) and Intersection/Inside actions (Utilsplugin2).

If something is wrong, your can change shortcuts some other ways, if all works as desired - close this bug please :)

comment:13 by dieterdreist, 13 years ago

I reactivated the utilsplugin2 and the problem with "a" remapped returned:

loading plugin 'utilsplugin2' (version 26696)
Silent shortcut conflict: 'tools:intway' moved by 'tools:alignways' to 'Alt+Umschalt+I'.
Silent shortcut conflict: 'tools:selinside' moved by 'tools:intway' to 'A'.

comment:14 by rickmastfan67, 13 years ago

Has your "alignways" plugin been updated to [o26692]?

Because I have the latest of all the plugins mentioned in comment:#3 and "a" problem is NOT happening.

in reply to:  14 comment:15 by dieterdreist, 13 years ago

Replying to rickmastfan67:

Has your "alignways" plugin been updated to [o26692]?

Because I have the latest of all the plugins mentioned in comment:#3 and "a" problem is NOT happening.

No, the "alignways" plugin was in version 26606 (and JOSM reported in the plugins tab that this was the newest version available). I manually hit the button "update plugins" (never had to do this in the past, JOSM always updated automatically) and got the newest version. When was the plugin update mechanism changed? I think that it would be nice to notify about important changes like this.

comment:16 by dieterdreist, 13 years ago

Now it works for me (updated plugin manually).

in reply to:  16 ; comment:17 by rickmastfan67, 13 years ago

Replying to dieterdreist:

Now it works for me (updated plugin manually).

If the "a" problem is now gone, you should mark this as fixed. ;)

comment:18 by dieterdreist, 13 years ago

Resolution: fixed
Status: newclosed

in reply to:  17 comment:19 by dieterdreist, 13 years ago

Replying to rickmastfan67:

Replying to dieterdreist:

Now it works for me (updated plugin manually).

If the "a" problem is now gone, you should mark this as fixed. ;)

Set it to fixed. Actually the reason for not doing this immediately was that I was not sure, whether the issue was solved because someone updated the plugin code (=fixed) or because I manually updated the plugin on my system (=worksforme, potentially a problem with the JOSM plugin update mechanism).

Modify Ticket

Change Properties
Set your email in Preferences
Action
as closed The owner will remain tilusnet.
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.