Implement gesture for activating view source

RESOLVED FIXED in Firefox OS master

Status

RESOLVED FIXED
3 years ago
3 years ago

People

(Reporter: nyee, Assigned: drs)

Tracking

unspecified
FxOS-S6 (04Sep)
ARM
Gonk (Firefox OS)

Firefox Tracking Flags

(feature-b2g:2.5+, b2g-master fixed)

Details

Attachments

(2 attachments)

Comment hidden (empty)
(Reporter)

Updated

3 years ago
Blocks: 1188667
(Reporter)

Updated

3 years ago
Target Milestone: --- → FxOS-S5 (21Aug)
(Assignee)

Comment 1

3 years ago
Mirroring an email conversation we had about this:

> Hi Rob,
> 
> I'm thinking that we can ship the "View Source" feature as being off by default, and enabled in the "Developer" menu section of the Settings app. We would then just use the existing gesture to invoke it, which is 5-taps.
> 
> I don't expect very heavy use of this, and considering it will be disabled by default, I don't think that the gesture to invoke it matters very much. Do you have any thoughts?
> 
> Doug

> Hi Doug...
> 
> Is this a developer setting that's off by default? If so then it shouldn't cause any conflicts. If everyone on this list is ok with it, I'd say the only other remaining group to check with might be accessibility.
> 
> Thanks!
> Rob

> +Yura for a11y
> 
> Rob, yes, exactly.
> 
> Yura, could you take a look at the initial proposal here and tell us what you think?
> 
> Doug

> so the actual 5-taps gesture is fine, with screen reader it is going to be 5-doubletap gesture. One caveat, however, is the target that receives the tap events. I must be itself accessible with the screen reader. If it just bubbles to parent document then it should be fine, since the screen reader user would be able to perform 5-doubletap gesture when the screen reader focus is inside the app document (granted there’s no conflict with elements being actionable themselves).
> 
> hope this helps,
> yura
feature-b2g: --- → 2.5+
(Assignee)

Comment 2

3 years ago
Jacqueline, any thoughts on instead including an additional power menu option if the "View Source" Developer Menu option is enabled?
Component: Gaia → Gaia::System::Window Mgmt
Flags: needinfo?(jsavory)
Whiteboard: planned finished: 3 days
Target Milestone: FxOS-S5 (21Aug) → FxOS-S6 (04Sep)
(Assignee)

Updated

3 years ago
feature-b2g: 2.5+ → ---
(Assignee)

Updated

3 years ago
Keywords: feature
Created attachment 8653193 [details] [review]
[gaia] DouglasSherk:1188673-view-source-menu > mozilla-b2g:master
(Assignee)

Comment 4

3 years ago
Created attachment 8653197 [details] [diff] [review]
Add an option to the sleep menu to "View Source" if the developer setting for this is enabled.
(Assignee)

Updated

3 years ago
Attachment #8653193 - Flags: review?(kevingrandon)
(Assignee)

Updated

3 years ago
Status: NEW → ASSIGNED
Comment on attachment 8653193 [details] [review]
[gaia] DouglasSherk:1188673-view-source-menu > mozilla-b2g:master

I guess this is for the commit here, https://github.com/DouglasSherk/gaia/commit/b3929d5d0cc84349085a9fc322db44866820f537

Probably should loop that in with the actual implementation, but this seems fine to me. Thanks!
Attachment #8653193 - Flags: review?(kevingrandon) → review+
feature-b2g: --- → 2.5+
(Assignee)

Comment 6

3 years ago
https://github.com/mozilla-b2g/gaia/commit/679a3393136653ccdcc603ceb3b4918199f798e3
Status: ASSIGNED → RESOLVED
Last Resolved: 3 years ago
status-b2g-master: --- → fixed
Flags: needinfo?(jsavory)
Resolution: --- → FIXED
Hi Doug, I'm trying to create test cases for this feature.  Is the 5-tap gesture still valid?  if so, where should I tap?  I tapped within the app screen, but that didn't do anything for me.  Thanks!
Flags: needinfo?(drs)
heard back from drs. no 5-time tapping.
Flags: needinfo?(drs)
You need to log in before you can comment on or make changes to this bug.