https://bugs.winehq.org/show_bug.cgi?id=43998
Bug ID: 43998 Summary: treeview not drawn correctly Product: Wine Version: 2.19 Hardware: x86-64 URL: http://www.silversecond.com/WolfRPGEditor/ OS: Linux Status: UNCONFIRMED Severity: normal Priority: P2 Component: comctl32 Assignee: wine-bugs@winehq.org Reporter: galtgendo@o2.pl Distribution: ---
I've seen this problem in at least one other place, but WolfRPG Editor is easiest to get.
Once the project gets large enough (in file numbers) so that the map treeview gets vertically scrollable, a significant portion of it at the top turns blank. Also, there are other glitches during navigating the visible portion of the treeview up/down with a keyboard. You can navigate with keyboard/mouse into the invisible section, but the displayed items get quite messy.
https://bugs.winehq.org/show_bug.cgi?id=43998
Fabian Maurer dark.shadow4@web.de changed:
What |Removed |Added ---------------------------------------------------------------------------- CC| |dark.shadow4@web.de
--- Comment #1 from Fabian Maurer dark.shadow4@web.de --- Mind providing a screenshot and how to reproduce the bug? Maybe a demo project that can be loaded and shows the issue?
https://bugs.winehq.org/show_bug.cgi?id=43998
--- Comment #2 from Rafał Mużyło galtgendo@o2.pl --- Well, I'm not quite sure myself...
It's not a matter of the list size, at least not if it's created manually...
There's something funny: on the last 32 items (when list is 90 long), clicking on the item brings up the item, as for the other items...
First 13 isn't drawn at all, but are navigable with mouse/keyboard blindly. Once you reach item 17 with the keyboard going down, focus rectangle is drawn around the item. You can go from 17 to 16 and the focus rect will show up on 16, but not above and not going in the other direction. Above 17, going with the keyboard works, but mouse clicks select item one below the clicked one. The mouse/keyboard behavior on the limit between those items and those 32 proper ones is a bit too difficult to describe.
All in all, it's a mess, but - for the time being - I can quite figure out what exactly triggers this. I can only say, once it's triggered, it stays triggered.
https://bugs.winehq.org/show_bug.cgi?id=43998
--- Comment #3 from Rafał Mużyło galtgendo@o2.pl --- ...a bit more info from a different app suffering from the same problem:
There, the treeview is contained within a scrolled window (H&V). Anyway, it seems that the difference between the row you think you click in and the row you've actually clicked is a rounding problem. What I've seen is that there the treeview content looked like it started just below the line of combined height of a single arrow and the thumb. Yet, the clicks on the content acted as if the content started at the proper place - the top of the widget.
...OK, I found that out yesterday and it seems I've already forgotten why I did I called that 'a rounding problem' (cause it wasn't just a random thought)...will write more once I remember.
One more thing (but this might be already fixed, as I'm still on 2.22): the content tooltip interacts weirdly with scroll bar. While the tooltip itself is quite small, it seems to cover a part of the treeview by an invisible window, that prevents thumb being redrawn as it passes through the covered part - once it passes that area or the tooltip is triggered on a different part of the treeview, the thumb will be redrawn correctly.
https://bugs.winehq.org/show_bug.cgi?id=43998
--- Comment #4 from Rafał Mużyło galtgendo@o2.pl --- ...OK, in 3.0 it's still to be fixed.
https://bugs.winehq.org/show_bug.cgi?id=43998
--- Comment #5 from Rafał Mużyło galtgendo@o2.pl --- Created attachment 60342 --> https://bugs.winehq.org/attachment.cgi?id=60342 a tweak fixing the problem
To clarify shorthand: the problem with item position is still valid, the part with the tooltip *might* be fixed ("might" cause the tooltip no longer shows up at all...:sigh:).
Nevertheless, attached nearly trivial patch *for the most part* seems to fix the problem, though it raises a question of where does the desync comes from. I'm not calling it a complete fix, cause while it fixed the second app, in the Wolf Editor, while after a single resize things work properly, before that the treeview is blank (most likely one of the values used by TREEVIEW_ComputeItemRect isn't initialized early enough).
https://bugs.winehq.org/show_bug.cgi?id=43998
Piotr Caban piotr.caban@gmail.com changed:
What |Removed |Added ---------------------------------------------------------------------------- CC| |piotr.caban@gmail.com
--- Comment #6 from Piotr Caban piotr.caban@gmail.com --- It works for me now. Please retest.
https://bugs.winehq.org/show_bug.cgi?id=43998
tokktokk fdsfgs@krutt.org changed:
What |Removed |Added ---------------------------------------------------------------------------- CC| |fdsfgs@krutt.org
https://bugs.winehq.org/show_bug.cgi?id=43998
--- Comment #7 from Rafał Mużyło galtgendo@o2.pl --- (In reply to Piotr Caban from comment #6)
It works for me now. Please retest.
OK, it indeed seems your patch has fixed the problem, thanks.
https://bugs.winehq.org/show_bug.cgi?id=43998
--- Comment #8 from Gijs Vermeulen gijsvrm@gmail.com --- (In reply to Rafał Mużyło from comment #7) Then please mark this bug as fixed and add c195311c6230f01b92d93d16b60a42e39a63aa4c to the 'Fixed By SHA-1' field.
https://bugs.winehq.org/show_bug.cgi?id=43998
Piotr Caban piotr.caban@gmail.com changed:
What |Removed |Added ---------------------------------------------------------------------------- Status|UNCONFIRMED |RESOLVED Resolution|--- |FIXED Fixed by SHA1| |c195311c6230f01b92d93d16b60 | |a42e39a63aa4c
--- Comment #9 from Piotr Caban piotr.caban@gmail.com --- Marking as fixed.
https://bugs.winehq.org/show_bug.cgi?id=43998
Alexandre Julliard julliard@winehq.org changed:
What |Removed |Added ---------------------------------------------------------------------------- Status|RESOLVED |CLOSED
--- Comment #10 from Alexandre Julliard julliard@winehq.org --- Closing bugs fixed in 3.5.
https://bugs.winehq.org/show_bug.cgi?id=43998
Michael Stefaniuc mstefani@winehq.org changed:
What |Removed |Added ---------------------------------------------------------------------------- Target Milestone|--- |3.0.x
https://bugs.winehq.org/show_bug.cgi?id=43998
Michael Stefaniuc mstefani@winehq.org changed:
What |Removed |Added ---------------------------------------------------------------------------- Target Milestone|3.0.x |---
--- Comment #11 from Michael Stefaniuc mstefani@winehq.org --- Removing the 3.0.x milestone from bugs included in 3.0.1.