style="cursor:crosshair" Causes Link Hand to Display Over Entire Page

VERIFIED FIXED

Status

()

Core
DOM: CSS Object Model
VERIFIED FIXED
16 years ago
10 years ago

People

(Reporter: Felix Miata, Assigned: mkaply)

Tracking

Trunk
x86
OS/2
Points:
---

Firefox Tracking Flags

(Not tracked)

Details

(URL)

Attachments

(1 attachment)

(Reporter)

Description

16 years ago
2002040908 (not new)
OS/2 Java 1.3; modern

If I save this page to disk and remove style="cursor:crosshair" from the body
tag, normal mouse pointer behavior is manifest. Otherwise, hand never disappears
while mouse pointer is over body pane.
(Reporter)

Comment 1

16 years ago
I wanted to supply a screenshot, but my capture app somehow makes the pointer
revert from a hand to an arrow.

Comment 2

16 years ago
*** Bug 136621 has been marked as a duplicate of this bug. ***
Uh, what's the problem here? I see the mouse cursor as a cross-hair on
everything except links on this page, but that's what style="cursor: crosshair;"
is supposed to do, isn't it? What's the problem?
(Reporter)

Comment 4

16 years ago
What OS are you using? I never see a crosshair. I never see an arrow. I only see
a link hand.
Oh, hmm, so this is an OS/2 only issue then I guess, I'm using Win2k. Either
way, this is not a DOM problem, over to layout, but I'm not sure it belongs
there either.
(Assignee)

Comment 6

16 years ago
Yeah total Os/2 issue. We don't have the crosshair.

Taking.
Assignee: jst → mkaply
Status: UNCONFIRMED → NEW
Ever confirmed: true
(Reporter)

Comment 7

16 years ago
Where does PMVIew 2000 (my screenshot shooter) get its crosshair mouse pointer
from? Do you mean we, the Mozilla OS/2 porter? Or, we, the OS/2 operating system
supplier?
(Assignee)

Comment 8

16 years ago
Both.

OS/2 doesn't have a number of pointers, so we created versions and bundle them 
as part of the browser.

This is just one I need to add to that list.

I'm working on it right now.
(Assignee)

Comment 9

16 years ago
Created attachment 78620 [details] [diff] [review]
Fix for problem and rewrite of cursor code

Rewrite of cursor code.

Adds support for this cursor and also fixes pointer leak.

There are also two binary cursors that accompany this patch.
Comment on attachment 78620 [details] [diff] [review]
Fix for problem and rewrite of cursor code

r=pedemont
Attachment #78620 - Flags: review+
(Assignee)

Comment 11

16 years ago
Comment on attachment 78620 [details] [diff] [review]
Fix for problem and rewrite of cursor code

sr=blizzard (platform specific code)

I'm not changing the license in resource.h
Attachment #78620 - Flags: superreview+

Comment 12

16 years ago
Comment on attachment 78620 [details] [diff] [review]
Fix for problem and rewrite of cursor code

a=asa (on behalf of drivers) for checkin to the 1.0 branch
Attachment #78620 - Flags: approval+
(Assignee)

Comment 13

16 years ago
Fix checked into 1.0 branch and trunk
Status: NEW → RESOLVED
Last Resolved: 16 years ago
Resolution: --- → FIXED
(Reporter)

Comment 14

16 years ago
Less than 12 hours from initial report to resolved fixed? Mama mia! We need some
clones of you to work on more bugs. 8-))
(Reporter)

Comment 15

16 years ago
Verified fixed in 2002041109
Status: RESOLVED → VERIFIED

Comment 16

16 years ago
adding branch verification keyword.  This bug has been verified in a comment,
just moving that to the keyword field.
Keywords: verified1.0.0

Updated

10 years ago
Component: DOM: Views and Formatting → DOM: CSS Object Model
QA Contact: stummala → general
You need to log in before you can comment on or make changes to this bug.