[Bug 30486] New: Tanner L-Edit shows blank hotkey field

http://bugs.winehq.org/show_bug.cgi?id=30486

Bug #: 30486
           Summary: Tanner L-Edit shows blank hotkey field
           Product: Wine
           Version: 1.4
          Platform: x86
        OS/Version: Linux
            Status: NEW
          Severity: normal
          Priority: P2
         Component: comctl32
        AssignedTo: [e-mail]
        ReportedBy: [e-mail]
    Classification: Unclassified

Tanner's L-Edit uses the msctls_hotkey32 control from comctl32
in its Setup / Application / Keyboard dialog, but under Wine,
the control is always blank.  It should say 'None' (or the local equivalent)
initially.  (The caret is positioned properly, though.)
'winetricks comctl32' works around the problem.
Wine's control works fine in a standalone demo.

The problem exists in many versions of the app, including the 7.12 
Student Version downloadable from various sites as ledit.zip, 
2972402 bytes, sha1sum f1a5825e63cee03f2cfb912cfa8d47c5a22aa684
That version requires 'winetricks mfc42' to start, and warns
about wanting 256 color mode.  Using Xephyr to get 256 color mode
doesn't seem to help.

[Bug 30486] Tanner L-Edit shows blank hotkey field

http://bugs.winehq.org/show_bug.cgi?id=30486

Dan Kegel  changed:

What    |Removed                     |Added
----------------------------------------------------------------------------
           Keywords|                            |download
                URL|                            |http://forum.rficdesign.com
                   |                            |/Attachments/ledit.zip

[Bug 30486] Tanner L-Edit shows blank hotkey field

http://bugs.winehq.org/show_bug.cgi?id=30486

--- Comment #1 from Dan Kegel  2012-04-19 20:20:42 CDT ---
Created attachment 39868
  --> http://bugs.winehq.org/attachment.cgi?id=39868
Screenshot of bad dialog in Wine; see blank control under "Press new shortcut
key"

[Bug 30486] Tanner L-Edit shows blank hotkey field

http://bugs.winehq.org/show_bug.cgi?id=30486

--- Comment #2 from Dan Kegel  2012-04-19 20:43:36 CDT ---
Created attachment 39869
  --> http://bugs.winehq.org/attachment.cgi?id=39869
Little standalone demo of hotkey control; works fine in wine

[Bug 30486] Tanner L-Edit shows blank hotkey field

http://bugs.winehq.org/show_bug.cgi?id=30486

Darmis  changed:

What    |Removed                     |Added
----------------------------------------------------------------------------
                 CC|                            |[e-mail]

--- Comment #3 from Darmis  2012-04-24 22:59:45 CDT ---
I'm taking a browse around here.

[Bug 30486] Tanner L-Edit shows blank hotkey field

http://bugs.winehq.org/show_bug.cgi?id=30486

--- Comment #4 from Bill Alley  2012-04-27 13:41:43 CDT ---
Created attachment 39938
  --> http://bugs.winehq.org/attachment.cgi?id=39938
InvalidateRect() patch

This patch seems to make the app happy. I do not know if it's correct, so I am
not submitting it to wine-patches yet.

[Bug 30486] Tanner L-Edit shows blank hotkey field

http://bugs.winehq.org/show_bug.cgi?id=30486

Dan Kegel  changed:

What    |Removed                     |Added
----------------------------------------------------------------------------
  Attachment #39869|0                           |1
        is obsolete|                            |

--- Comment #5 from Dan Kegel  2012-06-20 14:28:56 CDT ---
Created attachment 40627
  --> http://bugs.winehq.org/attachment.cgi?id=40627
Small program to reproduce problem

Now that I understand what's going on, here's a tiny program
that reproduces the problem.

[Bug 30486] Tanner L-Edit shows blank hotkey field

http://bugs.winehq.org/show_bug.cgi?id=30486

Dan Kegel  changed:

What    |Removed                     |Added
----------------------------------------------------------------------------
           Keywords|                            |source, testcase

[Bug 30486] Tanner L-Edit shows blank hotkey field

http://bugs.winehq.org/show_bug.cgi?id=30486

Dan Kegel  changed:

What    |Removed                     |Added
----------------------------------------------------------------------------
  Attachment #40627|0                           |1
        is obsolete|                            |

--- Comment #6 from Dan Kegel  2012-06-24 12:07:13 CDT ---
Created attachment 40684
  --> http://bugs.winehq.org/attachment.cgi?id=40684
Small program to reproduce problem (right file this time)

Grr.  Fat-fingered the upload.  Here's the right file.

[Bug 30486] Tanner L-Edit shows blank hotkey field

http://bugs.winehq.org/show_bug.cgi?id=30486

Nikolay Sivov  changed:

What    |Removed                     |Added
----------------------------------------------------------------------------
  Attachment #40684|text/x-csrc                 |text/plain
          mime type|                            |

[Bug 30486] Tanner L-Edit shows blank hotkey field

http://bugs.winehq.org/show_bug.cgi?id=30486

André H.  changed:

What    |Removed                     |Added
----------------------------------------------------------------------------
                 CC|                            |[e-mail]

--- Comment #7 from André H.  2013-08-17 06:17:41 CDT ---
(In reply to comment #4)
> Created attachment 39938 [details]
> InvalidateRect() patch
>
> This patch seems to make the app happy. I do not know if it's correct, so I am
> not submitting it to wine-patches yet.

Maybe you could add some tests?

[Bug 30486] Tanner L-Edit shows blank hotkey field

https://bugs.winehq.org/show_bug.cgi?id=30486

[e-mail] changed:

What    |Removed                     |Added
----------------------------------------------------------------------------
                 CC|                            |[e-mail]