Modify

Opened 2 years ago

Closed 2 years ago

#22784 closed enhancement (fixed)

[patch] Link to context sensitive help is missing for "Search for objects by preset"

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 Presets menu item "Search for objects by preset"
  2. Press F1

What is the expected result?

wiki:Help/Action/TaggingPresetSearchPrimitive

What happens instead?

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

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

Note that the action is also available in the Edit menu with the identical result.

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 "Search for objects by preset"[patch] Link to context sensitive help is missing for "Search for objects by preset"
  • src/org/openstreetmap/josm/gui/tagging/presets/TaggingPresetSearchPrimitiveDialog.java

     
    11// License: GPL. For details, see LICENSE file.
    22package org.openstreetmap.josm.gui.tagging.presets;
    33
     4import static org.openstreetmap.josm.gui.help.HelpUtil.ht;
    45import static org.openstreetmap.josm.tools.I18n.tr;
    56
    67import java.awt.event.ActionEvent;
     
    3435        public Action() {
    3536            super(tr("Search for objects by preset..."), "dialogs/search", tr("Search for objects by their presets."),
    3637                    Shortcut.registerShortcut("preset:search-objects", tr("Presets: {0}", tr("Search for objects by preset...")),
    37                     KeyEvent.VK_F3, Shortcut.SHIFT), false);
    38             putValue("toolbar", "presets/search-objects");
    39             MainApplication.getToolbar().register(this);
     38                    KeyEvent.VK_F3, Shortcut.SHIFT), true, "presets/search-objects", true);
     39            setHelpId(ht("/Action/TaggingPresetSearchPrimitive"));
    4040        }
    4141
    4242        @Override

I also moved the toolbar registration and toolbarId setup to the constructor call. I don't know why are there such setups in a few places like here.

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.