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)

x86
Windows 8.1
defect

Tracking

(Not tracked)

RESOLVED WORKSFORME

People

(Reporter: jwilde, Unassigned)

References

Details

(Whiteboard: feature=work)

Attachments

(1 file)

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.
Assignee: hello → nobody
Attached patch wipSplinter Review
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.)
Depends on: 636564
Whiteboard: metro-beta
Attachment #657478 - Attachment is patch: true
Product: Firefox → Firefox for Metro
Version: unspecified → Trunk
Whiteboard: metro-beta → [metro-mvp]
Status: ASSIGNED → NEW
Keywords: feature
Summary: Utilize native scrollbars for touch scrollbars → Implement scroll indicators for touch scrolling
No longer blocks: 775718
No longer depends on: 636564
See Also: → 636564
We can't implement this until we can theme anonymous scroll bars.
Blocks: 775718
Depends on: 636564
Priority: -- → P1
Whiteboard: [metro-mvp] → [metro-mvp] [LOE:1]
Priority: P1 → --
Blocks: 831984
Whiteboard: [metro-mvp] [LOE:1] → [metro-mvp] [LOE:1] feature=work
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
Depends on: metro-omtc
Priority: -- → P3
Component: General → Browser
Blocks: 881081
No longer blocks: 775718
No longer depends on: metro-omtc
Status: NEW → RESOLVED
Closed: 11 years ago
Resolution: --- → WORKSFORME
No longer blocks: 831984
OS: Windows 8 Metro → Windows 8.1
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: