Modify

Opened 4 years ago

Closed 4 years ago

Last modified 3 years ago

#18768 closed defect (fixed)

[Patch] Windows menu needs an own help topic

Reported by: Hb--- Owned by: team
Priority: trivial Milestone: 20.03
Component: Core Version:
Keywords: help Cc:

Description

Currently the Windows menu lacks of an own help topic. It shares one [1] with the Toggle/Sidebar actions [2] named /Help/ToggleDialogs [3].

Maintenance, context sensitivity and translation are lacking under this bad structure. Help pages with only one topic are best.

So MainMenu.java needs to be changed to create an own target topic [4].
Target should be /Help/Menu/Windows [5].

[1] https://josm.openstreetmap.de/browser/trunk/src/org/openstreetmap/josm/gui/MainMenu.java#L405

[2] https://josm.openstreetmap.de/browser/trunk/src/org/openstreetmap/josm/actions/DialogsToggleAction.java#L37

[3] https://josm.openstreetmap.de/wiki/Help/ToggleDialogs

[4] https://josm.openstreetmap.de/wiki/DevelopersGuide/HelpSystem/HelpTopicsList

[5] https://josm.openstreetmap.de/wiki/Help/Menu/Windows

Attachments (4)

18768patch.diff (711 bytes ) - added by Hb--- 4 years ago.
Patch to replace /ToggleDialogs with /Help/Menu/Windows/
18768.patch (711 bytes ) - added by Hb--- 4 years ago.
Same patch, new file extension
full-path-18768.patch (802 bytes ) - added by Hb--- 4 years ago.
Same patch, starting with directory structure here: https://josm.openstreetmap.de/browser/josm/trunk
second-18768.patch (755 bytes ) - added by Hb--- 4 years ago.
To target Help/Menu/Windows

Download all attachments as: .zip

Change History (19)

by Hb---, 4 years ago

Attachment: 18768patch.diff added

Patch to replace /ToggleDialogs with /Help/Menu/Windows/

by Hb---, 4 years ago

Attachment: 18768.patch added

Same patch, new file extension

comment:1 by Hb---, 4 years ago

Summary: Windows menu needs an own help topic[Patch] Windows menu needs an own help topic

comment:2 by skyper, 4 years ago

The patches do not include the corresponding file.

by Hb---, 4 years ago

Attachment: full-path-18768.patch added

Same patch, starting with directory structure here: https://josm.openstreetmap.de/browser/josm/trunk

comment:3 by Hb---, 4 years ago

Milestone: 20.03

comment:4 by Don-vip, 4 years ago

Priority: normaltrivial

comment:5 by Don-vip, 4 years ago

Resolution: fixed
Status: newclosed

In 16102/josm:

fix #18768 - add own help topic to Windows menu (patch by Hb---)

comment:6 by Hb---, 4 years ago

Thanks for submitting, but it turns out that my patch was crap. I missed the correct directory structure.

Currently the help topic is /Help/Windows
It needs to be /Help/Menu/Windows

The new patch is attached.

by Hb---, 4 years ago

Attachment: second-18768.patch added

To target Help/Menu/Windows

comment:7 by Hb---, 4 years ago

Resolution: fixed
Status: closedreopened

comment:8 by Klumbumbus, 4 years ago

Resolution: fixed
Status: reopenedclosed

In 16103/josm:

fix #18768 - Fix Windows menu help topic (patch by Hb---)

comment:9 by Don-vip, 4 years ago

In 16105/josm:

see #18768 - checkstyle

comment:10 by Klumbumbus, 4 years ago

Aww, and I was so confident that I won't screw this up... :D

comment:11 by Don-vip, 4 years ago

I probably would have missed it too :D

comment:12 by leni, 4 years ago

Is there an impact for the translated pages? What happens to /ToggleDialogs pages?

Is it enough to add the translation macro to Help/Menu/Windows and create the xx:Help/Menu/Windows pages?

in reply to:  12 comment:13 by Hb---, 4 years ago

Replying to leni:

What happens to /ToggleDialogs pages?

That page remains and is now solely for the Editbar and the Sidebar.

Is it enough to add the translation macro to Help/Menu/Windows and create the xx:Help/Menu/Windows pages?

Yes. But the original page in English is currently incomplete and describes only two parts of the menu. Please see https://josm.openstreetmap.de/browser/josm/trunk/src/org/openstreetmap/josm/gui/MainMenu.java?rev=16102#L399 for the third group.

comment:14 by skyper, 3 years ago

This fix included the wrong path, see #21000.

comment:15 by Klumbumbus, 3 years ago

In 17925/josm:

fix #21000, see #18768 - Fix Windows menu help topic (patch by skyper)

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.