Correct the stale facts in CLAUDE.md, and record this round's two lessons - #23
Merged
Merged
Conversation
… this round The interface table said InsightStack renders through Jekyll's `minima` theme and EquityStack is not published. Both were true when written and neither is now: the theme was removed in favour of `_layouts/default.html` (the same file says two paragraphs later not to reintroduce a theme), and EquityStack has had a landing page since PR #21. Replaced with what is actually deployed, dated, and with the three live URLs. The nested-duplicate-directory warning named `spss_scripts/spss_scripts/` and `kumu_maps/kumu_maps/` in all three repositories. Those folders exist only in InsightStack, which has twelve of them; FieldStack and EquityStack have none. Each copy now says which repository it is describing. Added the two things this round taught. `.row span` carries white-space: nowrap for the language tag, so a description added as another span inherits it and the page scrolls to 1384px against a 390px viewport, which is invisible on a desktop. And copy that lists a folder's contents tells a reader nothing about when to open it. EquityStack's design section also gains the house-style paragraph the other two carry, now that it has assets/css/stack.css. Co-Authored-By: Claude Opus 5 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_01DMYhECx2Dzbqad6KJxWZ7t
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
2 participants
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.
Documentation only. No code changes.
The interface table said this repository has no HTML and is not published. It has a landing page, merged in #21, live at https://varnasr.github.io/EquityStack/. The same table said InsightStack renders through Jekyll's
minimatheme, which was dropped for_layouts/default.html. Replaced with what is actually deployed, dated, with the three live URLs.The nested-duplicate-directory warning named
spss_scripts/spss_scripts/andkumu_maps/kumu_maps/as if they were folders here. They are InsightStack's, twelve of them; this repository has none.The design section gains the house-style paragraph the other two carry, now that this repository has
assets/css/stack.css: 2px borders, no shadows, no border-radius, the three typefaces, colour bands rather than floating cards, saffron as the single accent.Two lessons from the mobile-overflow round added: the
white-space: nowrapinheritance that made a page scroll to 1384px at a 390px viewport, and the rule that copy naming a module's contents does not tell a reader when to open it.🤖 Generated with Claude Code
https://claude.ai/code/session_01DMYhECx2Dzbqad6KJxWZ7t
Generated by Claude Code