420339 - Cursor isn't changed on panel or popup elements

Status: RESOLVED FIXED

(Core :: Widget: Cocoa, defect, P1)

Description

[Kazuho Okui]

User-Agent:       Mozilla/5.0 (Macintosh; U; Intel Mac OS X 10.5; en-US; rv:1.9b3) Gecko/2008020511 Firefox/3.0b3
Build Identifier: Mozilla/5.0 (Macintosh; U; Intel Mac OS X 10.5; en-US; rv:1.9b3) Gecko/2008020511 Firefox/3.0b3

CSS "cursor" attribute doesn't work when I set the attribute to a element on a panel element.



Reproducible: Always

Steps to Reproduce:
1. Crete a panel with child element
2. Set "cursor: pointer" to the child element.
Actual Results:  
Mouse cursor doesn't change on the child element.

Expected Results:  
Mouse cursor should be pointer when I hover the child element.

This problem happens only on Firefox 3 beta on Mac OS X.

Comment 1

[Kazuho Okui]

Attachment: Sample XUL file

Could you test this XUL file? It works fine on Windows, but doesn't work on Mac. I have tested on Firefox 3 beta 5.

Comment 2

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

Won't block until it's confirmed.

Enn: is this in the right component?

Comment 3

[Neil Deakin]

I remember seeing this before, yes.

Comment 4

[Kazuho Okui]

Does someone work on this issue? You can check the issue with TwitterFox.

https://addons.mozilla.org/en-US/firefox/addon/5081

Comment 5

[Kathleen :Brade]

Confirming this bug.  This is a Mac-specific bug; Windows and Linux do NOT exhibit this bug.

I am seeing this bug with my Firefox extension in Firefox 3.0.1 in a <popup>.  No cursor changes are made.  Firefox 2 did not have this problem.  Modifying summary to include popup.

Can someone suggest what part of the code to investigate for this bug?

Comment 6

[Kathleen :Brade]

cc'ing Josh since he may be able to suggest where to look for this bug.

Comment 7

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

Might this have been fixed by bug 496601?

Comment 8

[Kathleen :Brade]

I opened the testcase attached to this bug on the nightly build from yesterday [Mozilla/5.0 (Macintosh; U; Intel Mac OS X 10.5; en-US; rv:1.9.3a1pre) Gecko/20090908 Minefield/3.7a1pre].

The bug remains.  Can someone point me to the relevant code?  Thanks!

Comment 9

[Markus Stange [:mstange]]

nsEventStateManager::UpdateCursor seems like a good place to start.

Comment 10

[Markus Stange [:mstange]]

Oh, maybe nsCocoaWindow just needs to implement SetCursor by forwarding it to its mPopupContentView. Do you want to take this bug, Kathleen?

Comment 11

[Markus Stange [:mstange]]

Attachment: v1

Comment 12

[Markus Stange [:mstange]]

http://hg.mozilla.org/mozilla-central/rev/1ddd0b1839d0

Comment 13

[Kazuho Okui]

It seems cursor behavior is fixed, however, tooltip.openPopup(target, "after_pointer", 0, 0, false, false) function displays tooltip/menu popup wrong place or won't appear. I attached sample XUL code.

Comment 14

[Kazuho Okui]

Attachment: tooltip won't appear on cursor:pointer attribute

Comment 15

[Markus Stange [:mstange]]

That's a completely different bug. Please file a new bug report for that, complete with steps to reproduce and actual vs expected results.

Comment 16

[Kazuho Okui]

Confirmed that the bug is fixed on Firefox 3.5.4, however, it still happens in Firefox 3.6b1.

Comment 17

[Markus Stange [:mstange]]

This hasn't landed on the 1.9.1 tree yet, so it can't be fixed in Firefox 3.6.
You'll need to test a trunk build (i.e. 3.7a1pre or something).

Comment 18

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

Comment on attachment 403018 [details] [diff] [review]
v1

a192=beltzner

Comment 19

[Markus Stange [:mstange]]

http://hg.mozilla.org/releases/mozilla-1.9.2/rev/bfa772edcca9