Repository navigation
Lead with winget, and thin the install panel - #270
Merged
Merged
Conversation
The hero's prominent command was `curl … install.sh | bash` and the smaller one below it was `winget install fstubner.netscli`. os-tabs.ts already swapped those per-OS at runtime, so a Windows visitor with JavaScript got the winget line -- but the server-rendered default did not, so the most prominent command on the page was a bash pipeline until a script ran, and stayed one for anyone without JavaScript and for every crawler reading the HTML. The static markup now matches what Windows visitors were already getting. `winget install netscli`, not `winget install fstubner.netscli`. Checked against the live catalog rather than assumed: manifests/f/fstubner/netscli publishes `Moniker: netscli`, and the desktop package publishes `Moniker: netscli-gui`, so both short forms resolve. The canonical identifiers stay in the install guide. The panel was dense for two avoidable reasons. Cargo had a row on all three platforms -- the same `cargo install netscli` three times -- and it is now one clause in the note under the panel with the verification and packet-capture links. And "Verifies the download against the hash in its manifest" appeared four times in a single Windows panel, on winget and scoop under both Desktop app and CLI. It is "Hash-verified" now; the sentence lives in the install guide, and here it only has to separate those rows from the direct download below them. 20 install rows to 17, six Try it commands to five, 26 copy controls on the page. The direct-download button was accent text on an accent tint inside an accent border: the loudest control in the section, on the row that is the least recommended route. A direct download is unsigned and unverified while the package-manager rows above it check a hash, so the green was pointing at the wrong option. Neutral now, with the accent kept for hover and focus. astro check 0 errors, a11y 0 violations in both themes, CSS budget 42.
Contributor
|
Site preview: https://pr-270.netscli-site-preview.pages.dev Built from b2244fb with Production is unaffected: netscli.com is served from GitHub Pages via |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Two of the open items: swap the winget and shell-script bars (using
winget install netscli), and thin an install section that read as a wall.The swap had already happened — but only in JavaScript
os-tabs.tspicks hero commands per OS, and Windows visitors already gotwinget install fstubner.netsclias the primary bar with the PowerShell script below it. The server-rendered default did not: it wascurl … install.sh | bashas the most prominent command on the page, until a script ran.So the bash pipeline was the loudest thing on the page for anyone without JavaScript, and for every crawler reading the HTML. The static markup now matches what Windows visitors were already being shown, and the runtime swap still adapts it for macOS and Linux.
winget install netscli— checked, not assumedI looked at the live catalog rather than trusting the local templates:
Both short forms resolve. The canonical
fstubner.netscli/fstubner.netscli.guistay in the install guide for anyone who wants to be unambiguous.What made the panel dense
Two things, both measurable:
cargo install netscli, three times. It is one clause in the note under the panel now, beside the verification and packet-capture links.Hash-verifiednow. The full sentence lives in the install guide; on the row it only has to separate those from the direct download below them.20 install rows → 17 · 6 Try it commands → 5 · 26 copy controls on the page.
The green download button
Accent text on an accent tint inside an accent border — the loudest control in the section, sitting on the row that is the least recommended route. A direct download is unsigned and unverified; the package-manager rows above it check a hash. The colour was pointing at the wrong option.
Neutral now, matching the command bars beside it, with the accent kept for hover and focus so it still reads as a control.
Verification
astro check0 errors across 54 files · a11y 0 violations in both themes · CSS budget 42 · hero primary/secondary confirmed swapped in the built HTML.