Ticket #2234 (closed defect: fixed)

Opened 18 months ago

Last modified 4 months ago

Translation can cause JosmActions to illegally handle shortcuts

Reported by: stoecker Owned by: team
Priority: major Component: Core
Version: Keywords:
Cc:

Description

When titles are translated, the corresponding JosmActions can have equal names and the Main map shortcut handler is built using these names. This results in conflicts of these actions.

Solution: Either change storage or change all JosmAction to use marktr() and translate later. Second solution requires lots of code changes and probably breaks plugins.

Attachments

Change History

Changed 4 months ago by jttt

  • status changed from new to closed
  • resolution set to fixed

(In [3227]) Fix #2234 Translation can cause JosmActions to illegally handle shortcuts

Changed 4 months ago by jttt

  • status changed from closed to reopened
  • resolution fixed deleted

Fix reverted, some parts of JOSM relies on actions being saved by name in InputMap

Changed 4 months ago by stoecker

Did you find a place where this happens. I searched, but could not really find these places.

Changed 4 months ago by jttt

Escape is registered in UnselectAllAction.

I think proper fix for this issue will be replacing direct access to Main.contentPane by method, that takes action and shortcut and register them. Then it wouldn't matter how are actions stored internally.

Changed 4 months ago by stoecker

Right. Sounds good.

Changed 4 months ago by jttt

  • status changed from reopened to closed
  • resolution set to fixed

Fixed in r3252

Add/Change #2234 (Translation can cause JosmActions to illegally handle shortcuts)

Author


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


Change Properties
<Author field>
Action
as closed
as The resolution will be set. Next status will be 'closed'
The resolution will be deleted. Next status will be 'reopened'
 
Note: See TracTickets for help on using tickets.