feat(website): add Longshot player site and team pages #104
No reviewers
Labels
No labels
area/accessibility
area/audio
area/input
area/install
area/menu
area/renderer
area/saves
area/upscaling
good first issue
gpu/amd
gpu/igpu
gpu/intel
gpu/nvidia
help wanted
kind
bug
kind
chore
kind
docs
kind
feature
kind
performance
kind
question
needs measurements
platform/linux
platform/steam-deck
platform/windows
preview
priority
critical
priority
high
priority
low
status
blocked
status
confirmed
status
duplicate
status
invalid
status
needs-info
status
wontfix
testing wanted
No milestone
No project
No assignees
2 participants
Notifications
Due date
No due date set.
Dependencies
No dependencies set
Reference
MagicBeanDev/Longshot!104
Loading…
Reference in a new issue
No description provided.
Delete branch "bibiburl/Longshot:docs/longshot-website"
Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?
Why
Players need a clear front door to Longshot: installation help, release information, troubleshooting and the people behind the project. This adds the team magicbean website and documentation experience at
/Longshot/while retaining the maintained project documentation as its source.What changed
838c977with this head and measure raw asset bytes, not browser load time; fallback Sans fonts may not download when Atkinson is used.docs/common-issues.md, existing topic anchors are preserved, and native disclosures support keyboard/no-JavaScript use and the shared print expansion.gpu/vk/vk_menu.cppandhost/nn_vi.c; no runtime code changed.website/: Astro/Starlight site with Longshot branding, locally bundled fonts, release and development panels, player support, Discord links, a team roster with Git avatars, and an About page that credits the original Hookshot project.website/scripts/prepare-content.mjsandformat-manual.mjs: import the canonical README, contributor guide and selected docs. Installation comes first, platform steps become cards, and advanced reference sections can be expanded. Existing commands, links and headings are preserved; linked disclosures open automatically.docs/: concise player guides and support routes. The README's two current reporting links point to Longshot; historical credits remain intact.5eaaf7f, preserving upstream's new Longshot Launcher instructions. The requirement review records the download-description change; hardware and game requirements are unchanged.Flags and settings
No game settings or runtime flags change. Website build settings are documented in
website/README.md:DOCS_SITE_URLcontrols canonical metadata, andDOCS_SOURCE_REPOSITORY/DOCS_SOURCE_BRANCHcontrol edit links. Preview builds deliberately target this fork branch. Before production, build with the intended source repository and branch.Measurements
Baseline: main @
5eaaf7f. This PR @ 57f6577. Website-only validation on Linux using Node 22.23.4 and the pinned dependencies; no gameplay measurements are claimed.5eaaf7fwebsite/directoryReproduce from
website/with Node 22.23.4 or newer:Open
/Longshot/,/Longshot/guides/graphics-and-controls/,/Longshot/reference/manual/,/Longshot/team/and/Longshot/about/. Generated pages are not committed.Testing
Tested: website build, route/anchor/asset checks, search-index generation, source-review guards and all nine website test files. The manual transform tests preserve source content and fenced code. Independent UI source review found no actionable issue in the manual and About changes.
Graphics validation also includes five passing README/menu coverage tests (
python -I -B tests/test_readme_menu_rows.py), source-drift and generation tests, and DOM-fixture sequences for combined filtering, reset, deep links (including repeated permalinks) and print-state restoration. Generated HTML has 26 complete cards, seven groups and unique anchors. Independent UI source review found one repeated-permalink issue; it is fixed and covered by a regression test. These fixture/source checks are not rendered-browser verification.Issue-search regression checks cover panel ordering before delayed overview updates, focus/scroll after layout, repeated navigation, disconnected-page focus, and keeping results, pagination and feedback together. Generated HTML confirms the complete movable panel.
Symptom-card validation preserves all canonical checklist/help text and seven legacy anchors; generated HTML contains seven native cards, 21 numbered checks and seven help disclosures. Bean-line tests cover all ten choices, immediate-repeat avoidance across reloads and blocked storage. Independent UI source/metric review identified one long mobile phrase; it was shortened before the final build.
Font conversion was checked by decompression and comparison of character maps, all 7,441 glyph outlines/advances and layout/shaping tables. The combined preview build passes all nine website test files and 918 link/asset checks. Compiled CSS resolves to all three emitted WOFF2 files, with no emitted Libertinus TTFs; animation and reduced-motion rules are present. Rendered animation smoothness and browser timing remain unmeasured.
Not tested: rendered desktop/mobile appearance, keyboard and screen-reader behavior, light/dark themes, live search interaction and printed output. Browser automation was unavailable. No Cloudflare deployment, game launch/build or runtime test suite was run; this change does not modify runtime code. Full
tests/run_all.pyis not claimed.Needs other testers:
Screenshots
Pending browser/user visual review. No screenshots are claimed. The author's local preview is not a hosted review URL; other contributors must run the commands above.
Risk and rollback
Main risks are responsive layout, imported Markdown presentation, dated progress data and production source-link configuration. Revert the website/docs changes to roll back. There is no automatic site deployment in this PR. The companion Site PR packages the generated website; it must use a reviewed Longshot build before deployment.
Reviewer notes
Ready for review at the author's request. The validation limitations and unchecked items below remain outstanding; this status change does not certify merge or deployment readiness. Companion change: MagicBeanDev/Site PR #1. Documentation stays in Longshot; Site copies only built output. No pushes target Hookshot.
Issues
No issue closure requested.
Checklist
For everyone
type(scope): description).main. (Based on5eaaf7f; the last fetched upstream is6c90418. A further rebase remains before merge.)python -I -B tests/run_all.pyrun and passing (any failure that also fails onmainis named above).third_party/untouched (a new folder with aPROVENANCE.mdinstead).If applicable
Assisted-by:/Co-Authored-By:trailers.Pushed 8fcee1d: minimum/required and recommended system specs now share one README source across the manual and compatibility page. Existing thresholds and caveats remain unchanged. Validation: 15 pages, 808 local links/assets and anchors, all seven website test files passing; generated tables match on both pages. UI source review complete; rendered desktop/mobile and print checks remain pending.
Pushed 2626467: save-transfer and update guides now have Easy beans launcher steps and Advanced beans reference sections. Verified launcher control names against its maintained save guide. Backup/replacement guidance stays visible. Validation: 15 pages, 814 local links/assets and anchors, all seven website test files passing; all original save commands/reference tables/links and all 12 prior section anchors retained. Local preview refreshed. Rendered browser checks remain pending; no save files were accessed or transferred.
Pushed 901fac2: applied the five UI review recommendations and removed the LOCAL PREVIEW badge. Shared-memory users no longer receive a contradictory VRAM-entry request; warnings remain. Automatic memory selection updates when GPU changes, while explicit overrides persist. Added mobile navigation, clearer hero/support wording and an adaptive specs scroll cue. Validation: 15 pages, 889 local links/assets and anchors, all seven website test files passing, including the shared-memory regression; generated navigation parity and support ordering checked. Independent source review resolved its finding. Local preview refreshed; rendered browser/keyboard/mobile checks remain pending.
Pushed f10594c: graphics guide now builds 26 searchable setting cards across seven source-defined sections, with choices/defaults from the maintained README. Added section filters, restart badges, permalinks and print/reset handling. Corrected the stale README frame-cap restart note after checking vk_menu.cpp and host/nn_vi.c; runtime code unchanged. Validation: 15 pages, 922 local links/assets plus anchors, 8 website test files and 5 README/menu tests pass. UI source-review permalink finding fixed and regression-tested. Browser access remains unavailable, so rendered desktop/mobile, keyboard, themes and print review remain pending; PR stays WIP. Local preview refreshed.
Pushed 470114d to fix the reported front-page search landing problem. Issue-search links now move the whole search panel (input, filters, results and feedback) before the development overview, then focus/position the input after layout. Delayed feed rendering cannot insert overview content above it. Local preview refreshed. Validation: 15-page build, 922 local links/assets and anchors, all 8 website test files pass, including navigation regression fixtures. Generated HTML structure checked; rendered browser scrolling remains unverified because browser access is unavailable.
Pushed c200a2f: shortened the front-page search caption to “Searches reported issues”, as requested. Local preview refreshed; website build, 922 local links/assets plus anchors and all 8 website test files pass.
Pushed e292024: removed the progress coverage paragraph and its script update, as requested. Local preview refreshed. Build, 922 local links/assets and anchors, and all 8 website test files pass. Confirmed the paragraph is absent from generated HTML and compiled JavaScript. Common-issues redesign is advisory only at this point; no changes to that guide.
Pushed 40d0546: replaced the common-issues wall of reporting advice with seven symptom cards, each offering three short checks, related guides and optional Still stuck help with search/Discord links. Existing anchors retained; maintained content remains in docs/common-issues.md. Added ten homepage bean punchlines that change once per load and avoid immediate repeats with session storage. Validation: 15 pages, 918 local links/assets plus anchors and all 9 website test files pass. Independent UI source review completed; one long phrase shortened for mobile. Generated HTML checked. Local preview refreshed; rendered browser verification remains unavailable, so PR stays WIP.
Pushed 838c977 for the reported card layout issues: removed the full-row expansion that moved right-hand cards down, gave headers a marker/text grid with reset margins, and inset keyboard focus outlines within padded headers. Local preview refreshed. Build, 918 local links/assets and anchors, all 9 website test files pass; compiled CSS confirms no expanded column span. Browser visual verification remains unavailable.
WIP: feat(website): add Longshot player site and team pagesto feat(website): add Longshot player site and team pagesPushed d5a225d: full-glyph WOFF2 fonts save 828,460 emitted asset bytes; the heading font is 61.1% smaller. Added the main homepage’s gentle hero bean bob, paused on hover/focus and disabled for reduced motion. Build, 918 link/asset checks and all 9 website test files pass. Font glyphs/metrics verified; browser appearance and timing remain unverified. Local preview refreshed; PR remains ready for review.
Once we have a build available, I’d like to add screenshots of the launcher and the in-game settings menu to the relevant how-to guides. Showing the actual controls beside each step should make installation, setup and settings easier to understand.
These should show our UI only: no game assets or gameplay imagery, and no personal paths, account details or save information. For the in-game menu, we should first establish a capture method that excludes the game scene behind it; cropping alone may not be enough if the menu is translucent.
To keep maintenance efficient, I suggest:
This is a follow-up proposal once a build is available. No screenshots or capture automation are included yet. Feedback on the simplest reliable capture and review workflow would be welcome.
Pushed a17ea51: added “No game files included” in the releases card’s central copy, explicitly stating that we do not host or distribute game files, game assets or keys and users must provide their own supported copy. Preview refreshed; 15-page build, 918 link/asset checks and all nine website test files pass. Rendered visual checks remain outstanding.
Pushed 57f6577: moved the game-files disclaimer outside the release card, directly underneath it in red (source-color contrast 9.70:1 dark, 6.89:1 light). Checked 105 available Longshot PRs plus 6 Site PRs; added missing profiles simonwjackson (#138/#137/#136), snaildev99 (#134/#131), and futondestroyer (#108), with profile API avatar URLs. The GitHub sync account is not listed as a person. Existing roster order and bibiburl-last preference retained. Build, 918 link/asset checks and all nine website test files pass; generated HTML confirms 12 cards and the separate disclaimer. Preview refreshed. Rendered visual checks remain outstanding.
0074997f30ef3312f7d4Hi @bibiburl, thanks for the site! I've pushed some work on top of your branch. Your commits are all still there, rebased onto
main, with two commits of mine added after them:Preview: https://pr-104-longshot-website.hckr-5e0.workers.dev (uploaded by hand from
ef3312f7).ci(website): deploy and previews. Merging tomaindeploys the site to longshot.magicbean.dev, and it refreshes hourly so the progress data stays current. A PR from a Longshot branch gets a preview on every push. A PR from a fork gets one when a maintainer adds thepreviewlabel. That label only works once this workflow is onmain.feat(website): the rework./Longshot/. The URLs are reorganised:/install,/requirements,/settings,/saves,/saves/transfer,/help,/help/report,/help/antivirus,/progress,/contributing,/about,/about/team.site.tsreplacessite.config.mjs.vk_menu.cpp), Updates & saves, Transfer a save (around the launcher's importer and exporter), Help (problems grouped by when they happen), Antivirus warnings (step by step for Windows) and Report a problem./manual/is gone: links into the README go to the README on Forgejo.worker/index.tsserves the launcher's permanent download links (/download,/download/launcher/<channel>/<os>), which redirect to git-cdn.magicbean.dev. This code comes from MagicBeanDev/Site#7.Checks:
npm run buildandnpm run checkpass (the site check covers 14 pages, links and anchors; 43 of 43 tests pass).tests/run_all.pyhas 2 failures on my machine,test_bake_portabilityandtest_builder_log_read_only, which I haven't looked into.Shout if anything here goes against what you had in mind.
ef3312f7d4c9b33e0b41