474693 - cursor (pointer) over scrollcorner is the wrong one (ugly nwse-resize instead of good se-resize)

Status: RESOLVED FIXED

(Toolkit :: Themes, defect)

Description

[David Baron :dbaron: (⌚️UTC-4, no longer working on Mozilla)]

The mouse pointer over the scrollcorner in the main Firefox window is the wrong one; it no longer matches the mouse pointer that you get if you move the mouse a few pixels further onto the actual window frame.

Steps to reproduce:
 1. load Firefox
 2. move the mouse to the lower-right edge of the bar at the bottom of the window where there are a bunch of dots indicating something you can grab to resize
 3. look at the mouse cursor
 4. move the mouse a few pixels down and to the right, so that it's over the actual window frame
 5. look at the mouse cursor again

Expected results:
 3. mouse cursor is the standard window resize cursor for resizing at the bottom right corner
 5. mouse cursor is the standard window resize cursor for resizing at the bottom right corner

Actual results:
 3. mouse cursor is a bizarre looking cursor indicating ability to drag something diagonally to the upper left or the lower right
 5. mouse cursor is the standard window resize cursor for resizing at the bottom right corner (as expected)

This regressed between x86_64 Linux nightlies 2009-01-18-01-mozilla-central and 2009-01-19-02-mozilla-central, which gives a regression range of:
http://hg.mozilla.org/mozilla-central/pushloghtml?fromchange=04a8a61b9999&tochange=afec2ee5b993

When I look at the area showing the bad cursor in DOM Inspector, I see that it's a xul:resizer element and a style rule in chrome://global/skin/resizer.css is giving it a 'cursor: nwse-resize'.

In the good build, it was getting a resizer[dir='bottomright'] { cursor: se-resize; } from chrome://global/skin/global.css .

Since I see that the patch in bug 240536 removed the rule that caused the good display and added the one that causes the bad display, I'm pretty confident that that patch is responsible.


I suspect this should be relatively simple to fix:  just remember that nwse-resize is not the same on all platforms as se-resize and nw-resize, and go back to using the more specific ones.

Comment 1

[David Baron :dbaron: (⌚️UTC-4, no longer working on Mozilla)]

(In reply to comment #0)
> I suspect this should be relatively simple to fix:  just remember that
> nwse-resize is not the same on all platforms as se-resize and nw-resize, and go
> back to using the more specific ones.

In particular:
 * on Windows, the platform provides only nwse-resize, etc., so we use those for nw-resize, se-resize, etc.
 * on GTK, the platform only provides nw-resize, se-resize, etc., so we actually make up our own nwse-resize that doesn't come from the platform and which therefore doesn't fit in with the cursor theme
 * on Mac, we make up all the diagonal ones ourselves, but the platform provides distinct n-resize, s-resize, and ns-resize cursors.

Comment 2

[(no longer active)]

Attachment: Patch (v1)

Separate the cursor rules based on exact direction of the resizer.

Comment 3

[Daniel Holbert [:dholbert]]

Attachment: screencast of bug on Linux

I just ran across this issue on my machine (and I was about to file a new bug, until I found this one already filed with a patch already posted -- nice! :) ).

Here's a quick ogg-video clip of it happening on Linux, FWIW.

Comment 4

[(no longer active)]

Comment on attachment 358148 [details] [diff] [review]
Patch (v1)

Dao: perhaps you can review this faster?  Thanks!

Comment 5

[(no longer active)]

http://hg.mozilla.org/mozilla-central/rev/8c2b629e1bb8

Comment 6

[(no longer active)]

Comment on attachment 358148 [details] [diff] [review]
Patch (v1)

Theme-only low-risk change which we need to take if bug 240536 is approved for the 1.9.1 branch.

Comment 7

[Mike Beltzner [:beltzner, not reading bugmail]]

Comment on attachment 358148 [details] [diff] [review]
Patch (v1)

a191=beltzner

Comment 8

[(no longer active)]

Attachment: 1.9.1 branch patch

1.9.1 patch based on the new patch in bug 240536.

Comment 9

[(no longer active)]

http://hg.mozilla.org/releases/mozilla-1.9.1/rev/0c9cb75bc9ba