1188673 - Implement gesture for activating view source

Status: RESOLVED FIXED

(Firefox OS Graveyard :: Gaia::System::Window Mgmt, defect)

Comment 1

[Doug Sherk (:drs) (inactive)]

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

Comment 2

[Doug Sherk (:drs) (inactive)]

Jacqueline, any thoughts on instead including an additional power menu option if the "View Source" Developer Menu option is enabled?

Comment 3

[GitHub Autolander Bot]

Attachment: [gaia] DouglasSherk:1188673-view-source-menu > mozilla-b2g:master

Comment 4

[Doug Sherk (:drs) (inactive)]

Attachment: Add an option to the sleep menu to "View Source" if the developer setting for this is enabled.

Comment 5

[Kevin Grandon :kgrandon]

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!

Comment 6

[Doug Sherk (:drs) (inactive)]

https://github.com/mozilla-b2g/gaia/commit/679a3393136653ccdcc603ceb3b4918199f798e3

Comment 7

[No-Jun Park [:njpark]]

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!

Comment 8

[No-Jun Park [:njpark]]

heard back from drs. no 5-time tapping.