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 122303 - Panel menu tooltips are inaccurate.
Panel menu tooltips are inaccurate.
Status: RESOLVED FIXED
Product: gnome-panel
Classification: Other
Component: panel
2.3.x
Other Linux
: Normal minor
: ---
Assigned To: Panel Maintainers
Panel Maintainers
: 133092 (view as bug list)
Depends on:
Blocks:
 
 
Reported: 2003-09-15 01:13 UTC by msaavedra
Modified: 2004-12-22 21:47 UTC
See Also:
GNOME target: ---
GNOME version: ---


Attachments
Proposed patch. (9.20 KB, patch)
2003-10-08 16:10 UTC, Vincent Untz
none Details | Review

Description msaavedra 2003-09-15 01:13:16 UTC
This can be reproduced as follows:
1. Open the applications menu, navigate to a submenu (say, "System Tools"),
and add that submenu to the panel, using the "Add this as menu to panel"
option.
2.Hover over the new menu that has been created. The tooltip inexplicably
says "Main Menu".

Expected behavior:
The tooltip should say "System Tools". In other words, it should take the
folder name under applications:/// that the menu was created from. It seems
like the tip "Main Menu" is hard coded, though, or at least very difficult
to change.
Comment 1 Vincent Untz 2003-10-08 16:10:23 UTC
Created attachment 20570 [details] [review]
Proposed patch.
Comment 2 Vincent Untz 2003-10-08 16:14:28 UTC
Here's a patch for this (without the ChangeLog entry as it's not
complete).
What it does is install and use a property "tooltip" for the panel
menu buttons. The tooltip is stored in gconf like for the drawers.

There's one thing I'm not sure about : in panel.c, I don't add a
tooltip when to the panel menu button when it is created from within
drop_menu(). I'm not sure what it should be (I've not looked a lot a
this).

What do you think ?
Comment 3 Vincent Untz 2003-10-20 13:55:05 UTC
Mark/Arvind: can you review the patch?
Comment 4 Vincent Untz 2003-11-04 22:47:44 UTC
Mark/Arvind: ping :-)
Comment 5 Mark McLoughlin 2003-11-06 12:00:43 UTC
Sorry about that Vincent - the patch looks good, I'd only change one
thing - make it:

-  const char *menu_path)
+  MenuFinfo  *menu_info)

{
  char     *menu_path;
  gboolean  use_menu_path;
  char     *tooltip;

  if (!menu_info) {
    menu_path     = NULL;
    use_menu_path = FALSE;
    tooltip       = _("Main Menu");
  } else {
    menu_path     = menu_info->menudir;
    use_menu_path = TRUE;
    tooltip       = menu_info->dir_name ? menu_info->dir_name : 
  }

  panel_menu_button_create (toplevel, position, menu_path, 
                            use_menu_path, tooltip);
}
    

Just to make it a bit more clear
Comment 6 Vincent Untz 2003-11-13 00:40:04 UTC
Committed to HEAD (with above changes incorporated).
Comment 7 Vincent Untz 2004-02-02 11:57:23 UTC
*** Bug 133092 has been marked as a duplicate of this bug. ***