Closed
Bug 777610
Opened 12 years ago
Closed 11 years ago
Work - Implement scroll indicators for touch scrolling
Categories
(Firefox for Metro Graveyard :: Browser, defect, P3)
Tracking
(Not tracked)
RESOLVED
WORKSFORME
People
(Reporter: jwilde, Unassigned)
References
Details
(Whiteboard: feature=work)
Attachments
(1 file)
9.39 KB,
patch
|
Details | Diff | Splinter Review |
Rather than having the main dragger track and update the position of xul:box elements, it would be more efficient to just style the normal scrollbars to look like touch scrollbars.
This also allows us to more easily support scroll bars in scrollable frames in the future.
From there, to implement 775718, we need to selectively remove the style when the user is not using touch and hide them when the user is not panning.
Reporter | ||
Updated•12 years ago
|
Assignee: hello → nobody
Reporter | ||
Comment 1•12 years ago
|
||
Here's the initial WIP patch that I put together for this. It displays touch-style scrollbars everywhere.
In chrome, it works well and superimposes tiny scrollbars over the scrollable region. In content, it pushes page contents sideways and isn't overlayed. It doesn't look terrible, but not great either.
To land this, we'd need to:
- Add support for tracking whether we're in touch or mouse and display this or native scrollbar styling accordingly.
- Support proper overlaying of the scrollbars on content. (Possible solutions in bug 636564.)
- Add CSS transitions to the scrollbar. (Possible solutions in bug 636564.)
Updated•12 years ago
|
Attachment #657478 -
Attachment is patch: true
Updated•12 years ago
|
Product: Firefox → Firefox for Metro
Version: unspecified → Trunk
Updated•12 years ago
|
Whiteboard: metro-beta → [metro-mvp]
Updated•12 years ago
|
Status: ASSIGNED → NEW
Keywords: feature
Summary: Utilize native scrollbars for touch scrollbars → Implement scroll indicators for touch scrolling
Updated•12 years ago
|
Comment 3•12 years ago
|
||
We can't implement this until we can theme anonymous scroll bars.
Updated•12 years ago
|
Priority: -- → P1
Updated•12 years ago
|
Whiteboard: [metro-mvp] → [metro-mvp] [LOE:1]
Updated•12 years ago
|
Priority: P1 → --
Updated•12 years ago
|
Blocks: 831984
Whiteboard: [metro-mvp] [LOE:1] → [metro-mvp] [LOE:1] feature=work
Updated•12 years ago
|
Keywords: feature
Summary: Implement scroll indicators for touch scrolling → Work - Implement scroll indicators for touch scrolling
Whiteboard: [metro-mvp] [LOE:1] feature=work → feature=work
Updated•12 years ago
|
Depends on: metro-omtc
Updated•12 years ago
|
Priority: -- → P3
Updated•12 years ago
|
Component: General → Browser
Updated•11 years ago
|
No longer depends on: metro-omtc
Updated•11 years ago
|
Status: NEW → RESOLVED
Closed: 11 years ago
Resolution: --- → WORKSFORME
Assignee | ||
Updated•10 years ago
|
OS: Windows 8 Metro → Windows 8.1
You need to log in
before you can comment on or make changes to this bug.
Description
•