Open Bug 309537 Opened 19 years ago Updated 15 years ago

When autoscrolling, cursor should indicate current scrolling direction

Categories

(SeaMonkey :: UI Design, defect)

x86
All
defect
Not set
normal

Tracking

(Not tracked)

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)

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).
Attached image 2oclock cursor (.cur)
Attached image 3oclock (.cur) cursor
Attached image 5oclock (.cur) cursor
Attachment #196978 - Attachment description: Autoscroll cursor (.cur) 12oclock → 12oclock toward north, toward top cursor (.cur)
These 8 cursors are all 326 bytes, 32bits-based, 2 colors and build/based on
attachment 169214 [details].
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)
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.
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 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 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+
Component: General → XP Apps: GUI Features
Blocks: 280331
Whiteboard: [cst:r?]
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+
Whiteboard: [cst:r?] → [cst: patch is reviewed]
Depends on: 275174
Marking dependency on bug 275174
Whiteboard: [cst: patch is reviewed] → [cst: patch is reviewed, waiting for backend fixes]
-> somebody else
Assignee: cst → nobody
QA Contact: general → guifeatures
Component: XP Apps: GUI Features → UI Design
You need to log in before you can comment on or make changes to this bug.