[Bug 18949] New: Few GUI bugs - demonstrated with test application (attachment)

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

Summary: Few GUI bugs - demonstrated with test application
                    (attachment)
           Product: Wine
           Version: 1.1.22
          Platform: PC
        OS/Version: Linux
            Status: UNCONFIRMED
          Severity: normal
          Priority: P2
         Component: -unknown
        AssignedTo: [e-mail]
        ReportedBy: [e-mail]

Created an attachment (id=21797)
 --> (http://bugs.winehq.org/attachment.cgi?id=21797)
This application demonstrates mentioned bugs.

Please test this application on Windows first to see how it should work
properly.

1) ListView
    a) it's possible to resize column size even the HDN_ITEMCHANGING,
HDN_BEGINTRACK, HDN_ITEMCHANGED and HDN_ENDTRACK
    (WM_NOTIFY) messages are caught in WindowProc and forced to return TRUE.
    b) why it's possible to move (drag) column headers while the order can't be
changed?
    c) font changed in WM_NOTIFY->NM_CUSTOMDRAW->CDDS_ITEMPREPAINT works only
for the first column, and even there the
    font is bad - it's only ITALIC, while it should be BOLD+ITALIC+UNDERLINE.

2) CheckBox
    a) the font is not BOLD (it's the same font as used in LV) - a problem in
CreateFontIndirect()?

3) Menu - MF_OWNERDRAW
    a) 2 problems: the default item in owner drawn menu should be indicated
with ODS_DEFAULT in the itemState member
    of DRAWITEMSTRUCT. Also, when it comes to owner-draw a defaulted item, the
BOLD font should be already selected in
    the DC (hDC in DRAWITEMSTRUCT) by OS.
    b) SetMenuInfo() with MIM_BACKGROUND for setting a menu background is not
implemented.
###    c) bug in MoveToEx()? I set MoveToEx(hDC, 1, top+10, NULL), but the
result is like it would
    be MoveToEx(hDC, 0, top+10, NULL). All drawing results are displaced by 1px
to the left. Demonstrated by
    the blue line - simply compare to Windows.

4) WM_NCLBUTTONDOWN - allows to move the window by clicking in its client area
    a) very buggy, makes difficult to use CheckBox, ComboBox, headers in LV and
possibly many more.

[Bug 18949] Few GUI bugs - demonstrated with test application (attachment)

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

--- Comment #1 from [e-mail]  2009-06-14 18:49:30 ---
Created an attachment (id=21798)
 --> (http://bugs.winehq.org/attachment.cgi?id=21798)
Screenshot.

[Bug 18949] Few GUI bugs - demonstrated with test application (attachment)

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

--- Comment #2 from Nikolay Sivov   2009-06-14 19:11:40 ---
(In reply to comment #0)
> Created an attachment (id=21797)

--> (http://bugs.winehq.org/attachment.cgi?id=21797) [details]
> This application demonstrates mentioned bugs.

Hi, attach source too please.

First of all open one bug for one problem so we could process them.
> Please test this application on Windows first to see how it should work
> properly.
> - ListView
>  a) it's possible to resize column size even the HDN_ITEMCHANGING,
>
> HDN_BEGINTRACK, HDN_ITEMCHANGED and HDN_ENDTRACK
>     (WM_NOTIFY) messages are caught in WindowProc and forced to return TRUE.
>     b) why it's possible to move (drag) column headers while the order can't be
> changed?

This belongs to comctl32.

It doesn't matter currently cause reordering is blocked in HDN_ENDDRAG.
Bug http://bugs.winehq.org/show_bug.cgi?id=3231 is about it and I'm currently
working on this issue.
> c) font changed in WM_NOTIFY->NM_CUSTOMDRAW->CDDS_ITEMPREPAINT works only
>
> for the first column, and even there the
>     font is bad - it's only ITALIC, while it should be BOLD+ITALIC+UNDERLINE.

Can't comment just now. I'm not very familiar with custom draw.
> - CheckBox
>  a) the font is not BOLD (it's the same font as used in LV) - a problem in
>
> CreateFontIndirect()?

Try to output a string to HDC without any control to be sure.
> - Menu - MF_OWNERDRAW
>  a) 2 problems: the default item in owner drawn menu should be indicated
>
> with ODS_DEFAULT in the itemState member
>     of DRAWITEMSTRUCT. Also, when it comes to owner-draw a defaulted item, the
> BOLD font should be already selected in
>     the DC (hDC in DRAWITEMSTRUCT) by OS.
>     b) SetMenuInfo() with MIM_BACKGROUND for setting a menu background is not
> implemented.

It's about user32 (or winex11 maybe).
> ###    c) bug in MoveToEx()? I set MoveToEx(hDC, 1, top+10, NULL), but the
> result is like it would
>     be MoveToEx(hDC, 0, top+10, NULL). All drawing results are displaced by 1px

MoveToEx is control independent. Open separate bug for it.
> to the left. Demonstrated by
>     the blue line - simply compare to Windows.
> - WM_NCLBUTTONDOWN - allows to move the window by clicking in its client area
>  a) very buggy, makes difficult to use CheckBox, ComboBox, headers in LV and

It deserves separate report too of course.
> possibly many more.

So please split it in such way: one problem -> one report.
After that post source code of test application (in this report maybe), I'll
take a look.

When you'll be ready with other reports, post here request to close this one.

Bye.

[Bug 18949] Few GUI bugs - demonstrated with test application (attachment)

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

Austin English  changed:

What    |Removed                     |Added
----------------------------------------------------------------------------
           Keywords|                            |download

[Bug 18949] Few GUI bugs - demonstrated with test application (attachment)

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

Jerome Leclanche  changed:

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

[Bug 18949] Few GUI bugs - demonstrated with test application (attachment)

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

Dmitry Timoshkov  changed:

What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|UNCONFIRMED                 |RESOLVED
         Resolution|                            |INVALID

--- Comment #3 from Dmitry Timoshkov   2009-06-15 00:19:10 ---
One bug per bug report please. Also it's always useful to report bugs
for latest Wine version.

[Bug 18949] Few GUI bugs - demonstrated with test application (attachment)

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

Dmitry Timoshkov  changed:

What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|RESOLVED                    |CLOSED

--- Comment #4 from Dmitry Timoshkov   2009-06-15 00:19:49 ---
Please open a separate bug report for each separate problem.