After an evaluation, GNOME has moved from Bugzilla to GitLab. Learn more about GitLab.
No new issues can be reported in GNOME Bugzilla anymore.
To report an issue in a GNOME project, go to GNOME GitLab.
Do not go to GNOME Gitlab for: Bluefish, Doxygen, GnuCash, GStreamer, java-gnome, LDTP, NetworkManager, Tomboy.
Bug 170623 - Explicitly register dynamic menu items hierarchies
Explicitly register dynamic menu items hierarchies
Status: RESOLVED FIXED
Product: GIMP
Classification: Other
Component: libgimp
git master
Other All
: Normal normal
: 2.4
Assigned To: GIMP Bugs
GIMP Bugs
Depends on:
Blocks:
 
 
Reported: 2005-03-17 00:29 UTC by Michael Terry
Modified: 2005-03-24 16:09 UTC
See Also:
GNOME target: ---
GNOME version: ---



Description Michael Terry 2005-03-17 00:29:37 UTC
Most plug-ins in Script-Fu register menu items like 
"Xtns->Script-Fu->Buttons->Round Button..."  The intervening menu items
Script-Fu and Buttons are generated on-the-fly as each script registers its leaf
menu item.

This makes it difficult to add mnemonic accessors to Script-Fu or Buttons.  To
add a mnemonic now, all scripts must know about the mnemonic (which creates a
headache for translators).  Ideally the dynamically generated non-leaf menu
items would have some way to be explicitly registered with a mnemonic once and
scripts could refer to the non-mnemonicized name.

This is an important accessibility issue since any break in the mnemonic chain
means that users can't drill down the menu with, for example, Alt+x+s+b+r.

This was originally discussed in bug #106991.
Comment 1 Michael Natterer 2005-03-17 18:21:08 UTC
This was supposed to be added for GIMP 2.4 anyway, good you opened
a bug for it :)
Comment 2 Michael Natterer 2005-03-24 16:09:17 UTC
Fixed in CVS:

2005-03-24  Michael Natterer  <mitch@gimp.org>

	Added API to explicitly register dynamic menu items hierarchies.
	Fixes bug #170623.

	* app/core/gimp.h: added "GSList *plug_in_menu_branches".

	* app/plug-in/plug-in-types.h

	* app/plug-in/plug-ins.[ch]: added API to register plug-in menu
	branches, just as for locale and help domans. Cleaned up handling
	of locale and help domains.

	(plug_ins_exit): free the registered menu branches.

	* app/actions/plug-in-actions.[ch] (plug_in_actions_add_branch):
	new function to explicitly add a menu branch action.

	(plug_in_actions_setup): add the registered menu branches to each
	new action group.

	(plug_in_actions_build_path): always strip the untranslated menu
	path from underlines before using it as hash table key or action
	name.

	* app/menus/plug-in-menus.c (plug_in_menus_add_proc): changed
	accordingly: strip underlines from untranslated menu paths before
	passing them to plug_in_menus_build_path().

	* app/core/gimp-gui.[ch]: added gimp_menus_create_branch() plus
	vtable entry to access the new stuff from the core. Renamed the
	functions desling with items from gimp_foo_entry() to
	gimp_foo_item().

	* app/gui/gui-vtable.c: implement create_branch() and add the
	branch action to all existing "plug-in" action groups. Note that
	we don't need to create any menus because that happens implicitly
	when adding menu items.

	* tools/pdbgen/pdb/plug_in.pdb (plugin_menu_branch_register): new
	PDB wrapper to access branch registering from plug-ins.

	* app/pdb/internal_procs.c
	* app/pdb/plug_in_cmds.c
	* libgimp/gimpplugin_pdb.[ch]: regenerated.

	* libgimp/gimp.def: changed accordingly.

	* plug-ins/script-fu/script-fu-scripts.c (script_fu_find_scripts):
	register the menu branches for all included scripts.