Closed Bug 135234 Opened 22 years ago Closed 22 years ago

[PATCH]Cursor is an I-beam when over scrollbar

Categories

(SeaMonkey :: Composer, defect, P3)

Product:
SeaMonkey
Component:
Composer
Type:
defect

Tracking

(Not tracked)

Status:
VERIFIED FIXED
Milestone:
mozilla1.0

People

(Reporter: akkzilla, Assigned: kinmoz)

References

Details

(Keywords: regression, Whiteboard: [adt2])

Attachments

(1 file)

Patch Rev 1
537 bytes, patch
glazou
: review+
sfraser_bugs
: superreview+
asa
: approval+
Details | Diff | Splinter Review 
In composer, the cursor is an i-beam even when over the scrollbar.  Kin says he
has an easy fix (yay Kin!)
see also bug 132851
Blocks: 132851
Attached patch Patch Rev 1 — — Splinter Review 
This patch qualifies a bit more what should use the text cursor. The
scrolled-content pseudo refers to the actual content area frame which I believe
is a sibling of the scrollbars so it prevents scrollbars from inheriting the
text cursor.
Cc'ing sfraser for super review ... anyone else want to review?
Status: NEW → ASSIGNED
Keywords: mozilla1.0, nsbeta1, regression
Priority: -- → P3
Whiteboard: FIX IN HAND, need r=, sr=, a=
Target Milestone: --- → mozilla1.0
Comment on attachment 77507 [details] [diff] [review]
Patch Rev 1

yay ! make that land asap !
r=glazman
Attachment #77507 - Flags: review+
Comment on attachment 77507 [details] [diff] [review]
Patch Rev 1

sr=sfraser
Attachment #77507 - Flags: superreview+
*** Bug 132851 has been marked as a duplicate of this bug. ***
Whiteboard: FIX IN HAND, need r=, sr=, a= → FIX IN HAND, need a=
nsbeta1+
Keywords: nsbeta1 → nsbeta1+
Summary: Cursor is an I-beam when over scrollbar → [PATCH]Cursor is an I-beam when over scrollbar
Whiteboard: FIX IN HAND, need a= → [adt2]FIX IN HAND, need a=
Keywords: adt1.0.0
adt1.0.0+ (on ADT's behalf) for approval for checkin.
Keywords: adt1.0.0 → adt1.0.0+
Comment on attachment 77507 [details] [diff] [review]
Patch Rev 1

a=asa (on behalf of drivers) for checkin to the 1.0 trunk
Attachment #77507 - Flags: approval+
Fix checked into the TRUNK:

    mozilla/editor/ui/composer/content/EditorOverride.css  revision 1.21
Status: ASSIGNED → RESOLVED
Closed: 22 years ago
Resolution: --- → FIXED
Whiteboard: [adt2]FIX IN HAND, need a= → [adt2]
Keywords: fixed1.0.0
verified in 4/23 trunk.

will verify on the branch also and comment here.
Status: RESOLVED → VERIFIED
verified in 4/23 branch build.
Keywords: verified1.0.0
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: