Closed
Bug 1161157
Opened 10 years ago
Closed 10 years ago
[Customizer Launcher] Enable developer mode confirm dialog
Categories
(Firefox OS Graveyard :: Gaia::Customizer, defect)
Tracking
(Not tracked)
RESOLVED
FIXED
People
(Reporter: pdahiya, Assigned: pdahiya)
References
Details
Attachments
(1 file)
For user not having developer mode enabled, customizer launcher should show help dialog and if possible take user to settings app to enable developer mode on. Scope of this bug is to discuss and finalize how to handle this use case.
https://bugzilla.mozilla.org/show_bug.cgi?id=1160235#c1
https://bugzilla.mozilla.org/show_bug.cgi?id=1160235#c2
https://bugzilla.mozilla.org/show_bug.cgi?id=1160235#c3
Assignee | ||
Updated•10 years ago
|
Blocks: spark-customizer-launcher
Assignee | ||
Comment 1•10 years ago
|
||
Setting NI flag for Doug and Justin to help decide next steps.
Flags: needinfo?(jdarcangelo)
Flags: needinfo?(drs)
Comment 2•10 years ago
|
||
We'll have to solve this problem in several apps, so let's get the discussion going on in here. I'm needinfoing Mike for visibility of this topic. Mike, no info is actually needed, though your discussion and opinion is welcome.
I'm not optimistic about us being able to toggle developer mode on-device -- despite how great that would be for UX -- based on how the conversation in bug 1160235 is going. Remember that developer mode requires the device to be rooted, which no matter how we look at this, is not something we're guaranteed to have.
Thus, regardless of what happens in bug 1160235, we'll have to present information to the user about how to toggle developer mode, so that seems like a good start to me. I would suggest making an overlay like this:
"
Developer mode not enabled
---
To gain access to this and other exciting customization features, you must enable developer mode.
(Cancel) (More Info)
"
The "Cancel" button would close the app. The "More Info" button would just link to a web page, probably with a video, explaining how to enable developer mode, and what the risks are.
I think that it would be advantageous to turn this overlay into a Bower package, or perhaps even a WC, which every app that needs it can import. We will need someone to drive this, so I think that starting with the Customizer Launcher is a good place to prototype it.
The actual overlay itself definitely requires some UX input. Jacqueline, what are your thoughts?
Flags: needinfo?(mhenretty)
Flags: needinfo?(jsavory)
Flags: needinfo?(drs)
Comment 3•10 years ago
|
||
I'm on board with the method proposed in comment 2. It would be nice if the system app could have a way to know if the device was rooted, and be able to toggle developer mode directly (or through an activity to settings perhaps) if so. But I'm not sure how feasible that is.
Flags: needinfo?(mhenretty)
Comment 4•10 years ago
|
||
You don't need to root the device to have developer mode access, you just need a way to set a gecko pref.
One way to do it is to use the "reset and root my phone" feature of the settings app, and then webide can toggle any pref.
Assignee | ||
Comment 5•10 years ago
|
||
As per discussion https://bugzilla.mozilla.org/show_bug.cgi?id=1160235#c11, we are exposing the dev mode state read-only through the navigator.hasFeature() api (Bug 1161677).
CL will check for this feature and show enable developer mode confirm dialog described in #comment 2.
Marking this bug dependent on Bug 1161677
Assignee | ||
Comment 6•10 years ago
|
||
The discussion in https://bugzilla.mozilla.org/show_bug.cgi?id=1160235#c11, points enabling full dev tools mode from device by using Settings app activity.
NI Doug to help if we have a bug open for it. If yes, we should mark dependency on that bug. Thanks!
Flags: needinfo?(drs)
Assignee | ||
Updated•10 years ago
|
Assignee: nobody → pdahiya
Assignee | ||
Comment 7•10 years ago
|
||
Marking dependency on Bug 1163889 to handle continue/next button from 'developer mode confirm dialog' to enable devtool mode.
Assignee | ||
Comment 8•10 years ago
|
||
Hi Justin
Please review attached web component for showing confirm dialog if dev perf on device is not enabled. Thanks!
Attachment #8610798 -
Flags: review?(jdarcangelo)
Updated•10 years ago
|
Blocks: spark-dev-mode
Flags: needinfo?(drs)
Comment 9•10 years ago
|
||
Comment on attachment 8610798 [details] [review]
PR with fix of Bug 1161157
Punam, looks good! I left you some comments in the PR regarding the public API you're exposing on the WC. IMO, this component should try and take care of as much logic as possible on its own without requiring the containing app to make any API calls on the component. That's my only issue with this PR as the rest of the code looks good. Thanks!
Flags: needinfo?(jdarcangelo)
Attachment #8610798 -
Flags: review?(jdarcangelo) → review-
Assignee | ||
Comment 10•10 years ago
|
||
(In reply to Justin D'Arcangelo [:justindarc] from comment #9)
> Comment on attachment 8610798 [details] [review]
> PR with fix of Bug 1161157
>
> Punam, looks good! I left you some comments in the PR regarding the public
> API you're exposing on the WC. IMO, this component should try and take care
> of as much logic as possible on its own without requiring the containing app
> to make any API calls on the component. That's my only issue with this PR as
> the rest of the code looks good. Thanks!
Hi Justin, I have updated the PR with your feedback. WC is handling show/hide in createdCallback eliminating the need to call it from containing app. I have left isPrefEnabled public for apps such as Launcher that needs to check for pref setting before processing the bulk of app logic. Please review. Thanks!
Assignee | ||
Updated•10 years ago
|
Attachment #8610798 -
Flags: review- → review?(jdarcangelo)
Comment 11•10 years ago
|
||
Comment on attachment 8610798 [details] [review]
PR with fix of Bug 1161157
Looks good! One nit I mentioned in the PR that you can address before you land (indentation).
Attachment #8610798 -
Flags: review?(jdarcangelo) → review+
Assignee | ||
Comment 12•10 years ago
|
||
Thanks Justin, fixed indentation and patch landed.
https://github.com/fxos/customizer-launcher/commit/8ff817cf5b0ae0c27d8bb24a837760b2df8edbc5
Assignee | ||
Updated•10 years ago
|
Status: NEW → RESOLVED
Closed: 10 years ago
Resolution: --- → FIXED
Updated•7 years ago
|
Flags: needinfo?(jsavory)
You need to log in
before you can comment on or make changes to this bug.
Description
•