code review 3433042: godoc: add separators between quick reference topics

godoc: add separators between quick reference topics

[adg, 2010-12-03 03:14:18]

Hello bradfitz (cc: golang-dev@googlegroups.com),

I'd like you to review this change.

[dangabrad, 2010-12-03 03:36:27]

LGTM
On Dec 2, 2010 7:14 PM, <adg@golang.org> wrote:
> Reviewers: bradfitz,
>
> Message:
> Hello bradfitz (cc: golang-dev@googlegroups.com),
>
> I'd like you to review this change.
>
>
> Description:
> godoc: add separators between quick reference topics
>
> Please review this at http://codereview.appspot.com/3433042/
>
> Affected files:
> M doc/all.css
> M lib/godoc/godoc.html
>
>
> Index: doc/all.css
> ===================================================================
> --- a/doc/all.css
> +++ b/doc/all.css
> @@ -92,6 +92,9 @@
> #nav-main .quickref {
> color: #444;
> }
> +#nav-main .quickref .sep {
> + color: #999;
> +}
> #search {
> width: 100px;
> margin-left: 0.5em;
> Index: lib/godoc/godoc.html
> ===================================================================
> --- a/lib/godoc/godoc.html
> +++ b/lib/godoc/godoc.html
> @@ -25,13 +25,13 @@
> <form method="GET" action="/search">
> {.section PkgRoots}
> {.repeated section PkgRoots}
> - <a href="/pkg/{@|html-esc}">{@|html-esc}</a>
> + <a href="/pkg/{@|html-esc}">{@|html-esc}</a> <span class="sep">|
> </span>
> {.end}
> {.or}
> References:
> {.end}
> - <a href="/pkg/">Packages</a>
> - <a href="/cmd/">Commands</a>
> + <a href="/pkg/">Packages</a> <span class="sep">|</span>
> + <a href="/cmd/">Commands</a> <span class="sep">|</span>
> <a href="/doc/go_spec.html">Specification</a>
> <input id="search" type="text" name="q" value="{.section
> Query}{Query|html-esc}{.or}code search{.end}" class="{.section
> Query}{.or}inactive{.end}" />
> </form>
>
>

[adg, 2010-12-03 03:38:04]

*** Submitted as http://code.google.com/p/go/source/detail?r=e9574cbe0ad9 ***

godoc: add separators between quick reference topics

R=bradfitz, dangabrad
CC=golang-dev
http://codereview.appspot.com/3433042