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 603505 - [PATCH] fix accelerators on custom buttons
[PATCH] fix accelerators on custom buttons
Status: RESOLVED DUPLICATE of bug 601861
Product: GnuCash
Classification: Other
Component: User Interface General
2.3.x
Other Linux
: Normal normal
: ---
Assigned To: David Hampton
Chris Shoemaker
Depends on:
Blocks:
 
 
Reported: 2009-12-01 17:39 UTC by Bill Nottingham
Modified: 2018-06-29 22:31 UTC
See Also:
GNOME target: ---
GNOME version: ---


Attachments
Use new_with_mnemonic, not new_with_label (504 bytes, patch)
2009-12-01 17:39 UTC, Bill Nottingham
accepted-commit_now Details | Review

Description Bill Nottingham 2009-12-01 17:39:39 UTC
Created attachment 148842 [details] [review]
Use new_with_mnemonic, not new_with_label

Downstream bug #1 (https://bugzilla.redhat.com/show_bug.cgi?id=533019)

Description of problem:
Accelerator (underlined letter) for the "Delete Transaction" button in the
"Delete transaction" dialog is wrong.

Version-Release number of selected component (if applicable):
gnucash-2.2.9-1.fc11.x86_64

How reproducible:
Always

Steps to Reproduce:
1. Delete a transaction
2. Look at confirmation dialog
3. Try pressing Alt-D

Actual results:
Shows as "_Delete transaction"; Alt-D does not work

Expected results:
"D" is underlined; Alt-D will activate button.

Additional info:
Other accelerators (e.g. Alt-C for Cancel) work correctly.  

Downstream bug #2 (https://bugzilla.redhat.com/show_bug.cgi?id=541915)

Description of problem:
When opening GnuCash for a file that is in use by another user, the dialog does
not use underline for the non-stock buttons.

Version-Release number of selected component (if applicable):
gnucash-2.2.9-2.fc12

How reproducible:
100%

Steps to Reproduce:
1.Make sure file has group read/write access and resides in a directory with
group read/write access for a group both users are in
2.As user1, start gnucash and open the file
3.Do the same as user2

Actual results:
GnuCash could not obtain the lock for ....

That database may be in use by another user, in which case you should not open
it. What would you like to do?

[_Open Anyway] [_Create New File] [Quit]

(the Q of Quit is underlined; nothing else is)

Expected results:
The O and C should be underlined, with no leading underscore character.  

...

Basically, gnc_gtk_dialog_add_button is using the wrong gtk function. Patch is against 2.3.7, but should apply to 2.2.x, I imagine.
Comment 1 Christian Stimming 2009-12-07 14:12:41 UTC
Comment on attachment 148842 [details] [review]
Use new_with_mnemonic, not new_with_label

Good point. Thanks a lot for this patch.
Comment 2 Christian Stimming 2009-12-08 21:10:05 UTC
Thanks for the patch - however, it was already reported in bug#601861 and applied r18419, so even though it is still in 2.3.7, it is no longer in SVN and thus will already be fixed in 2.3.8.

*** This bug has been marked as a duplicate of bug 601861 ***
Comment 3 John Ralls 2018-06-29 22:31:28 UTC
GnuCash bug tracking has moved to a new Bugzilla host. This bug has been copied to https://bugs.gnucash.org/show_bug.cgi?id=603505. Please update any external references or bookmarks.