Modify

Opened 2 years ago

Closed 2 years ago

#22786 closed enhancement (fixed)

[patch] Link to context sensitive help is missing for "Expert Mode"

Reported by: skyper Owned by: team
Priority: minor Milestone: 23.03
Component: Core Version:
Keywords: template_report help Cc:

Description

What steps will reproduce the problem?

  1. Hover mouse over View menu item "Expert Mode"
  2. Press F1

What is the expected result?

wiki:Help/ExpertMode

What happens instead?

Help content for the help topic Action/ExpertToggle is not available yet.

Please provide any additional information below. Attach a screenshot if possible.

Relative:URL: ^/trunk
Repository:UUID: 0c6e7542-c601-0410-84e7-c038aed88b3b
Last:Changed Date: 2023-03-02 08:22:33 +0100 (Thu, 02 Mar 2023)
Revision:18679
Build-Date:2023-03-03 02:30:57
URL:https://josm.openstreetmap.de/svn/trunk

Attachments (0)

Change History (2)

comment:1 by gaben, 2 years ago

Milestone: 23.03
Summary: Link to context sensitive help is missing for "Expert Mode"[patch] Link to context sensitive help is missing for "Expert Mode"
  • src/org/openstreetmap/josm/actions/ExpertToggleAction.java

     
    11// License: GPL. For details, see LICENSE file.
    22package org.openstreetmap.josm.actions;
    33
     4import static org.openstreetmap.josm.gui.help.HelpUtil.ht;
    45import static org.openstreetmap.josm.tools.I18n.tr;
    56
    67import java.awt.Component;
     
    113114                new ImageProvider("expert").setOptional(true),
    114115              tr("Enable/disable expert mode"),
    115116              null,
    116               false /* register toolbar */, null, false
     117              false /* register toolbar */, "expertmode", false
    117118        );
    118         setToolbarId("expertmode");
     119        setHelpId(ht("/ExpertMode"));
    119120        if (MainApplication.getToolbar() != null) {
    120121            MainApplication.getToolbar().register(this);
    121122        }

Edit: also moved the toolbarId setup to the constructor. One less line of code.

Last edited 2 years ago by gaben (previous) (diff)

comment:2 by Klumbumbus, 2 years ago

Resolution: fixed
Status: newclosed

In 18682/josm:

fix #22784, fix #22785, fix #22786 - Add context sensitive help links, patches by gaben

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.