Small net-internals UI tweaks for android.

Minor net-internals UI tweaks for android.

Set max-width of a couple text input fields, don't bold active tab name, shorten some tab names, and reduce list spacing.

BUG=139866
Committed: http://src.chromium.org/viewvc/chrome?view=rev&revision=158389

[eroman, 2012-09-24 19:10:01]

lgtm

On Mon, Sep 24, 2012 at 11:37 AM, <mmenke@chromium.org> wrote:

> Reviewers: eroman,
>
> Description:
> Minor net-internals UI tweaks for android.
>
> Set max-width of a couple text input fields, don't bold active tab name,
> shorten
> some tab names, and reduce list spacing.
>
> BUG=139866
>
> Please review this at
http://codereview.chromium.**org/10959073/<http://codereview.chromium.org/109...
>
> SVN Base:
svn://svn.chromium.org/chrome/**trunk/src/<http://svn.chromium.org/chrome/trunk/src/>
>
> Affected files:
>   M     chrome/browser/resources/net_**internals/category_tabs.html
>   M     chrome/browser/resources/net_**internals/export_view.html
>   M     chrome/browser/resources/net_**internals/hsts_view.html
>   M     chrome/browser/resources/net_**internals/main.css
>   M     chrome/browser/resources/net_**internals/tab_switcher_view.**css
>
>
> Index: chrome/browser/resources/net_**internals/category_tabs.html
> ==============================**==============================**=======
> --- chrome/browser/resources/net_**internals/category_tabs.html
> (revision 158318)
> +++ chrome/browser/resources/net_**internals/category_tabs.html
> (working copy)
> @@ -9,8 +9,8 @@
>    <a href="#dns" id=tab-handle-dns>DNS</a>
>    <a href="#sockets" id=tab-handle-sockets>Sockets<**/a>
>    <a href="#spdy" id=tab-handle-spdy>SPDY</a>
> -  <a href="#httpPipeline" id=tab-handle-http-pipeline>**HTTP
> Pipelining</a>
> -  <a href="#httpCache" id=tab-handle-http-cache>HTTP Cache</a>
> +  <a href="#httpPipeline" id=tab-handle-http-pipeline>**Pipelining</a>
> +  <a href="#httpCache" id=tab-handle-http-cache>**Cache</a>
>    <!-- Tab is only shown on Windows -->
>    <a href="#serviceProviders" id=tab-handle-service-**providers
> style="display: none;">SPIs</a>
>    <a href="#tests" id=tab-handle-tests>Tests</a>
> Index: chrome/browser/resources/net_**internals/export_view.html
> ==============================**==============================**=======
> --- chrome/browser/resources/net_**internals/export_view.html
> (revision 158318)
> +++ chrome/browser/resources/net_**internals/export_view.html
> (working copy)
> @@ -15,6 +15,10 @@
>    font-weight: bold;
>  }
>
> +#export-view-user-comments {
> +  max-width: 100%;
> +}
> +
>  </style>
>
>  <div id=export-view-tab-content class=content-box>
> Index: chrome/browser/resources/net_**internals/hsts_view.html
> ==============================**==============================**=======
> --- chrome/browser/resources/net_**internals/hsts_view.html
> (revision 158318)
> +++ chrome/browser/resources/net_**internals/hsts_view.html
> (working copy)
> @@ -21,8 +21,8 @@
>      Domain: <input type=text id=hsts-view-add-input type="url"
>               placeholder="example.com"><br>
>      Include subdomains: <input type="checkbox"
> id=hsts-view-check-input><br>
> -    Public key fingerprints: <input type=text id=hsts-view-add-pins
> style="width: 25em;"><br>
> -    <p style="font-size: small; color: gray; font-style: italic;
> margin-left: 2em; max-width: 40em;">(public key fingerprints are comma
> separated and consist of the hash function followed by a foreslash and the
> base64 encoded fingerprint, for example <tt>sha1/**
> Guzek9lMwR3KeIS8wwS9gBvVtIg=</**tt>)</p>
> +    Public key fingerprints: <input type=text id=hsts-view-add-pins
> style="width: 25em; max-width: 100%;"><br>
> +    <p style="font-size: small; color: gray; font-style: italic;
> margin-left: 2em; max-width: 40em; max-width: 100%;">(public key
> fingerprints are comma separated and consist of the hash function followed
> by a foreslash and the base64 encoded fingerprint, for example <tt>sha1/**
> Guzek9lMwR3KeIS8wwS9gBvVtIg=</**tt>)</p>
>      <input type=submit value="Add" id=hsts-view-add-submit>
>    </form>
>
> Index: chrome/browser/resources/net_**internals/main.css
> ==============================**==============================**=======
> --- chrome/browser/resources/net_**internals/main.css     (revision
> 158318)
> +++ chrome/browser/resources/net_**internals/main.css     (working copy)
> @@ -10,6 +10,10 @@
>    box-sizing: border-box;
>  }
>
> +ul {
> +  padding-left: 2em;
> +}
> +
>  /* This class is used to create the splitter widget in
>     ResizbleVerticalSplitView */
>  .vertical-splitter {
> Index: chrome/browser/resources/net_**internals/tab_switcher_view.**css
> ==============================**==============================**=======
> --- chrome/browser/resources/net_**internals/tab_switcher_view.**css
>    (revision 158318)
> +++ chrome/browser/resources/net_**internals/tab_switcher_view.**css
>    (working copy)
> @@ -26,7 +26,6 @@
>    background: white;
>  }
>
> -
>  .tab-switcher-view a:visited,
>  .tab-switcher-view a  {
>    color: black;
> @@ -34,5 +33,4 @@
>
>  .tab-switcher-view a.selected {
>    color: rgb(221, 75, 57);
> -  font-weight: bold;
>  }
>
>
>

[commit-bot: I haz the power, 2012-09-24 19:10:39]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/mmenke@chromium.org/10959073/3002

[commit-bot: I haz the power, 2012-09-24 21:27:01]

Change committed as 158389