Opened 14 years ago
Closed 13 years ago
#6245 closed defect (fixed)
Plugin utilsplugin2 has some shorcut conflicts with other plugins/JOSM core.
Reported by: | rickmastfan67 | Owned by: | team |
---|---|---|---|
Priority: | major | Milestone: | |
Component: | Plugin utilsplugin2 | Version: | latest |
Keywords: | Cc: |
Description (last modified by )
Just noticed this today. One of the shortcut conflicts is with the "alignways" plugin. The other one is with the "measurement" plugin.
loading plugin 'utilsplugin2' <version 25876> Silent shortcut conflict: 'menu:More tools' moved by 'subwindow:measurement' to 'Alt+B'. Silent shortcut conflict: 'tools:intway' moved by 'tools:alignways' to 'Alt+Shift+I'.
Attachments (0)
Change History (24)
comment:1 by , 14 years ago
Component: | Plugin → Plugin utilsplugin2 |
---|
follow-up: 3 comment:2 by , 14 years ago
Summary: | Plugin utilsplugin2 has some shorcut conflicts with 2 other plugins. → Plugin utilsplugin2 has some shorcut conflicts with other plugins/JOSM core. |
---|
comment:3 by , 14 years ago
Priority: | normal → major |
---|
Replying to rickmastfan67:
Here's another conflict that utilsplugin2 has. But this time, it's with a core JOSM shortcut. Just discovered this one today.
Silent shortcut conflict: 'properties:add' moved by 'menu:More tools' to 'Alt+C'.
After more investigating on this one, I've found out it's a perfect storm of plugins renaming shortcuts to have this happen. You would have to have michigan_left, measurement, and utilsplugin2 all installed and activated to have this happen.
Since the default 'properties:add' shortcut is 'Alt+B', it gets overridden by the first conflict you see mentioned in the original post. If you don't have michigan_left installed, it gets moved to 'Alt+Shift+M' and the problem with the #6282 gets created. When #6282 happens, the 'subwindow:measurement' forces 'properties:add' shortcut to 'Alt+C'. Then, 'properties:add' takes over 'Alt+C' (which is the shortcut for the 'subwindow:conflict') and thus forces 'subwindow:conflict' to 'Alt+Shift+C'.
All in all, a perfect storm of shortcut problems. Because of this, I'm upgrading this one to "major".
Here's what the CL says about all this when those three plugins only are active:
loading plugin 'michigan_left' <version 25190> loading plugin 'utilsplugin2' <version 25912> loading plugin 'measurement' <version 25199> Silent shortcut conflict: 'menu:More tools' moved by 'subwindow:measurement' to 'Alt+B'. < load data here > Silent shortcut conflict: 'properties:add' moved by 'menu:More tools' to 'Alt+C'. Silent shortcut conflict: 'subwindow:conflict' moved by 'properties:add' to 'Alt+Shift+C'. Silent shortcut conflict: 'subwindow:mappaint' moved by 'tools:michigan_left' to 'Alt+D'.
So, if you want to close #6282 and deal with it all in this ticket, I'd understand.
comment:4 by , 13 years ago
An update on this. In r4177, even with just utilsplugin2 installed, it has 4 "Silent shortcut conflicts" with JOSM core shortcuts. I mainly discovered this while trying to find out the conflicting plugin that #67 was being effected by. Anyways, here's the CL from r4177 with just utilsplugin2 being the only plugin installed:
loading plugin 'utilsplugin2' <version 26174> << Downloaded some data from OSM here >> Silent shortcut conflict: 'mapmode:parallel' moved by 'tools:splitobject' to 'Alt+Shift+P'. Silent shortcut conflict: 'subwindow:properties' moved by 'mapmode:parallel' to 'Alt+B'. Silent shortcut conflict: 'properties:add' moved by 'subwindow"properties' to 'Alt+C'. Silent shortcut conflict: 'subwindow:conflict' moved by 'properties:add' to 'Alt+Shift+C'.
comment:5 by , 13 years ago
It seems, there are no confilcts with core in 4186?
Alt-Shift-P= Parallel mode
Shift-P = split polygon
Alt-C = properties-add
Alt-Shift-C = confilcts panel
Conflict with alignways persists, but it is local plugin combination problem (just some rarely used keys are changed).
I think, we should change More Tools menu shortcut (not Alt-B) to fix subwindow:measurement.
comment:6 by , 13 years ago
The conflict(s) is(are) still there in r4186. They only show up once you have download some data from the OSM servers or open a previously saved *.osm file.
comment:7 by , 13 years ago
OK, the conflict exists :)
I think it is time to move split polygon from Shift-P to some other hotkey.
comment:9 by , 13 years ago
Resolution: | → fixed |
---|---|
Status: | new → closed |
comment:10 by , 13 years ago
Resolution: | fixed |
---|---|
Status: | closed → reopened |
follow-up: 12 comment:11 by , 13 years ago
I have moved 'More tools' shortcut to Ctrl-X. Only confict with AlignWays persists, but I do not think it needs a patch - there will be illogical shortcut and conflicts with other plugins.
I also think that AlignWays should be merged with Parallel Way in the core (not dragging the way, but ctrl- and shift- clicking ways, for example). Someone can create a ticket or patch.
comment:12 by , 13 years ago
comment:13 by , 13 years ago
In mixed plugin update the day before - [o26264], binary [o26265]. Next time I will add the revision number, thank you for example :)
comment:14 by , 13 years ago
Ah, ok. ;) Missed that update the other day. ;)
BTW, it seems the plugin didn't get published correctly. I'm trying to download the [o26269] version from inside JOSM and it's not happening. JOSM keeps saying that [o26254] is the latest version of the plugin (which is what I have). Something similar to this happened with the undelete plugin a few days ago which I brought up in #6525.
comment:15 by , 13 years ago
Strange - the plugin is new, but version in jar-manifest is old :( Looking for reason...
follow-up: 17 comment:16 by , 13 years ago
Arr, I forget to do 'svn update' before publishing binary.
Correct way is commit source - build new jar - commit jar, yes? Strange system :)
Now it should work, if real old version is installed... (I hope)
comment:17 by , 13 years ago
Replying to akks:
Arr, I forget to do 'svn update' before publishing binary.
Correct way is commit source - build new jar - commit jar, yes? Strange system :)
Now it should work, if real old version is installed... (I hope)
Update worked this time. ;)
On a side note, I just tested that new shortcut for the "More Tools" menu. Ctrl+X doesn't work for it. It seems to need to be an "Alt+(letter)" shortcut, at least in Windows. So, if the "More Tools" menu is to have a shortcut for it, it can only be the following letters in "Alt+*" (with the letter going where the star is): m o r e t l s. I don't know which ones are in use except for "T" (the "Tools" menu), "E" (the "Edit" menu), "S" (the "History" menu), and "M" (Measurement window). That would leave in theory, "o r l" as letters that could be used for the "More Tools" menu.
follow-up: 19 comment:18 by , 13 years ago
Alt-X works ( Xtended tools:) ), sorry for Ctrl-X (misspell). I can, of course, move it to Alt-O or Alt-R (Alt-L=Layers), but it would be less informative, especially considering many localizations.
comment:19 by , 13 years ago
Replying to anonymous:
Alt-X works ( Xtended tools:) ), sorry for Ctrl-X (misspell). I can, of course, move it to Alt-O or Alt-R (Alt-L=Layers), but it would be less informative, especially considering many localizations.
Ah, "Alt+X". That does work here. ;)
Anyways, the reason why I was suggesting the other letters is because when you hit the "Alt" button, all the menu's have the letter that activates them underlined (example: "F" gets underlined in "File".) "More Tools" is the only one that doesn't have an underlined letter because there is no "x" in the name. ;) Hope you get what I'm talking about. ;)
follow-up: 21 comment:20 by , 13 years ago
Ok, ok, I will change it to Alt-R some time later. In my Russian JOSM I can only imagine this extraordinary effect ;)
comment:21 by , 13 years ago
Replying to akks:
Ok, ok, I will change it to Alt-R some time later. In my Russian JOSM I can only imagine this extraordinary effect ;)
How was I suppose to know you are using the Russian version. :P
comment:22 by , 13 years ago
Figures, there is a plugin that uses "Alt+R" that I didn't realize. :(
Silent shortcut conflict: 'menu:More Tools' moved by 'plugin:roadsigns:dialog' to 'Alt+B'.
But to be honest, I really have never used that plugin much (since the only set of signs in it is German), so I'm just going to deactivate it in my copy of JOSM to eliminate the conflict on my end. ;)
comment:23 by , 13 years ago
Description: | modified (diff) |
---|
I think we can close this ticket, because there are no more conflicts with core.
comment:24 by , 13 years ago
Resolution: | → fixed |
---|---|
Status: | reopened → closed |
Here's another conflict that utilsplugin2 has. But this time, it's with a core JOSM shortcut. Just discovered this one today.