docs: honest homepage gallery, doc-drift note, CONTRIBUTING.md - #6
Merged
Merged
Conversation
Follow-up to #5 (already merged: the README's work-in-progress line and its placeholder-gallery removal). This covers the rest of the same review: - docs/templates/home.html: the hero now leads with the same bold "Work in progress: this has not yet been run on Android hardware" framing as the README, instead of a softer note. The 17-image demo grid is gone entirely, captions and the trailing fps table with it: one of those captures (spirograph) is a literal iPhone screen recording, status bar and all, and the rest are the same raylib-ios origin even where it isn't as visible. The gallery section is now a short paragraph saying so, with a link to raylib-ios's own site for the real captures until this build has Android ones. - docs/check-site.sh: dropped the assertion that the built homepage embeds lorenz.gif and tesseract.gif, since that grid no longer exists. Replaced with a comment on what to restore, and why, once real captures land. Nothing else in the gate depended on the grid. - README.md and the homepage hero both note that Jolt and the libraries this project sits on (raylib-jlt, jolt-lang/nrepl, the pinned raylib revision) are still evolving, so a detail here can go stale between releases, and a version mismatch is worth checking against deps.edn before it's filed as a bug. - Adds CONTRIBUTING.md: an invitation (this is community- and team-built, suggestions and missing-piece reports welcome) plus an etiquette section adapted from Clojure's own community etiquette guide (https://clojure.org/community/etiquette), linked from a new README Contributing section. No image assets deleted, only what the docs and the site homepage show.
burinc
added a commit
that referenced
this pull request
Sep 7, 2026
Same cleanup as #5 and #6, extended to the two guide pages, which an earlier grep missed because a zsh glob with no match (docs/guide/*.html, which doesn't exist) aborted the whole command before it reached the .md files that do. - docs/guide/porting-an-example.md: the opening spirograph.png had no caveat of any kind, and the four-image block further down was captioned "running on the phone", stated as fact rather than sibling-build fact. Both are raylib-ios captures. Removed, keeping the factual content each caption carried (screen dimensions, no-new-binding count) as plain text. - docs/guide/performance-on-a-phone.md: this guide's own opening paragraph already says every number came off the sibling build, not Android, which is exactly right, but its two embedded images (spirograph.gif, the penrose/kaleidoscope stills) were never covered by that sentence. Removed both; the numbers in their captions are already in the tables directly below each, so nothing is lost. - docs/site.edn: the :asset-dirs comment claimed docs/images was "referenced from the homepage and the guides", which stopped being true across three separate commits now. Updated to say what's actually there: raylib-ios captures, unreferenced everywhere in this repo's docs, kept configured so an Android capture dropped in later just works. No image files deleted, only what the docs show. A repo-wide grep for images/, .gif and .png across README.md, CONTRIBUTING.md, docs/guide/*.md and docs/templates/*.html now returns nothing.
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.
Why
Follow-up to #5, which already landed the README's work-in-progress line and the removal of its placeholder image gallery. This PR is the rest of the same review pass, done after #5 merged:
Changes
lorenz.gif/tesseract.gif(that grid no longer exists), replaced with a comment on what to restore once real Android captures land.deps.edn's pins before filing a mismatch as a bug.No image assets deleted, only what the docs and homepage show.