Keyboard shortcut duplicate in sign/verify message window. #1927

issue burger2 opened this issue on October 13, 2012
  1. burger2 commented at 12:28 AM on October 13, 2012: contributor

    in the "Signatures - Sign / Verify a Message" Window we have the keyboard shortcut alt+S and alt+V to switch tabs and the same shortcuts to Sign or Verify the message (These two buttons should be changed to alt+m and the shortcuts for the tabs should the same).

    Tested with English and Swedish translations and client version 0.7.0 on 64-bit windows 7 and 0.7.1rc (compiled latest sources from github) on 64-bit ubuntu 12.4.

  2. burger2 commented at 12:14 PM on November 18, 2012: contributor

    The bug is still in 0.7.1 swedish translation on windows 8 64-bit.

  3. luke-jr commented at 7:07 PM on November 18, 2012: member

    @burger2 Can you check https://www.transifex.com/projects/p/bitcoin/ and correct it if still a problem there?

  4. Diapolo commented at 9:43 PM on November 18, 2012: none

    @burger2 is right, the tab and the button to execute the sign / verify have the same shortcuts. I think it's a great mess to need to re-translate such strings for all used languages when we need to change them because of a dup shortcut :-/.

  5. burger2 commented at 9:46 PM on November 18, 2012: contributor

    I was gonna try compiling the latest sources from git and with the latest translations from transifex in my ubuntu install but after my ubuntu got upgraded to 12.10 I got the new database engine that gives me errors when compiling so I haven't done any testing =(

    I'll check transifex soon...

  6. luke-jr commented at 10:40 PM on November 18, 2012: member

    @Diapolo We don't, I can easily copy the old translations - and fixes like this to the English strings shouldn't affect others, though of course in this case the translation happens to have the same bug :)

  7. burger2 commented at 11:01 PM on November 18, 2012: contributor

    Now I got it compiled finally in ubuntu and I still got the same bug. I think I have an easy fix for it.

    Just change the shortcut key to M (Verify &Message) & (Sign &Message) of the button and leave the tabs shortcuts as they are. I tried this in QT creator design mode and just changed it and recompiled and it worked for me.

  8. burger2 commented at 11:28 PM on November 18, 2012: contributor

    Now I just need to learn how to use git to submit this =)

    
    diff --git a/src/qt/forms/signverifymessagedialog.ui b/src/qt/forms/signverifymessagedialog.ui
    index 8128bdf..4887110 100644
    --- a/src/qt/forms/signverifymessagedialog.ui
    +++ b/src/qt/forms/signverifymessagedialog.ui
    @@ -148,7 +148,7 @@
                 <string>Sign the message to prove you own this Bitcoin address</string>
                </property>
                <property name="text">
    -            <string>&amp;Sign Message</string>
    +            <string>Sign &amp;Message</string>
                </property>
                <property name="icon">
                 <iconset resource="../bitcoin.qrc">
    @@ -294,7 +294,7 @@
                 <string>Verify the message to ensure it was signed with the specified Bitcoin address</string>
                </property>
                <property name="text">
    -            <string>&amp;Verify Message</string>
    +            <string>Verify &amp;Message</string>
                </property>
                <property name="icon">
                 <iconset resource="../bitcoin.qrc">
    
  9. luke-jr commented at 11:31 PM on November 18, 2012: member

    I'm confused. Are we talking about English or Swedish? :/

  10. burger2 commented at 11:35 PM on November 18, 2012: contributor

    The bug is in both... the diff is for the English string so that translators notice the bug.

  11. burger2 referenced this in commit 877c9e6d72 on Nov 18, 2012
  12. burger2 referenced this in commit a6cbfe0e96 on Nov 21, 2012
  13. burger2 commented at 10:54 PM on November 21, 2012: contributor

    Gah... I'm not compatible with git... just doing all wrong again =(

  14. burger2 commented at 10:39 PM on March 29, 2013: contributor

    I forgot to close this... it's was fixed a long time ago =)

  15. burger2 closed this on Mar 29, 2013

  16. KolbyML referenced this in commit bf04df311e on Dec 5, 2020
  17. MarcoFalke locked this on Sep 8, 2021

github-metadata-mirror

This is a metadata mirror of the GitHub repository bitcoin/bitcoin. This site is not affiliated with GitHub. Content is generated from a GitHub metadata backup.
generated: 2026-04-13 21:16 UTC

This site is hosted by @0xB10C
More mirrored repositories can be found on mirror.b10c.me