Closed
Bug 135234
Opened 24 years ago
Closed 24 years ago
[PATCH]Cursor is an I-beam when over scrollbar
Categories
(SeaMonkey :: Composer, defect, P3)
SeaMonkey
Composer
Tracking
(Not tracked)
VERIFIED
FIXED
mozilla1.0
People
(Reporter: akkzilla, Assigned: kinmoz)
References
Details
(Keywords: regression, Whiteboard: [adt2])
Attachments
(1 file)
|
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
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
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 5•24 years ago
|
||
Comment on attachment 77507 [details] [diff] [review]
Patch Rev 1
sr=sfraser
Attachment #77507 -
Flags: superreview+
Comment 6•24 years ago
|
||
*** Bug 132851 has been marked as a duplicate of this bug. ***
Whiteboard: FIX IN HAND, need r=, sr=, a= → FIX IN HAND, need a=
Comment 7•24 years ago
|
||
nsbeta1+
Comment 8•24 years ago
|
||
adt1.0.0+ (on ADT's behalf) for approval for checkin.
Comment 9•24 years ago
|
||
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+
| Assignee | ||
Comment 10•24 years ago
|
||
Fix checked into the TRUNK:
mozilla/editor/ui/composer/content/EditorOverride.css revision 1.21
Status: ASSIGNED → RESOLVED
Closed: 24 years ago
Resolution: --- → FIXED
Updated•24 years ago
|
Keywords: fixed1.0.0
Comment 11•24 years ago
|
||
verified in 4/23 trunk.
will verify on the branch also and comment here.
Status: RESOLVED → VERIFIED
Updated•21 years ago
|
Product: Browser → Seamonkey
You need to log in
before you can comment on or make changes to this bug.
Description
•