http://bugs.winehq.org/show_bug.cgi?id=15334
Summary: richedit: cursor doesn't change when over a scrollbar Product: Wine Version: 1.1.5 Platform: PC-x86-64 OS/Version: Linux Status: UNCONFIRMED Severity: trivial Priority: P2 Component: richedit AssignedTo: wine-bugs@winehq.org ReportedBy: tsalacinski@gmail.com
Hi,
I've noticed this a long time ago, thought it was submitted, but I think it's not, so here it is:
When application is showing richedit widget with long content, it shows a scrollbar on the right side.
When user is hovering the mouse cursor over the scrollbar, it should change to an arrow, instead the "edit" type cursor stays on.
It's a minor issue, but should be reported. Cheers
http://bugs.winehq.org/show_bug.cgi?id=15334
--- Comment #1 from Lei Zhang thestig@google.com 2008-09-27 15:54:38 --- Can you name a sample app that has this problem? Wine wordpad maybe?
http://bugs.winehq.org/show_bug.cgi?id=15334
--- Comment #2 from Tomasz Sałaciński tsalacinski@gmail.com 2008-09-27 16:18:51 --- I've spotted in when I was installing a game (I was scrolling license window), but yes, WINE wordpad have the same issue. Just open it, hit enter and hold it until you'll see the scrollbar to the left. When over a scrollbar, the cursor remains as:
|
Instead of an arrow. It does that even if I'll move the mouse out of application's window and move it directly over the scroll bar without touch richedit field.
I can't really tell if it's a regression, because as far as I remember WINE was always doing that. I am not sure, though.
http://bugs.winehq.org/show_bug.cgi?id=15334
Jaime Rave jaimerave@gmail.com changed:
What |Removed |Added ---------------------------------------------------------------------------- CC| |jaimerave@gmail.com
--- Comment #3 from Jaime Rave jaimerave@gmail.com 2008-09-27 16:47:32 --- MSN Messenger 7.0 and 7.5 installers are affected by this bug. You can see it when you have to accept the license.
http://bugs.winehq.org/show_bug.cgi?id=15334
--- Comment #4 from Jaime Rave jaimerave@gmail.com 2008-09-27 16:54:04 --- In fact a lot of installers are affected by this bug. Must of them in the license text area.
http://bugs.winehq.org/show_bug.cgi?id=15334
Austin English austinenglish@gmail.com changed:
What |Removed |Added ---------------------------------------------------------------------------- Status|UNCONFIRMED |NEW Ever Confirmed|0 |1 Keywords| |download, source
--- Comment #5 from Austin English austinenglish@gmail.com 2008-09-29 16:22:29 --- Confirming in wine wordpad.
http://bugs.winehq.org/show_bug.cgi?id=15334
Dylan Smith dylan.ah.smith@gmail.com changed:
What |Removed |Added ---------------------------------------------------------------------------- AssignedTo|wine-bugs@winehq.org |dylan.ah.smith@gmail.com Status|NEW |ASSIGNED
--- Comment #6 from Dylan Smith dylan.ah.smith@gmail.com 2008-10-04 00:21:46 --- Created an attachment (id=16457) --> (http://bugs.winehq.org/attachment.cgi?id=16457) richedit: Show arrow cursor over scrollbar.
Thanks for pointing out the bug. I would have taken care of it earlier with other cursor changes if I had noticed it then.
http://bugs.winehq.org/show_bug.cgi?id=15334
Dylan Smith dylan.ah.smith@gmail.com changed:
What |Removed |Added ---------------------------------------------------------------------------- Keywords| |patch
http://bugs.winehq.org/show_bug.cgi?id=15334
--- Comment #7 from Dylan Smith dylan.ah.smith@gmail.com 2008-10-04 00:58:20 --- I found the there is an underlying problem with GetScrollBarInfo returning the scrollbar bounds in client co-ordinates rather than screen coordinates. This should be fixed first, since it would invalidate my richedit fix.
http://bugs.winehq.org/show_bug.cgi?id=15334
Dylan Smith dylan.ah.smith@gmail.com changed:
What |Removed |Added ---------------------------------------------------------------------------- Component|richedit |user32
http://bugs.winehq.org/show_bug.cgi?id=15334
--- Comment #8 from Dylan Smith dylan.ah.smith@gmail.com 2008-10-04 02:26:56 --- Created an attachment (id=16460) --> (http://bugs.winehq.org/attachment.cgi?id=16460) user32: Converted rcScrollBar to screen coordinates.
I modified the scrollbar bounds from client to screen coordinates in user32.
This seems to fix the user32 problem, but I don't usually work on user32.dll, so I am not sure if it is done properly yet. Probably needs a test and a review.
http://bugs.winehq.org/show_bug.cgi?id=15334
Dylan Smith dylan.ah.smith@gmail.com changed:
What |Removed |Added ---------------------------------------------------------------------------- Attachment #16457|0 |1 is obsolete| |
--- Comment #9 from Dylan Smith dylan.ah.smith@gmail.com 2008-10-04 02:29:05 --- Created an attachment (id=16461) --> (http://bugs.winehq.org/attachment.cgi?id=16461) richedit: Show arrow cursor over scrollbar.
This is a replacement for the previous patch, but I changed it to take into account that the GetScrollBarInfo function should actually be returning the scrollbar bounds in screen coordinates. All I did was move the ScreenToClient call until after the check to see if the cursor is over the scrollbar.
http://bugs.winehq.org/show_bug.cgi?id=15334
--- Comment #10 from Jaime Rave jaimerave@gmail.com 2008-10-06 21:39:39 --- Fixed in current GIT. Thanks Dylan.
http://bugs.winehq.org/show_bug.cgi?id=15334
Dmitry Timoshkov dmitry@codeweavers.com changed:
What |Removed |Added ---------------------------------------------------------------------------- Status|ASSIGNED |RESOLVED Resolution| |FIXED
--- Comment #11 from Dmitry Timoshkov dmitry@codeweavers.com 2008-10-06 21:45:07 --- Reported fixed.
http://bugs.winehq.org/show_bug.cgi?id=15334
--- Comment #12 from Tomasz Sałaciński tsalacinski@gmail.com 2008-10-07 02:24:47 --- Thanks! Right now I'm looking at Wine code to contribute myself. Thank you very much, Dylan.
http://bugs.winehq.org/show_bug.cgi?id=15334
Alexandre Julliard julliard@winehq.org changed:
What |Removed |Added ---------------------------------------------------------------------------- Status|RESOLVED |CLOSED
--- Comment #13 from Alexandre Julliard julliard@winehq.org 2008-10-10 10:11:01 --- Closing bugs fixed in 1.1.6.