Skip to content

fix(docs): version-picker shows 'latest' only on /versions/latest/ #438

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
Chris-Wolfgang merged 3 commits into from
Jul 25, 2026
Merged

fix(docs): version-picker shows 'latest' only on /versions/latest/ #438

Changes from all commits
Commits
Show all changes
3 commits
Select commit Hold shift + click to select a range
dea7557
fix(docs): surface 'latest' alias only on /versions/latest/
Chris-Wolfgang Jul 18, 2026
0efa8c5
docs(version-picker): correct comment about auto-select behavior
Chris-Wolfgang Jul 24, 2026
806ae41
Merge branch 'main' into fix/version-picker-latest-only-on-alias
Chris-Wolfgang Jul 25, 2026
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
24 changes: 18 additions & 6 deletions docfx_project/public/version-picker.js
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -49,8 +49,14 @@
}

function renderPicker(versions) {
// Detect the currently-viewed version from the URL.
var currentVersion = 'latest';
// Detect the currently-viewed version from the URL. Only a
// /versions/<v>/ path names a concrete version; the literal
// /versions/latest/ alias yields 'latest'. Anywhere else (site
// root, unversioned pages) currentVersion stays null so the
// 'latest' alias is NOT surfaced and the browser auto-selects the
// highest v* entry — matching the intent that 'latest' appears in
// the picker only on /versions/latest/.
var currentVersion = null;
var m = window.location.pathname.match(/\/versions\/([^\/]+)(?:\/|$)/);
if (m) {
currentVersion = m[1];
Expand Down Expand Up @@ -86,12 +92,18 @@
var optionCount = 0;
versions.forEach(function (v) {
if (!v || !v.version || !v.url) return;
// Skip the "latest" alias — the highest-numbered v* entry
// already represents the latest release; surfacing both is
// redundant in the picker. versions.json keeps the "latest"
// Skip the "latest" alias EXCEPT when the reader is actually
// on /versions/latest/. On every other page the highest-
// numbered v* entry already represents the latest release and
// surfacing both is redundant; on /versions/latest/ we NEED
// 'latest' in the list because without it the browser auto-
// selects the first concrete version (whichever v* is top of
// the list), and choosing that same value doesn't fire
// `change`, so the reader can't navigate away to the
// concrete-version URL. versions.json keeps the "latest"
// entry so other consumers (links, scripts) can still
// resolve it.
if (v.version === 'latest') return;
if (v.version === 'latest' && currentVersion !== 'latest') return;
var opt = document.createElement('option');
opt.value = v.url;
opt.textContent = v.version;
Expand Down