881807 - Simplify 'advanced settings' part of options panel

Status: RESOLVED FIXED

(DevTools :: Framework, defect)

Description

[Joe Walker [:jwalker]]

There are too many * and so on. How about:

----

[ ] Enable chrome debugging
[ ] Enable remote debugging
[ ] Disable JavaScript
    (This session only, will reload page)

----

When either of the first 2 are enabled we show an alert that says 'need to restart'.

The definition of 'session' is vague, but probably good enough to ensure people are not surprised.

Thoughts?

Comment 1

[Girish Sharma [:Optimizer]]

I agree. Although I think that the double dagger I added in bug 872670 might be actually useful (removes redundant text in case of multiple non supported tools) and its not an always visible thing too.

Comment 2

[Mike Ratcliffe [:miker] [:mratcliffe] [:mikeratcliffe]]

Enable chrome debugging and enable remote debugging both require a browser restart ... how would you like us to represent that?

Comment 3

[Girish Sharma [:Optimizer]]

(In reply to Michael Ratcliffe [:miker] [:mratcliffe] from comment #2)
> Enable chrome debugging and enable remote debugging both require a browser
> restart ... how would you like us to represent that?

We have many options:
- A notification door hanger, just like when you install a normal addon, it says restart now.
- A notification panel on toolbox with a button to restart
- A div at the bottom, just like IE's notification system, with a button to restart.

Comment 4

[Mike Ratcliffe [:miker] [:mratcliffe] [:mikeratcliffe]]

(In reply to Girish Sharma [:Optimizer] from comment #3)
> (In reply to Michael Ratcliffe [:miker] [:mratcliffe] from comment #2)
> > Enable chrome debugging and enable remote debugging both require a browser
> > restart ... how would you like us to represent that?
> 
> We have many options:
> - A notification door hanger, just like when you install a normal addon, it
> says restart now.
> - A notification panel on toolbox with a button to restart
> - A div at the bottom, just like IE's notification system, with a button to
> restart.

They would all happen after the checkbox is clicked and I think that being forewarned is better. I think that mixing the two methods will be fine, especially because "this session only" is only ever likely to be linked with one option:
[ ] Enable chrome debugging *
[ ] Enable remote debugging *
[ ] Disable JavaScript
    (This session only, will reload page)

* Requires restart

Comment 6

[Joe Walker [:jwalker]]

(In reply to Michael Ratcliffe [:miker] [:mratcliffe] from comment #4)
> (In reply to Girish Sharma [:Optimizer] from comment #3)
> > (In reply to Michael Ratcliffe [:miker] [:mratcliffe] from comment #2)
> > > Enable chrome debugging and enable remote debugging both require a browser
> > > restart ... how would you like us to represent that?
> > 
> > We have many options:
> > - A notification door hanger, just like when you install a normal addon, it
> > says restart now.
> > - A notification panel on toolbox with a button to restart
> > - A div at the bottom, just like IE's notification system, with a button to
> > restart.
> 
> They would all happen after the checkbox is clicked and I think that being
> forewarned is better.

How does being forewarned make a difference? You click the checkbox, get a popup, and maybe a restart link appears. How is your life better/worse off compared to if you knew that you would have to restart before you clicked the checkbox?

On the other hand, having the text there slows everyone down because it gives you more to read.

Comment 7

[Mike Ratcliffe [:miker] [:mratcliffe] [:mikeratcliffe]]

(In reply to Joe Walker [:jwalker] from comment #6)
> (In reply to Michael Ratcliffe [:miker] [:mratcliffe] from comment #4)
> > (In reply to Girish Sharma [:Optimizer] from comment #3)
> > > (In reply to Michael Ratcliffe [:miker] [:mratcliffe] from comment #2)
> > > > Enable chrome debugging and enable remote debugging both require a browser
> > > > restart ... how would you like us to represent that?
> > > 
> > > We have many options:
> > > - A notification door hanger, just like when you install a normal addon, it
> > > says restart now.
> > > - A notification panel on toolbox with a button to restart
> > > - A div at the bottom, just like IE's notification system, with a button to
> > > restart.
> > 
> > They would all happen after the checkbox is clicked and I think that being
> > forewarned is better.
> 
> How does being forewarned make a difference? You click the checkbox, get a
> popup, and maybe a restart link appears. How is your life better/worse off
> compared to if you knew that you would have to restart before you clicked
> the checkbox?
> 
> On the other hand, having the text there slows everyone down because it
> gives you more to read.

My point was that people have a tendency not to read notifications and may be surprised when settings do not immediately take effect.

Comment 8

[Heather Arthur [:harth]]

+1 for just adding a Restart link after toggling, and not cluttering up the panel before.

Maybe just add text right below the pref after toggling:
"This option requires a restart [Restart Now]"

Comment 9

[Joe Walker [:jwalker]]

(In reply to Michael Ratcliffe [:miker] [:mratcliffe] from comment #7)
> (In reply to Joe Walker [:jwalker] from comment #6)
> > (In reply to Michael Ratcliffe [:miker] [:mratcliffe] from comment #4)
> > > (In reply to Girish Sharma [:Optimizer] from comment #3)
> > > > (In reply to Michael Ratcliffe [:miker] [:mratcliffe] from comment #2)
> > > > > Enable chrome debugging and enable remote debugging both require a browser
> > > > > restart ... how would you like us to represent that?
> > > > 
> > > > We have many options:
> > > > - A notification door hanger, just like when you install a normal addon, it
> > > > says restart now.
> > > > - A notification panel on toolbox with a button to restart
> > > > - A div at the bottom, just like IE's notification system, with a button to
> > > > restart.
> > > 
> > > They would all happen after the checkbox is clicked and I think that being
> > > forewarned is better.
> > 
> > How does being forewarned make a difference? You click the checkbox, get a
> > popup, and maybe a restart link appears. How is your life better/worse off
> > compared to if you knew that you would have to restart before you clicked
> > the checkbox?
> > 
> > On the other hand, having the text there slows everyone down because it
> > gives you more to read.
> 
> My point was that people have a tendency not to read notifications and may
> be surprised when settings do not immediately take effect.

Right, but I'm not sure we can't use "people don't read" as a reason to have more text.

Heather's solution is nice - possibly harder than an alert, but would look nicer, and due to the link, be easier to use.

Comment 10

[Girish Sharma [:Optimizer]]

I was thinking this : 
- A div at the bottom, just like IE's notification system, with a button to restart.

(the third option from my suggestions)

This should be :
- Catching enough  to get attention.
- Has the restart button to be helpful
- Is temporary, less clutter, clearly laid out to say that "restart is required"

Comment 11

[Joe Walker [:jwalker]]

(In reply to Girish Sharma [:Optimizer] from comment #10)
> I was thinking this : 
> - A div at the bottom, just like IE's notification system, with a button to
> restart.
> 
> (the third option from my suggestions)
> 
> This should be :
> - Catching enough  to get attention.
> - Has the restart button to be helpful
> - Is temporary, less clutter, clearly laid out to say that "restart is
> required"

Is there an advantage to putting the div at the bottom? Isn't that just further for your eye and mouse to travel?

How about this combination of all the ideas:

[ ] Disable JavaScript
    (This session only, will reload page)
[ ] Enable chrome debugging
[ ] Enable remote debugging
    <Browser restart required [Restart]>

(The section in <...> is hidden unless a browser restart is required, i.e. when the state of either of the 2 'enable ... debugging' checkboxes isn't what it was to start with)

Comment 12

[Mike Ratcliffe [:miker] [:mratcliffe] [:mikeratcliffe]]

(In reply to Joe Walker [:jwalker] from comment #11)
> (In reply to Girish Sharma [:Optimizer] from comment #10)
> > I was thinking this : 
> > - A div at the bottom, just like IE's notification system, with a button to
> > restart.
> > 
> > (the third option from my suggestions)
> > 
> > This should be :
> > - Catching enough  to get attention.
> > - Has the restart button to be helpful
> > - Is temporary, less clutter, clearly laid out to say that "restart is
> > required"
> 
> Is there an advantage to putting the div at the bottom? Isn't that just
> further for your eye and mouse to travel?
> 
> How about this combination of all the ideas:
> 
> [ ] Disable JavaScript
>     (This session only, will reload page)
> [ ] Enable chrome debugging
> [ ] Enable remote debugging
>     <Browser restart required [Restart]>
> 
> (The section in <...> is hidden unless a browser restart is required, i.e.
> when the state of either of the 2 'enable ... debugging' checkboxes isn't
> what it was to start with)

+1 to Joe's suggestion.

Comment 13

[Girish Sharma [:Optimizer]]

Agreed to Joe. On it now.

Comment 14

[Girish Sharma [:Optimizer]]

Attachment: patch v1.0

Makes the changes as required as per the discussions above.

Comment 15

[Girish Sharma [:Optimizer]]

Attachment: patch v1.1

forgot to save osx/toolbox.css before qref'ing

Comment 16

[Girish Sharma [:Optimizer]]

Note that this bug and bug 872670 should be landed in the same push, 872670 first then this

Comment 17

[Phil Ringnalda (:philor)]

https://hg.mozilla.org/integration/fx-team/rev/7c1c4bb1263a

Comment 18

[Victor Porof [:vporof][:vp]]

https://hg.mozilla.org/mozilla-central/rev/7c1c4bb1263a