code review 6063054: doc: don't wrap package names in directory listing

doc: don't wrap package names in directory listing

Fixes issue 3522.

[adg, 2012-04-19 20:51:07]

Hello golang-dev@googlegroups.com,

I'd like you to review this change to
https://go.googlecode.com/hg

[dsymonds, 2012-04-19 23:02:39]

It would be clearer and more robust to apply a class to that TD in the
HTML, and then name the TD in the CSS rather than relying on the
particular layout.

[adg, 2012-04-20 13:41:05]

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

Please take another look.

[dsymonds, 2012-04-20 13:42:59]

LGTM

http://codereview.appspot.com/6063054/diff/4001/doc/style.css
File doc/style.css (right):

http://codereview.appspot.com/6063054/diff/4001/doc/style.css#newcode111
doc/style.css:111: div#page.wide table.dir td.name {
I'd probably use "package-name" as the class, but this is fine.

[adg, 2012-04-20 14:04:18]

On 20 April 2012 09:42,  <dsymonds@golang.org> wrote:
> I'd probably use "package-name" as the class, but this is fine.

Its a directory name, not a package name. Hence .dir .name.

[adg, 2012-04-20 14:04:18]

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

doc: don't wrap package names in directory listing

Fixes issue 3522.

R=golang-dev, dsymonds
CC=golang-dev
http://codereview.appspot.com/6063054

[dsymonds, 2012-04-20 23:01:26]

On Apr 21, 2012 12:04 AM, "Andrew Gerrand" <adg@golang.org> wrote:

> On 20 April 2012 09:42,  <dsymonds@golang.org> wrote:
> > I'd probably use "package-name" as the class, but this is fine.
>
> Its a directory name, not a package name. Hence .dir .name.

See the CL description for the source of my confusion.  ;-)