Closed Bug 128534 Opened 23 years ago Closed 23 years ago

Composer and mail compose use the wrong cursor when not over text content

Categories

(SeaMonkey :: Composer, defect)

defect
Not set
normal

Tracking

(Not tracked)

VERIFIED FIXED
mozilla1.0

People

(Reporter: sfraser_bugs, Assigned: cmanske)

Details

Attachments

(1 file)

In older builds, I'm sure we had some CSS that specified that the cursor should be an I-beam when you hovered over any part of the contents of the mail compose or composer window (or a text field, or textarea). But at some point that changed. Now, I get the arrow cursor when hovering over composer content that is not a text frame. This is confusing, and wrong.
Keywords: nsbeta1, regression
cc: tucson
No we never had I-Beam over the entire content area in any 6.x or 4.x version. And neither does Word or Word Perfect. When cursor is not over text, like an image, it makes sense to use an arrow cursor to indicate that when you click on an object, it will select it. Any other opinions before I mark this invalid?
I agree with the arrow over images and other objects, but I think an I-beam over the rest of the content makes sense. Every other Mac word processor acts this way. Maybe this needs to be platform-specific?
Attached patch Fix. (deleted) — Splinter Review
If this doesn't bother any Windows or Linux users, I'm ok with using I-Beam text cursor over entire content area. Not worth the hassle for OS-subversions.
Status: NEW → ASSIGNED
Keywords: regressionpatch, review
Whiteboard: FIX IN HAND, need r=,sr=
Target Milestone: --- → mozilla1.0
I checked some linux editors: xemacs, gedit, kedit and kword all use an I-beam over the entire content area. gvim is the only dissenter, with an arrow everywhere. So Linux people would generally expect the same as mac, I-beam over text and blank areas (I suspect that other word processors like abiword and staroffice also follow that convention).
Comment on attachment 72426 [details] [diff] [review] Fix. r=brade
Attachment #72426 - Flags: review+
Whiteboard: FIX IN HAND, need r=,sr= → FIX IN HAND, need sr=
Comment on attachment 72426 [details] [diff] [review] Fix. sr=sfraser
Attachment #72426 - Flags: superreview+
Whiteboard: FIX IN HAND, need sr= → FIX IN HAND, reviewed
Comment on attachment 72426 [details] [diff] [review] Fix. a=asa (on behalf of drivers) for checkin to the 1.0 trunk
Attachment #72426 - Flags: approval+
checked in
Status: ASSIGNED → RESOLVED
Closed: 23 years ago
Keywords: patch, review
Resolution: --- → FIXED
Whiteboard: FIX IN HAND, reviewed
Verified on Win XP using the 04-30 trunk build. Marking VERIFIED.
Status: RESOLVED → VERIFIED
Product: Browser → Seamonkey
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Creator:
Created:
Updated:
Size: