[Bug 396033] New: yast-gtk has conflicting 'f' accelerator in users and groups
https://bugzilla.novell.com/show_bug.cgi?id=396033 Summary: yast-gtk has conflicting 'f' accelerator in users and groups Product: openSUSE 11.0 Version: RC 1 Platform: Other OS/Version: Other Status: NEW Severity: Minor Priority: P5 - None Component: YaST2 AssignedTo: bnc-team-screening@forge.provo.novell.com ReportedBy: jpr@novell.com QAContact: jsrain@novell.com CC: rpmcruz@alunos.dcc.fc.up.pt Found By: --- yast-gtk has conflicting 'f' accelerator in users and groups, alt-f is for both a tab and the finish buttom -- Configure bugmail: https://bugzilla.novell.com/userprefs.cgi?tab=email ------- You are receiving this mail because: ------- You are on the CC list for the bug.
https://bugzilla.novell.com/show_bug.cgi?id=396033 User rpmcruz@alunos.dcc.fc.up.pt added comment https://bugzilla.novell.com/show_bug.cgi?id=396033#c1 Ricardo Cruz <rpmcruz@alunos.dcc.fc.up.pt> changed: What |Removed |Added ---------------------------------------------------------------------------- Summary|yast-gtk has conflicting 'f' accelerator in |yast has conflicting 'f' accelerator in users |users and groups |and groups --- Comment #1 from Ricardo Cruz <rpmcruz@alunos.dcc.fc.up.pt> 2008-05-30 15:00:00 MDT --- Surely a problem with yast2-qt as well, right? ( /usr/lib/YaST2/bin/y2base users qt ) -- Configure bugmail: https://bugzilla.novell.com/userprefs.cgi?tab=email ------- You are receiving this mail because: ------- You are on the CC list for the bug.
https://bugzilla.novell.com/show_bug.cgi?id=396033 Ricardo Cruz <rpmcruz@alunos.dcc.fc.up.pt> changed: What |Removed |Added ---------------------------------------------------------------------------- Summary|yast has conflicting 'f' accelerator in users |conflicting 'f' accelerator in users and groups |and groups | -- Configure bugmail: https://bugzilla.novell.com/userprefs.cgi?tab=email ------- You are receiving this mail because: ------- You are on the CC list for the bug.
https://bugzilla.novell.com/show_bug.cgi?id=396033 Cyril Hrubis <chrubis@novell.com> changed: What |Removed |Added ---------------------------------------------------------------------------- AssignedTo|bnc-team-screening@forge.provo.novell.com |jsuchome@novell.com -- Configure bugmail: https://bugzilla.novell.com/userprefs.cgi?tab=email ------- You are receiving this mail because: ------- You are on the CC list for the bug.
https://bugzilla.novell.com/show_bug.cgi?id=396033 User jsuchome@novell.com added comment https://bugzilla.novell.com/show_bug.cgi?id=396033#c2 Jiří Suchomel <jsuchome@novell.com> changed: What |Removed |Added ---------------------------------------------------------------------------- CC| |jsuchome@novell.com AssignedTo|jsuchome@novell.com |sh@novell.com --- Comment #2 from Jiří Suchomel <jsuchome@novell.com> 2008-06-02 13:03:40 MDT --- Which is probably general problem of shorcut distribution. -- Configure bugmail: https://bugzilla.novell.com/userprefs.cgi?tab=email ------- You are receiving this mail because: ------- You are on the CC list for the bug.
https://bugzilla.novell.com/show_bug.cgi?id=396033 User sh@novell.com added comment https://bugzilla.novell.com/show_bug.cgi?id=396033#c3 Stefan Hundhammer <sh@novell.com> changed: What |Removed |Added ---------------------------------------------------------------------------- AssignedTo|sh@novell.com |rpmcruz@alunos.dcc.fc.up.pt --- Comment #3 from Stefan Hundhammer <sh@novell.com> 2008-06-03 03:23:08 MDT --- Tabs shouldn't have shortcuts in the first place - see bug #117725 comment #6. If an application tries to set a shortcut for a tab, it even gets stripped off: void YDumbTab::addItem( YItem * item ) { YSelectionWidget::addItem( item ); // Remove any keyboard shortcut marker there might be // to avoid confusing the YShortcutManager item->setLabel( YShortcut::cleanShortcutString( item->label() ) ); } But the YGDumbTab sets the shortcut as the first thing in its reimplemented addItem() method and ignores the item->setLabel() call later in the base class method (which is correctly called in the reimplemented method).
From YGDumbTab.cc:
virtual void addItem (YItem *item) { GtkWidget *tab_label, *image = 0, *label; string str = YGUtils::mapKBAccel (item->label()); label = gtk_label_new_with_mnemonic (str.c_str()); gtk_widget_show (label); ... ... ... YDumbTab::addItem (item); } -- Configure bugmail: https://bugzilla.novell.com/userprefs.cgi?tab=email ------- You are receiving this mail because: ------- You are on the CC list for the bug.
https://bugzilla.novell.com/show_bug.cgi?id=396033 User sh@novell.com added comment https://bugzilla.novell.com/show_bug.cgi?id=396033#c5 --- Comment #5 from Stefan Hundhammer <sh@novell.com> 2008-06-03 03:58:46 MDT --- P.S. It would probably be better to call the base class method as the first thing in the reimplemented method, then any changes to the item in the base class method would be be effective in the derived class. -- Configure bugmail: https://bugzilla.novell.com/userprefs.cgi?tab=email ------- You are receiving this mail because: ------- You are on the CC list for the bug.
https://bugzilla.novell.com/show_bug.cgi?id=396033 User rpmcruz@alunos.dcc.fc.up.pt added comment https://bugzilla.novell.com/show_bug.cgi?id=396033#c6 Ricardo Cruz <rpmcruz@alunos.dcc.fc.up.pt> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|NEW |RESOLVED Resolution| |FIXED --- Comment #6 from Ricardo Cruz <rpmcruz@alunos.dcc.fc.up.pt> 2008-06-03 19:56:24 MDT --- I see... At least this bugger is fixed then. -- Configure bugmail: https://bugzilla.novell.com/userprefs.cgi?tab=email ------- You are receiving this mail because: ------- You are on the CC list for the bug.
participants (1)
-
bugzilla_noreply＠novell.com