DevTools: improved layout for Kbd Shortcuts & Settings

DevTools: improved layout for Kbd Shortcuts & Settings

Increase information density and address broken layout.

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

[paulirish, 2015-08-11 18:27:27]

paulirish@chromium.org changed reviewers:
+ lushnikov@chromium.org

[paulirish, 2015-08-11 18:27:28]

screenshot in bug. PTAL

[paulirish, 2015-08-12 23:37:13]

where u at bro

[lushnikov, 2015-08-13 04:21:37]

On 2015/08/12 23:37:13, paulirish wrote:
> where u at bro

Hey Paul!

Two columns are great! A few things:
1. I liked "<shortcut> : <description>" more then newly introduced
"<description> : <shortcut>". This is due to it's easier to scan descriptions
with eyes when they 
all start at the same vertical column. Can we leave that as-is?
2. There's an empty "Console" section in shortcuts - can we fix this while we're
here?

[lushnikov, 2015-08-13 04:21:56]

nit comment

https://codereview.chromium.org/1291483002/diff/20001/Source/devtools/front_e...
File Source/devtools/front_end/ui/helpScreen.css (right):

https://codereview.chromium.org/1291483002/diff/20001/Source/devtools/front_e...
Source/devtools/front_end/ui/helpScreen.css:108: 
stray line

[paulirish, 2015-08-13 20:56:42]

On 2015/08/13 at 04:21:37, lushnikov wrote:
> On 2015/08/12 23:37:13, paulirish wrote:
> > where u at bro
> 
> Hey Paul!
> 
> Two columns are great! A few things:
> 1. I liked "<shortcut> : <description>" more then newly introduced
"<description> : <shortcut>". This is due to it's easier to scan descriptions
with eyes when they 
> all start at the same vertical column. Can we leave that as-is?

Sure. done.

> 2. There's an empty "Console" section in shortcuts - can we fix this while
we're here?


Looked into this the other day. It's been like this day 1.
Unfortunately it's more work than I can tackle right now, but we've isolated the
root cause.
Filed as crbug.com/519663


ptal

[paulirish, 2015-08-15 19:16:26]

okay. patch uploaded with our linux/windows adjustments.

[pfeldman, 2015-08-17 17:55:33]

pfeldman@chromium.org changed reviewers:
+ pfeldman@chromium.org

[pfeldman, 2015-08-17 17:55:33]

lgtm

[pfeldman, 2015-08-17 17:55:37]

The CQ bit was checked by pfeldman@chromium.org

[commit-bot: I haz the power, 2015-08-17 17:56:06]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1291483002/100001
View timeline at
 https://chromium-cq-status.appspot.com/patch-timeline/1291483002/100001

[commit-bot: I haz the power, 2015-08-17 20:29:18]

The CQ bit was unchecked by commit-bot@chromium.org

[commit-bot: I haz the power, 2015-08-17 20:29:20]

Try jobs failed on following builders:
  linux_chromium_rel_ng on tryserver.chromium.linux (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.linux/builders/linux_chromium_...)

[paulirish, 2015-08-17 21:00:04]

The CQ bit was checked by paulirish@chromium.org

[commit-bot: I haz the power, 2015-08-17 21:00:30]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1291483002/100001
View timeline at
 https://chromium-cq-status.appspot.com/patch-timeline/1291483002/100001

[commit-bot: I haz the power, 2015-08-17 23:50:51]

The CQ bit was unchecked by commit-bot@chromium.org

[commit-bot: I haz the power, 2015-08-17 23:50:52]

Try jobs failed on following builders:
  linux_chromium_rel_ng on tryserver.chromium.linux (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.linux/builders/linux_chromium_...)

[paulirish, 2015-08-20 23:03:57]

The CQ bit was checked by paulirish@chromium.org

[paulirish, 2015-08-20 23:03:57]

The patchset sent to the CQ was uploaded after l-g-t-m from
pfeldman@chromium.org
Link to the patchset: https://codereview.chromium.org/1291483002/#ps120001
(title: "rebase")

[commit-bot: I haz the power, 2015-08-20 23:04:28]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1291483002/120001
View timeline at
 https://chromium-cq-status.appspot.com/patch-timeline/1291483002/120001

[commit-bot: I haz the power, 2015-08-21 02:23:08]

Committed patchset #7 (id:120001) as
https://src.chromium.org/viewvc/blink?view=rev&revision=200951