From 8b89e95de3193f7363c912845b43a7c72d3e1d95 Mon Sep 17 00:00:00 2001 From: Chris Broadfoot Date: Fri, 6 Jan 2017 10:27:24 -0800 Subject: [PATCH] godoc/dl: change "All versions" to "Archived versions" This change makes each release show only once on the whole page. The current stable releases are not shown under the archive. This change also fixes expandos to work correctly on IDs that are not valid CSS selectors (i.e., "#go1.7.4") to ensure that the file listing for a given version is expanded. Updates golang/go#17574. Change-Id: I7ff1041be3e0072286772ffa545a124764f81945 Reviewed-on: https://go-review.googlesource.com/34911 Reviewed-by: Brad Fitzpatrick --- godoc/dl/dl.go | 79 +++++++++++++++++++++++------------------- godoc/dl/dl_test.go | 6 ++-- godoc/dl/tmpl.go | 10 +++--- godoc/static/godocs.js | 3 +- godoc/static/static.go | 3 +- 5 files changed, 55 insertions(+), 46 deletions(-) diff --git a/godoc/dl/dl.go b/godoc/dl/dl.go index d86e784d..7c1b15cb 100644 --- a/godoc/dl/dl.go +++ b/godoc/dl/dl.go @@ -255,44 +255,53 @@ func filesToReleases(fs []File) (stable, unstable, archive []Release) { if r == nil { return } - if r.Stable { - // Show all stable versions in the archive - archive = append(archive, *r) - - // Show the most two recent major version under "Stable". - if len(stable) < 2 { - if len(stable) == 0 { - // Most recent stable version. - stableMaj, stableMin, _ = parseVersion(r.Version) - } else if maj, _, _ := parseVersion(r.Version); maj == stableMaj { - // Older minor version of most recent major version. - return - } - // Split the file list into primary/other ports for the stable releases. - // NOTE(cbro): This is only done for stable releases because maintaining the historical - // nature of primary/other ports for older versions is infeasible. - // If freebsd is considered primary some time in the future, we'd not want to - // mark all of the older freebsd binaries as "primary". - // It might be better if we set that as a flag when uploading. - r.SplitPortTable = true - r.Visible = true // Toggle open all stable releases. - stable = append(stable, *r) - } else if len(stable) == 1 { - // Show the second most recent major version (i.e., security release) + if !r.Stable { + if len(unstable) != 0 { + // Only show one (latest) unstable version. + return } + maj, min, _ := parseVersion(r.Version) + if maj < stableMaj || maj == stableMaj && min <= stableMin { + // Display unstable version only if newer than the + // latest stable release. + return + } + unstable = append(unstable, *r) + } + + // Reports whether the release is the most recent minor version of the + // two most recent major versions. + shouldAddStable := func() bool { + if len(stable) >= 2 { + // Show up to two stable versions. + return false + } + if len(stable) == 0 { + // Most recent stable version. + stableMaj, stableMin, _ = parseVersion(r.Version) + return true + } + if maj, _, _ := parseVersion(r.Version); maj == stableMaj { + // Older minor version of most recent major version. + return false + } + // Second most recent stable version. + return true + } + if !shouldAddStable() { + archive = append(archive, *r) return } - if len(unstable) != 0 { - // Only show one (latest) unstable version. - return - } - maj, min, _ := parseVersion(r.Version) - if maj < stableMaj || maj == stableMaj && min <= stableMin { - // Display unstable version only if newer than the - // latest stable release. - return - } - unstable = append(unstable, *r) + + // Split the file list into primary/other ports for the stable releases. + // NOTE(cbro): This is only done for stable releases because maintaining the historical + // nature of primary/other ports for older versions is infeasible. + // If freebsd is considered primary some time in the future, we'd not want to + // mark all of the older freebsd binaries as "primary". + // It might be better if we set that as a flag when uploading. + r.SplitPortTable = true + r.Visible = true // Toggle open all stable releases. + stable = append(stable, *r) } for _, f := range fs { if r == nil || f.Version != r.Version { diff --git a/godoc/dl/dl_test.go b/godoc/dl/dl_test.go index 05d85ad3..3f61fe9e 100644 --- a/godoc/dl/dl_test.go +++ b/godoc/dl/dl_test.go @@ -87,7 +87,7 @@ func TestFilesToReleases(t *testing.T) { {Version: "go1.5", OS: "windows"}, {Version: "go1.5beta1", OS: "windows"}, } - stable, unstable, all := filesToReleases(fs) + stable, unstable, archive := filesToReleases(fs) if got, want := len(stable), 2; want != got { t.Errorf("len(stable): got %v, want %v", got, want) } else { @@ -101,8 +101,8 @@ func TestFilesToReleases(t *testing.T) { if got, want := len(unstable), 0; want != got { t.Errorf("len(unstable): got %v, want %v", got, want) } - if got, want := len(all), 6; want != got { - t.Errorf("len(all): got %v, want %v", got, want) + if got, want := len(archive), 4; want != got { + t.Errorf("len(archive): got %v, want %v", got, want) } } diff --git a/godoc/dl/tmpl.go b/godoc/dl/tmpl.go index d9f12f52..e92f9b76 100644 --- a/godoc/dl/tmpl.go +++ b/godoc/dl/tmpl.go @@ -135,13 +135,12 @@ information about Go releases. {{end}} {{with .Archive}} -
- +
-

All versions▾

+

Archived versions▾

{{template "releases" .}}
@@ -218,8 +217,7 @@ $(document).ready(function() { {{define "releases"}} {{range .}} -
- {{/* NOTE(cbro): versions may show multiple times. Don't use "id". */}} +
diff --git a/godoc/static/godocs.js b/godoc/static/godocs.js index e4d13699..1fb8b91e 100644 --- a/godoc/static/godocs.js +++ b/godoc/static/godocs.js @@ -246,7 +246,8 @@ function fixFocus() { function toggleHash() { // Open all of the toggles for a particular hash. - var els = $(window.location.hash + ", a[name='" + window.location.hash.substring(1) + "']"); + var els = $(document.getElementById(window.location.hash.substring(1)), + $("a[name='" + window.location.hash.substring(1) + "']")); while (els.length) { for (var i = 0; i < els.length; i++) { var el = $(els[i]); diff --git a/godoc/static/static.go b/godoc/static/static.go index 0190fe75..70a4d931 100644 --- a/godoc/static/static.go +++ b/godoc/static/static.go @@ -816,7 +816,8 @@ function fixFocus() { function toggleHash() { // Open all of the toggles for a particular hash. - var els = $(window.location.hash + ", a[name='" + window.location.hash.substring(1) + "']"); + var els = $(document.getElementById(window.location.hash.substring(1)), + $("a[name='" + window.location.hash.substring(1) + "']")); while (els.length) { for (var i = 0; i < els.length; i++) { var el = $(els[i]);