[Bug 49487] New: list control custom draw in report view shows blank item if handler clears uItemState CDIS_SELECTED flag

List overview All Threads

newer

older

[Bug 52548] New: GIMP 2.2.x...

[Bug 43866] New: Crash while...

WineHQ Bugzilla

30 Jun 2020 30 Jun '20

8:13 a.m.

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

Bug ID: 49487 Summary: list control custom draw in report view shows blank item if handler clears uItemState CDIS_SELECTED flag Product: Wine Version: 5.0.1 Hardware: x86 OS: Mac OS X Status: UNCONFIRMED Severity: normal Priority: P2 Component: -unknown Assignee: wine-bugs@winehq.org Reporter: victimofleisure@yahoo.com

I'm the developer of a free software application called Polymeter. The issue occurs when running Polymeter under Mac OS / High Sierra. I have reproduced it on multiple machines. I install my software using WineBottler (both 1.8.6 and 4.0.1 exhibited the issue). WineBottler uses WineHQ 5.0 according to their page.

The issue is per-item background color for *selected* list control items in report view. The usual method for achieving per-item color is to request item notifications, and when the prepaint notification is received, set the per-item background color (pLVCD->clrTextBk). However by default this only works for non-selected items: selected items continue to use the system default background color (typically a dark blue) instead of the per-item color. The workaround is for the custom draw handler to also clear the "selected" flag (CDIS_SELECTED) in the item state (pLVCD->uItemState). This technique is fairly common and can be observed in the wild, e.g. here.

https://social.msdn.microsoft.com/Forums/vstudio/en-US/40234af5-a9e7-433f-a3...

However in my application under WineBottler, this technique causes the list item in question to be completely white, both text and background.

I enclose a code snippet below. This may be related to 39721 or even a duplicate of it, but it's hard to tell because that report is not as specific and doesn't mention selected items. If list control per-item background color is broken in all cases regardless of selection, that would explain my issue obviously, but that seems doubtful since many Windows programs use list control custom draw.

Best wishes, Chris Korda https://victimofleisure.github.io/Polymeter/

