Remove the autosizing modes in favor of a dev site link.

Remove the autosizing modes in favor of a dev site link.

Per discussion on http://crbug.com/319092, the autosizing configurations
are going to be confusing to users. This change switches back to just
a single "text autosizing" checkbox that emulates Chrome on Android's
text autosizing behavior. This change also adds a link at the bottom of the
screen emulation page for more information on the developers.google.com site.

A followup patch will add help icons next to some of the checkboxes for
light-weight contextual information.

Preview screenshot: http://pr.gg/devsitelink.png

BUG=319092
Committed: https://src.chromium.org/viewvc/blink?view=rev&revision=163062

[skobes, 2013-11-20 20:06:41]

lgtm

https://codereview.chromium.org/77723002/diff/1/Source/devtools/front_end/Ove...
File Source/devtools/front_end/OverridesSupport.js (right):

https://codereview.chromium.org/77723002/diff/1/Source/devtools/front_end/Ove...
Source/devtools/front_end/OverridesSupport.js:200: * Chromium on mobile uses a
device scale adjustment for fonts used in text autosizing for
I would leave this comment as "Android", since it is Android-specific.

[pdr., 2013-11-21 06:40:42]

Updated! PTAL

Questions for reviewers:
1) Do you like the plan of adding a link to the dev site and a small contextual
help icon.
2) Should we add a link to the bottom of every one of the emulation pages
instead of just the screen emulation page?
3) [minor, ux] At the moment the link doesn't stick to the bottom of the page.
If that's important, we'll need to wrap the upper half of each of the emulation
tabs in a flexbox to push the dev site link to the bottom of the page.

[skobes, 2013-11-21 19:33:48]

> 1) Do you like the plan of adding a link to the dev site and a small
contextual
> help icon.

yes

> 2) Should we add a link to the bottom of every one of the emulation pages
> instead of just the screen emulation page?

Do we have interesting / useful help content for the other pages?

> 3) [minor, ux] At the moment the link doesn't stick to the bottom of the page.
> If that's important, we'll need to wrap the upper half of each of the
emulation
> tabs in a flexbox to push the dev site link to the bottom of the page.

I think it should stay next to the checkboxes, not stick to the bottom...

[pdr., 2013-11-25 03:57:14]

@pfeldman, @johnme, may I have your opinion of this patch?

[pfeldman, 2013-11-26 16:21:07]

red patches always lgtm!

[johnme, 2013-11-26 16:41:20]

> 1) Do you like the plan of adding a link to the dev site and a small
contextual
> help icon.

lgtm

> 2) Should we add a link to the bottom of every one of the emulation pages
> instead of just the screen emulation page?

Screen seems most useful (as settings like Text Autosizing are very subtle).

> 3) [minor, ux] At the moment the link doesn't stick to the bottom of the page.

Don't mind.

[johnme, 2013-11-26 16:41:26]

https://codereview.chromium.org/77723002/diff/70002/Source/devtools/front_end...
File Source/devtools/front_end/OverridesView.js (right):

https://codereview.chromium.org/77723002/diff/70002/Source/devtools/front_end...
Source/devtools/front_end/OverridesView.js:264: // - use text autosizing.
While you're editing this, the text autosizing value seems to always be "1". Is
it worth listing it for each device?

[pdr., 2013-12-03 03:34:17]

On 2013/11/26 16:41:26, johnme wrote:
>
https://codereview.chromium.org/77723002/diff/70002/Source/devtools/front_end...
> File Source/devtools/front_end/OverridesView.js (right):
> 
>
https://codereview.chromium.org/77723002/diff/70002/Source/devtools/front_end...
> Source/devtools/front_end/OverridesView.js:264: // - use text autosizing.
> While you're editing this, the text autosizing value seems to always be "1".
Is
> it worth listing it for each device?

Good point, fixed. We now have to support 3 or 4 values in the string passed to
WebInspector.OverridesSupport.DeviceMetrics.parseSetting but I think it's
cleaner overall.

[commit-bot: I haz the power, 2013-12-03 03:34:36]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/pdr@chromium.org/77723002/160001

[commit-bot: I haz the power, 2013-12-03 05:48:08]

Change committed as 163062