16928 - Cursor not changed properly over scrollbars in Linux.

Status: RESOLVED FIXED

(Core :: XUL, defect, P4)

Description

[dsanti00]

This is a minor thing, but it's there, and it's very easy to reproduce.

What happens: The cursor is not changed correctly sometimes when it is over a
scrollbar on the Linux build.

Version: This is the 10/20/1999 build of Mozilla for Linux.
More info about my system: Linux 2.2.12, SMP, RAM, hard disk, the works. You
probably won't need this info if I'm right about what causes this.


How to reproduce: Start up the browser and go to a web site. Position the window
so that the scrollbars become necessary, and then further position it so that
there is some text or a link under the scrollbar. Now put the mouse over the
scrollbar in a location that would be over text or a link if the scrollbar
wasn't there. The cursor will change to either the little I-beam if the cursor
is over a scrollbar in a location covering text, and the little hand pointing
left used to signify the mouse is over a link if the cursor is over the
scrollbar in a place that the scrollbar is covering a link. Minor thing, of
course. :)

Comment 1

[dsanti00]

I've added this link, just as an example. Where to see this is in the left
frame, which shows the sections of the site. Size it so that a horizontal
scrollbar is required, and you should see it happen on the vertical scrollbar.

Comment 2

[Eli Goldberg]

Moving to XP Toolkit/Widgets, and assigning to trudelle/claudius.

Comment 3

[Eli Goldberg]

[Verified still present on 1999102808 Linux build; unable to reproduce on
1999103108 Win32 build.]

Comment 4

[Peter Trudelle]

reassigning to pavlov as p4 for m16

Comment 5

[Peter Trudelle]

Mass-moving all M16 non-feature bugs to M17, which we still consider to be 
part of beta2

Comment 6

[Kevin Buhr]

The problem is that, as noted in mozilla/widget/src/windows/nsWindow.cpp,
nsWindow::SetCursor can't currently depend on "aCursor != mCursor" to decide if
the cursor actually needs changing.  This test was commented out in the Windows
code (to change the cursor unconditionally) but was left in the GTK code.

Here's a patch against the beta1 branch that comments out the test in GTKs
"nsWidget.c" and "nsWindow.c".  It fixes this problem and other cursor
irregularities on my Linux machine.

Comment 7

[Kevin Buhr]

Attachment: patch against beta1 branch to always set the cursor

Comment 8

[Peter Trudelle]

move to m18

Comment 9

[Peter Trudelle]

mass-moving all bugs to m21 that are not dofood+, or nsbeta2+

Comment 10

[Stuart Parmenter]

*** Bug 44534 has been marked as a duplicate of this bug. ***

Comment 11

[Chris Lyon]

*** Bug 118800 has been marked as a duplicate of this bug. ***

Comment 12

[Jesse Glick]

BTW #118800 just says that insertion caret always appears over <textarea>
scrollbars, in case that was not an obvious correlary from this report.

Comment 13

[Tomi Leppikangas]

I think this has been fixed? WFM build 2002042910