void CMappingBar::OnListCustomDraw(NMHDR* pNMHDR, LRESULT* pResult) { NMLVCUSTOMDRAW* pLVCD = reinterpret_cast<NMLVCUSTOMDRAW*>(pNMHDR); *pResult = CDRF_DODEFAULT; if (theApp.m_bIsMidiLearn) { // if learning MIDI input switch (pLVCD->nmcd.dwDrawStage) { case CDDS_PREPAINT: *pResult = CDRF_NOTIFYITEMDRAW; break; case CDDS_ITEMPREPAINT: // this will NOT work with LVS_SHOWSELALWAYS; see uItemState in NMCUSTOMDRAW doc if (pLVCD->nmcd.uItemState & CDIS_SELECTED) { // if item selected pLVCD->clrTextBk = MIDI_LEARN_COLOR; // customize item background color // trick system into using our custom color instead of selection color pLVCD->nmcd.uItemState &= ~CDIS_SELECTED; // clear item's selected flag } break; } } }

-- Do not reply to this email, post in Bugzilla using the above URL to reply. You are receiving this mail because: You are watching all bug changes.

Show replies by date

WineHQ Bugzilla

30 Jun 30 Jun

8:37 a.m.

New subject: [Bug 49487] list control custom draw in report view shows blank item if handler clears uItemState CDIS_SELECTED flag

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

--- Comment #1 from Nikolay Sivov bunglehead@gmail.com --- Hi, Chris.

Thank you for detailed report. Could you describe how to reproduce this issue in your application? We seem to have some logic in prepaint_setup() that overrides colors for CDIS_SELECTED, but I'd like to see a test scenario with your application to see how it breaks.

-- Do not reply to this email, post in Bugzilla using the above URL to reply. You are receiving this mail because: You are watching all bug changes.

WineHQ Bugzilla

9:12 a.m.

New subject: [Bug 49487] list control custom draw in report view shows blank item if handler clears uItemState CDIS_SELECTED flag

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

--- Comment #2 from RCK victimofleisure@yahoo.com --- Created attachment 67604 --> https://bugs.winehq.org/attachment.cgi?id=67604 Demo app

This is the simplest app I can make that demonstrates the issue. It's a stock Visual Studio 2012 dialog app with very minor modifications. The dialog has only a virtual (owner data) list control. The list control uses the custom draw technique described in the bug report to display *selected* list items in bright red. Under Wine (if the bug is reproducible) the selected list items will display incorrectly or more likely not display at all.

-- Do not reply to this email, post in Bugzilla using the above URL to reply. You are receiving this mail because: You are watching all bug changes.

WineHQ Bugzilla

9:18 a.m.

New subject: [Bug 49487] list control custom draw in report view shows blank item if handler clears uItemState CDIS_SELECTED flag

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

--- Comment #3 from RCK victimofleisure@yahoo.com --- (In reply to Nikolay Sivov from comment #1)

...

Hi, Chris.

Thank you for detailed report. Could you describe how to reproduce this issue in your application? We seem to have some logic in prepaint_setup() that overrides colors for CDIS_SELECTED, but I'd like to see a test scenario with your application to see how it breaks.

Hi. I attached a demo app to my original report. The zip file includes 32-bit and 64-bit static build executables as well as all sources. It's a stock Visual Studio 2012 dialog app with very minor modifications. The dialog has only a virtual (owner data) list control. The list control uses the custom draw technique described in the bug report to display *selected* list items in bright red. Under Wine (if the bug is reproducible), the selected list items will display incorrectly or more likely will not display at all.

https://bugs.winehq.org/attachment.cgi?id=67604

-- Do not reply to this email, post in Bugzilla using the above URL to reply. You are receiving this mail because: You are watching all bug changes.

WineHQ Bugzilla

9:21 a.m.

New subject: [Bug 49487] list control custom draw in report view shows blank item if handler clears uItemState CDIS_SELECTED flag

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

--- Comment #4 from RCK victimofleisure@yahoo.com --- I should have mentioned that my case involves a virtual (owner-data) list control. I haven't explored whether the issue would also appear with a non-virtual list control. Apologies for the omission. Best wishes, Chris

-- Do not reply to this email, post in Bugzilla using the above URL to reply. You are receiving this mail because: You are watching all bug changes.

WineHQ Bugzilla

9:24 a.m.

New subject: [Bug 49487] list control custom draw in report view shows blank item if handler clears uItemState CDIS_SELECTED flag

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

--- Comment #5 from Nikolay Sivov bunglehead@gmail.com --- Created attachment 67605 --> https://bugs.winehq.org/attachment.cgi?id=67605 diff

Ok, thanks. Color gets replaced back to default one in subitems loop. Attach diff disables that. It will need more testing and examination of all possible paths.

-- Do not reply to this email, post in Bugzilla using the above URL to reply. You are receiving this mail because: You are watching all bug changes.

WineHQ Bugzilla

9:25 a.m.

New subject: [Bug 49487] list control custom draw in report view shows blank item if handler clears uItemState CDIS_SELECTED flag

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

Nikolay Sivov bunglehead@gmail.com changed:

--- Comment #6 from Nikolay Sivov bunglehead@gmail.com --- Confirming.

-- Do not reply to this email, post in Bugzilla using the above URL to reply. You are receiving this mail because: You are watching all bug changes.

WineHQ Bugzilla

9:34 a.m.

New subject: [Bug 49487] list control custom draw in report view shows blank item if handler clears uItemState CDIS_SELECTED flag

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

--- Comment #7 from RCK victimofleisure@yahoo.com --- Created attachment 67606 --> https://bugs.winehq.org/attachment.cgi?id=67606 Image of desired output

This is a screen shot of the demo app running under Windows 7. The selected item is bright red, rather than the default selection color (typical dark blue).

-- Do not reply to this email, post in Bugzilla using the above URL to reply. You are receiving this mail because: You are watching all bug changes.

WineHQ Bugzilla

22 Jul 22 Jul

5:57 a.m.

New subject: [Bug 49487] list control custom draw in report view shows blank item if handler clears uItemState CDIS_SELECTED flag

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

Nikolay Sivov bunglehead@gmail.com changed:

--- Comment #8 from Nikolay Sivov bunglehead@gmail.com --- This is fixed now, c32720349cd4a6f4c32fdcff6a4cce054fbfe33e. Please retest.

-- Do not reply to this email, post in Bugzilla using the above URL to reply. You are receiving this mail because: You are watching all bug changes.

WineHQ Bugzilla

5:57 a.m.

New subject: [Bug 49487] ListView control custom draw in report view shows blank item if handler clears uItemState CDIS_SELECTED flag

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

Nikolay Sivov bunglehead@gmail.com changed:

-- Do not reply to this email, post in Bugzilla using the above URL to reply. You are receiving this mail because: You are watching all bug changes.

WineHQ Bugzilla

1 Aug 1 Aug

10:46 a.m.

New subject: [Bug 49487] ListView control custom draw in report view shows blank item if handler clears uItemState CDIS_SELECTED flag

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

Alexandre Julliard julliard@winehq.org changed:

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

--- Comment #9 from Alexandre Julliard julliard@winehq.org --- Closing bugs fixed in 5.14.

-- Do not reply to this email, post in Bugzilla using the above URL to reply. You are receiving this mail because: You are watching all bug changes.

WineHQ Bugzilla

16 Sep 16 Sep

8:51 p.m.

New subject: [Bug 49487] ListView control custom draw in report view shows blank item if handler clears uItemState CDIS_SELECTED flag

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

Michael Stefaniuc mstefani@winehq.org changed:

What |Removed |Added ---------------------------------------------------------------------------- Target Milestone|--- |5.0.x

-- Do not reply to this email, post in Bugzilla using the above URL to reply. You are receiving this mail because: You are watching all bug changes.

WineHQ Bugzilla

10 Nov 10 Nov

8 p.m.

New subject: [Bug 49487] ListView control custom draw in report view shows blank item if handler clears uItemState CDIS_SELECTED flag

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

Michael Stefaniuc mstefani@winehq.org changed:

What |Removed |Added ---------------------------------------------------------------------------- Target Milestone|5.0.x |---

--- Comment #10 from Michael Stefaniuc mstefani@winehq.org --- Removing the 5.0.x milestone from bug fixes included in 5.0.3.

-- Do not reply to this email, post in Bugzilla using the above URL to reply. You are receiving this mail because: You are watching all bug changes.

WineHQ Bugzilla

31 Jan 31 Jan

2:59 p.m.

New subject: [Bug 49487] ListView control custom draw in report view shows blank item if handler clears uItemState CDIS_SELECTED flag

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

RCK victimofleisure@yahoo.com changed:

--- Comment #11 from RCK victimofleisure@yahoo.com --- Hi. I am sorry to report that this bug was not fixed. Nikolay Sivov's 2020-07-22 revision c32720349cd4a6f4c32fdcff6a4cce054fbfe33e has no effect on this bug. The simple test program I provided in comment 2 still fails.

-- Do not reply to this email, post in Bugzilla using the above URL to reply. You are receiving this mail because: You are watching all bug changes.

WineHQ Bugzilla

4:09 p.m.

New subject: [Bug 49487] ListView control custom draw in report view shows blank item if handler clears uItemState CDIS_SELECTED flag

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

--- Comment #12 from Nikolay Sivov bunglehead@gmail.com --- I just checked and it works fine for me on wine 7.1 - I get red background for selected item. What is your full wine version as "wine --version" reports?

-- Do not reply to this email, post in Bugzilla using the above URL to reply. You are receiving this mail because: You are watching all bug changes.

WineHQ Bugzilla

1 Feb 1 Feb

3:28 p.m.

New subject: [Bug 49487] ListView control custom draw in report view shows blank item if handler clears uItemState CDIS_SELECTED flag

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

--- Comment #13 from RCK victimofleisure@yahoo.com --- Thanks for your reply. I’ll look into that. It might take me a few days to get an answer. It looks like the fix was released in 5.x, so it should definitely be present in 6.x too yes?

-- Do not reply to this email, post in Bugzilla using the above URL to reply. You are receiving this mail because: You are watching all bug changes.

WineHQ Bugzilla

3:38 p.m.

New subject: [Bug 49487] ListView control custom draw in report view shows blank item if handler clears uItemState CDIS_SELECTED flag

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

--- Comment #14 from Nikolay Sivov bunglehead@gmail.com --- Sure, it should work in 6.x as well. I'm curious though, how did you come to conclusion that this commit has no effect in a first place?

-- Do not reply to this email, post in Bugzilla using the above URL to reply. You are receiving this mail because: You are watching all bug changes.

WineHQ Bugzilla

2 Feb 2 Feb

9:41 p.m.

New subject: [Bug 49487] ListView control custom draw in report view shows blank item if handler clears uItemState CDIS_SELECTED flag

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

--- Comment #15 from RCK victimofleisure@yahoo.com --- Mac users reported the issue. To rule out version trouble, I need to borrow a Mac. Hopefully I’ll manage that this weekend.

-- Do not reply to this email, post in Bugzilla using the above URL to reply. You are receiving this mail because: You are watching all bug changes.

WineHQ Bugzilla

13 Feb 13 Feb

1:08 p.m.

New subject: [Bug 49487] ListView control custom draw in report view shows blank item if handler clears uItemState CDIS_SELECTED flag

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

--- Comment #16 from RCK victimofleisure@yahoo.com --- I withdraw my objection, and confirm that bug 49487 is definitely fixed in Wine 5.22 (the highest Wine version I can test). Apologies for the false alarm.

-- Do not reply to this email, post in Bugzilla using the above URL to reply. You are receiving this mail because: You are watching all bug changes.

WineHQ Bugzilla

1:09 p.m.

New subject: [Bug 49487] ListView control custom draw in report view shows blank item if handler clears uItemState CDIS_SELECTED flag

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

RCK victimofleisure@yahoo.com changed:

--- Comment #17 from RCK victimofleisure@yahoo.com --- I withdraw my objection, and confirm that bug 49487 is definitely fixed in Wine 5.22 (the highest Wine version I can test). Apologies for the false alarm.

-- Do not reply to this email, post in Bugzilla using the above URL to reply. You are receiving this mail because: You are watching all bug changes.

WineHQ Bugzilla

1:10 p.m.

New subject: [Bug 49487] ListView control custom draw in report view shows blank item if handler clears uItemState CDIS_SELECTED flag

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

Nikolay Sivov bunglehead@gmail.com changed:

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

--- Comment #18 from Nikolay Sivov bunglehead@gmail.com --- Thanks, closing again.

-- Do not reply to this email, post in Bugzilla using the above URL to reply. You are receiving this mail because: You are watching all bug changes.

1191

Age (days ago)

1784

Last active (days ago)

wine-bugs@winehq.org

20 comments

1 participants

tags (0)

participants (1)

WineHQ Bugzilla