Open
Bug 309537
Opened 19 years ago
Updated 15 years ago
When autoscrolling, cursor should indicate current scrolling direction
Categories
(SeaMonkey :: UI Design, defect)
Tracking
(Not tracked)
NEW
People
(Reporter: csthomas, Unassigned)
References
(Depends on 2 open bugs, Blocks 1 open bug, )
Details
(Whiteboard: [cst: patch is reviewed, waiting for backend fixes])
Attachments
(9 files)
326 bytes,
image/x-icon
|
Details | |
326 bytes,
image/x-icon
|
Details | |
326 bytes,
image/x-icon
|
Details | |
326 bytes,
image/x-icon
|
Details | |
326 bytes,
image/x-icon
|
Details | |
326 bytes,
image/x-icon
|
Details | |
326 bytes,
image/x-icon
|
Details | |
326 bytes,
image/x-icon
|
Details | |
1.53 KB,
patch
|
db48x
:
review+
neil
:
superreview+
|
Details | Diff | Splinter Review |
When you're autoscrolling, the mouse cursor should be replaced with something indicating the direction in which you're moving. The 9-cursor demo at the bottom of the page in the URL demonstrates what I mean (if you assume you had middle-clicked in the center of the boxes).
Updated•19 years ago
|
Comment 1•19 years ago
|
||
Comment 2•19 years ago
|
||
Comment 3•19 years ago
|
||
Comment 4•19 years ago
|
||
Comment 5•19 years ago
|
||
Comment 6•19 years ago
|
||
Comment 7•19 years ago
|
||
Comment 8•19 years ago
|
||
Updated•19 years ago
|
Attachment #196978 -
Attachment description: Autoscroll cursor (.cur) 12oclock → 12oclock toward north, toward top cursor (.cur)
Comment 9•19 years ago
|
||
These 8 cursors are all 326 bytes, 32bits-based, 2 colors and build/based on
attachment 169214 [details].
Reporter | ||
Comment 10•19 years ago
|
||
This patch probably requires the patch from bug 309547. I'd like to use directional indicators like the ones that are attached, but presently, it looks like setCursor only accepts a small number of cursors. If bug 309547 isn't accepted on the branch, this patch is probably irrelevant, since I'd hope setCursor handles custom cursors for gecko 1.9 and we can use the actual icons.
Attachment #197008 -
Flags: superreview?(neil.parkwaycc.co.uk)
Attachment #197008 -
Flags: review?(db48x)
Reporter | ||
Comment 11•19 years ago
|
||
Oh... I didn't realize those were 32-bit icons. For 256-color compatibility, the current autoscroll indicator uses hand-picked colors, so we may want to do the same here.
Comment 12•19 years ago
|
||
Chris, The attached cursors are 32bits with 2 colors only (black and white). If you want 256 colors cursors, let me know. White color is used for outline, particularly for cases when the cursor hovers over a darker background. If you want these cursors as 16-bits ones, please let me know. No problem.
Comment 13•19 years ago
|
||
Comment on attachment 197008 [details] [diff] [review] Use the "move" cursor This is already a win on Linux possibly because popups capture the mouse. Note: 2-bit, 32-pixel cursors are fine.
Comment 14•19 years ago
|
||
Comment on attachment 197008 [details] [diff] [review] Use the "move" cursor >+ document.defaultView.setCursor("auto"); Also known as "window" - please fix before check in!
Attachment #197008 -
Flags: superreview?(neil.parkwaycc.co.uk) → superreview+
Updated•19 years ago
|
Component: General → XP Apps: GUI Features
Reporter | ||
Updated•19 years ago
|
Whiteboard: [cst:r?]
Comment 15•19 years ago
|
||
Comment on attachment 197008 [details] [diff] [review] Use the "move" cursor simple enough. I wouldn't check it in until it actually works though.
Attachment #197008 -
Flags: review?(db48x) → review+
Reporter | ||
Updated•19 years ago
|
Whiteboard: [cst:r?] → [cst: patch is reviewed]
Comment 16•18 years ago
|
||
Marking dependency on bug 275174
Reporter | ||
Updated•18 years ago
|
Whiteboard: [cst: patch is reviewed] → [cst: patch is reviewed, waiting for backend fixes]
Reporter | ||
Comment 17•16 years ago
|
||
-> somebody else
Assignee: cst → nobody
QA Contact: general → guifeatures
You need to log in
before you can comment on or make changes to this bug.
Description
•