code review 6449105: godoc: make id attributes unique

godoc: make id attributes unique

Some browsers (e.g. IE9) ignore the case of 'id' attributes
which can lead to conflicts. Prefix non-generated 'id's with
"pkg-" to make them different from any generated attribute.

Also: Added missing entry for "Other packages" to top-level
index.

Fixes issue 3851.

[gri, 2012-08-07 20:15:55]

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

I'd like you to review this change to
https://code.google.com/p/go

[dsymonds, 2012-08-07 23:44:47]

LGTM

[adg, 2012-08-08 00:09:36]

NOT LGTM

Some browsers don't like underscores in ids.

[gri, 2012-08-08 00:30:54]

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

Please take another look.

[dsymonds, 2012-08-08 00:33:21]

Per http://www.w3schools.com/tags/att_standard_id.asp, id attributes
  * Must begin with a letter A-Z or a-z
  * Can be followed by: letters (A-Za-z), digits (0-9), hyphens ("-"), and
underscores ("_")
  * In HTML, all values are case-insensitive

If we are trying to avoid underscores, maybe these should start with "z-" or
"toc-" or something.

[adg, 2012-08-08 00:37:50]

+1 for "toc-" or "godoc-"

[gri, 2012-08-08 00:42:27]

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

Please take another look.

[adg, 2012-08-08 00:44:32]

LGTM

[gri, 2012-08-08 00:46:03]

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

godoc: make id attributes unique

Some browsers (e.g. IE9) ignore the case of 'id' attributes
which can lead to conflicts. Prefix non-generated 'id's with
"pkg-" to make them different from any generated attribute.

Also: Added missing entry for "Other packages" to top-level
index.

Fixes issue 3851.

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