release(#395): v2.0.1#396
Merged
Merged
Conversation
… polish (#394) Seven findings from Iman's mobile UX audit on the v2.0.0 production site (4 iPhone screenshots from CEO at 08:59 / 09:00). Primary fix addresses a real nav regression introduced when the quickstart chip landed in PR #387 without also marking the main-page links as visible-on-mobile. Fixes (cross-page): 1. HIGH — Mobile nav was dropping the 3 main-page links on every interior page. The <700px collapse rule `.titlebar__right a:not(.is-cta):not(.always) { display: none }` was hiding everything except the current-page link + quickstart + github →. Mobile readers couldn't navigate between sections. Fix: added class="always" to the 3 main-page nav links on all 4 pages. Mobile nav now shows: how it works · quickstart · architecture · skills · github → (5 items consistently). The changelog → link stays hidden on mobile (acceptable — external one-off). 2. MEDIUM — Copy-as-Markdown-for-AI button floated orphaned on mobile because the eyebrow had no explicit flex layout (the button's inline margin-left:auto was a no-op). Fix: made .page-header__eyebrow + .hero__eyebrow flex-wrap containers with explicit gap. Added <700px override so the button drops to its own line below the pill+subtitle, sitting flush-left rather than awkward-right. 3. MEDIUM — Eyebrow subtitles too long on mobile. Trimmed: - architecture: "one fork, five layers, optional sibling repo" -> "5 layers, optional sibling repo" - skills: "53 active slash commands - grouped by what you're trying to do" -> "53 active slash commands, grouped by outcome" - how-it-works: kept ("one realistic day, in plain English") 4. MEDIUM — how-it-works.html had visible content redundancy. Tagline + ai-lead both said "a walkthrough of one realistic day". The ai-lead was added in PR #387 for /geo-audit G12 (first-500- tokens lead). Fix: CSS-hidden via visually-hidden pattern (.ai-lead-hidden class — clip + position:absolute + 1px). DOM stays for AI crawlers + screen readers (aria-hidden="false" preserves access); sighted users no longer see the duplicated prose. Homepage hero polish (LOW): 5. "Built by me2resh" moved from BETWEEN tagline and subhead to AFTER the CTAs. Previously broke the headline -> subhead reading flow; now sits as a footer-flavoured byline below the CTA block. 6. Version line ("v1.1.0 · changelog") dimmed further (opacity 0.7 -> 0.55, font-size 14px -> 13px) so it doesn't compete with the tagline for first glance. 7. Hero inline link "See what a day with it looks like ->" was wrapping mid-phrase on mobile. Shortened to "See a day with it ->" + added white-space:nowrap so it stays as one unit. Site-counts drift: PASS. (Subtitle change for skills had to keep "active" between digit and "slash commands" so the drift regex doesn't flag it as a 53-vs-54 mismatch — same pattern used since v2.0.0 prep.) Closes #393 Co-authored-by: me2resh <ahmed.abdelaliem@gmail.com>
✅ Deploy Preview for apexyard ready!
To edit notification comments on pull requests, go to your Netlify project configuration. |
atlas-apex
commented
May 24, 2026
atlas-apex
left a comment
Collaborator
Author
There was a problem hiding this comment.
Code Review: PR #396
Commit: b96d77c61aeb0cd1fed2ffc5d68e84697b2de1a6
Summary
v2.0.1 PATCH release cherry-picking PR #394's mobile UX hotfix (#393) onto upstream/main, plus the CHANGELOG append. Site-only — no framework code touched. The cherry-pick-from-main strategy sidesteps the v1.4→v2.0 squash divergence on dev.
Checklist Results
- Architecture & Design: N/A (no framework code)
- Code Quality: Pass (CSS + HTML only; class additions consistent with existing
.alwaysconvention; visually-hidden pattern correctly implemented with clip + position) - Testing: N/A (cosmetic; visual mobile check listed as Netlify-preview manual step)
- Security: Pass (no auth/crypto/secrets surface; external links use
rel="noopener") - Performance: Pass (~14 lines of CSS added; no perf concern)
- PR Description & Glossary: Pass (6-term glossary; well-structured)
- Summary Bullet Narrative: Pass (every bullet carries what + why)
- Technical Decisions (AgDR):N/A (
agdr: not-applicablemarker legitimate for pure release-cut) - Adopter Handbooks: N/A (no architecture/general/language-matched handbooks fire — pure site changes)
Diff-scope verification
All 5 files match expected scope:
| File | +/− | Notes |
|---|---|---|
CHANGELOG.md |
+18 / 0 | Release section append only |
site/architecture.html |
+13 / −4 | .always class + eyebrow flex + subtitle trim |
site/how-it-works.html |
+30 / −4 | .always + eyebrow flex + .ai-lead-hidden visually-hidden CSS |
site/index.html |
+16 / −10 | .always + eyebrow mobile drop + hero re-order + version-pill bump to v2.0.1 (13px / 0.55 / link to /tag/v2.0.1) |
site/skills.html |
+13 / −4 | .always + eyebrow flex + subtitle trim |
No framework files (hooks/skills/agents/rules/templates/workflows) touched.
CHANGELOG parallelism
v2.0.1 mirrors v2.0.0's shape: ## [version] — date + ### <release-theme> sub-heading + narrative paragraph + ### Fixed + ### Compatibility. Pattern matches; correctly placed above v2.0.0.
Commit-level verification
ec498ce(cherry-pick of #394) — 4 site/ files, includes the version-pill conflict resolution bumpingv2.0.0→v2.0.1in the tag link. Authorship preserved.b96d77c(CHANGELOG) — single-file commit, 18 additions, 0 deletions. Refs #395.
CI
All blocking checks green: lychee, markdownlint-cli2, Verify Ticket ID, site-counts drift detection, Header rules, Redirect rules. Pages changed - apexyard is NEUTRAL (advisory). Mergeable: YES.
Issues Found
None.
Suggestions
None — release shape is clean and matches the v2.0.0 precedent.
Verdict
APPROVED
Reviewed by Rex (Code Reviewer Agent)
Reviewed commit: b96d77c61aeb0cd1fed2ffc5d68e84697b2de1a6
3 tasks
6 tasks
3 tasks
OmarElaraby26
pushed a commit
to OmarElaraby26/apexyard
that referenced
this pull request
May 27, 2026
* fix(me2resh#393): mobile UX regressions — nav main pages + eyebrow + content polish (me2resh#394) Seven findings from Iman's mobile UX audit on the v2.0.0 production site (4 iPhone screenshots from CEO at 08:59 / 09:00). Primary fix addresses a real nav regression introduced when the quickstart chip landed in PR me2resh#387 without also marking the main-page links as visible-on-mobile. Fixes (cross-page): 1. HIGH — Mobile nav was dropping the 3 main-page links on every interior page. The <700px collapse rule `.titlebar__right a:not(.is-cta):not(.always) { display: none }` was hiding everything except the current-page link + quickstart + github →. Mobile readers couldn't navigate between sections. Fix: added class="always" to the 3 main-page nav links on all 4 pages. Mobile nav now shows: how it works · quickstart · architecture · skills · github → (5 items consistently). The changelog → link stays hidden on mobile (acceptable — external one-off). 2. MEDIUM — Copy-as-Markdown-for-AI button floated orphaned on mobile because the eyebrow had no explicit flex layout (the button's inline margin-left:auto was a no-op). Fix: made .page-header__eyebrow + .hero__eyebrow flex-wrap containers with explicit gap. Added <700px override so the button drops to its own line below the pill+subtitle, sitting flush-left rather than awkward-right. 3. MEDIUM — Eyebrow subtitles too long on mobile. Trimmed: - architecture: "one fork, five layers, optional sibling repo" -> "5 layers, optional sibling repo" - skills: "53 active slash commands - grouped by what you're trying to do" -> "53 active slash commands, grouped by outcome" - how-it-works: kept ("one realistic day, in plain English") 4. MEDIUM — how-it-works.html had visible content redundancy. Tagline + ai-lead both said "a walkthrough of one realistic day". The ai-lead was added in PR me2resh#387 for /geo-audit G12 (first-500- tokens lead). Fix: CSS-hidden via visually-hidden pattern (.ai-lead-hidden class — clip + position:absolute + 1px). DOM stays for AI crawlers + screen readers (aria-hidden="false" preserves access); sighted users no longer see the duplicated prose. Homepage hero polish (LOW): 5. "Built by me2resh" moved from BETWEEN tagline and subhead to AFTER the CTAs. Previously broke the headline -> subhead reading flow; now sits as a footer-flavoured byline below the CTA block. 6. Version line ("v1.1.0 · changelog") dimmed further (opacity 0.7 -> 0.55, font-size 14px -> 13px) so it doesn't compete with the tagline for first glance. 7. Hero inline link "See what a day with it looks like ->" was wrapping mid-phrase on mobile. Shortened to "See a day with it ->" + added white-space:nowrap so it stays as one unit. Site-counts drift: PASS. (Subtitle change for skills had to keep "active" between digit and "slash commands" so the drift regex doesn't flag it as a 53-vs-54 mismatch — same pattern used since v2.0.0 prep.) Closes me2resh#393 Co-authored-by: me2resh <ahmed.abdelaliem@gmail.com> * chore(me2resh#395): CHANGELOG v2.0.1 — mobile UX hotfix - Append [2.0.1] section above [2.0.0] - Site-only release; framework unchanged - 7 fixes from PR me2resh#394 (mobile nav, eyebrow stacking, content redundancy, hero polish, subtitle trims) Refs me2resh#395 --------- Co-authored-by: me2resh <ahmed.abdelaliem@gmail.com>
5 tasks
mosta7il
pushed a commit
to mosta7il/apexyard
that referenced
this pull request
Jun 8, 2026
* fix(me2resh#393): mobile UX regressions — nav main pages + eyebrow + content polish (me2resh#394) Seven findings from Iman's mobile UX audit on the v2.0.0 production site (4 iPhone screenshots from CEO at 08:59 / 09:00). Primary fix addresses a real nav regression introduced when the quickstart chip landed in PR me2resh#387 without also marking the main-page links as visible-on-mobile. Fixes (cross-page): 1. HIGH — Mobile nav was dropping the 3 main-page links on every interior page. The <700px collapse rule `.titlebar__right a:not(.is-cta):not(.always) { display: none }` was hiding everything except the current-page link + quickstart + github →. Mobile readers couldn't navigate between sections. Fix: added class="always" to the 3 main-page nav links on all 4 pages. Mobile nav now shows: how it works · quickstart · architecture · skills · github → (5 items consistently). The changelog → link stays hidden on mobile (acceptable — external one-off). 2. MEDIUM — Copy-as-Markdown-for-AI button floated orphaned on mobile because the eyebrow had no explicit flex layout (the button's inline margin-left:auto was a no-op). Fix: made .page-header__eyebrow + .hero__eyebrow flex-wrap containers with explicit gap. Added <700px override so the button drops to its own line below the pill+subtitle, sitting flush-left rather than awkward-right. 3. MEDIUM — Eyebrow subtitles too long on mobile. Trimmed: - architecture: "one fork, five layers, optional sibling repo" -> "5 layers, optional sibling repo" - skills: "53 active slash commands - grouped by what you're trying to do" -> "53 active slash commands, grouped by outcome" - how-it-works: kept ("one realistic day, in plain English") 4. MEDIUM — how-it-works.html had visible content redundancy. Tagline + ai-lead both said "a walkthrough of one realistic day". The ai-lead was added in PR me2resh#387 for /geo-audit G12 (first-500- tokens lead). Fix: CSS-hidden via visually-hidden pattern (.ai-lead-hidden class — clip + position:absolute + 1px). DOM stays for AI crawlers + screen readers (aria-hidden="false" preserves access); sighted users no longer see the duplicated prose. Homepage hero polish (LOW): 5. "Built by me2resh" moved from BETWEEN tagline and subhead to AFTER the CTAs. Previously broke the headline -> subhead reading flow; now sits as a footer-flavoured byline below the CTA block. 6. Version line ("v1.1.0 · changelog") dimmed further (opacity 0.7 -> 0.55, font-size 14px -> 13px) so it doesn't compete with the tagline for first glance. 7. Hero inline link "See what a day with it looks like ->" was wrapping mid-phrase on mobile. Shortened to "See a day with it ->" + added white-space:nowrap so it stays as one unit. Site-counts drift: PASS. (Subtitle change for skills had to keep "active" between digit and "slash commands" so the drift regex doesn't flag it as a 53-vs-54 mismatch — same pattern used since v2.0.0 prep.) Closes me2resh#393 Co-authored-by: me2resh <ahmed.abdelaliem@gmail.com> * chore(me2resh#395): CHANGELOG v2.0.1 — mobile UX hotfix - Append [2.0.1] section above [2.0.0] - Site-only release; framework unchanged - 7 fixes from PR me2resh#394 (mobile nav, eyebrow stacking, content redundancy, hero polish, subtitle trims) Refs me2resh#395 --------- Co-authored-by: me2resh <ahmed.abdelaliem@gmail.com>
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.
v2.0.1 — 2026-05-24
PATCH release. Mobile UX hotfix for the v2.0.0 marketing site. No framework code touched — site-only.
This PR will tag
v2.0.1onmainafter merge.Fixed
fix(#393)Main-page nav restored on mobile —architecture,skills, andhow it workslinks were hidden by the<700pxcollapse rule on all 4 site pages. Addedclass="always"so they stay visible. Mobile readers can move between sections again.fix(#393)Eyebrow row wraps cleanly — the "Copy as Markdown for AI" button no longer crowds the pill+subtitle row at narrow widths. Drops to its own line below the eyebrow on mobile.fix(#393)Duplicated lead text hidden from sighted users on/how-it-works— the#ai-leadblock (added in v2.0.0 to satisfy/geo-auditG12) is now visually-hidden via clip+position trick. AI crawlers and screen readers still consume it; sighted users no longer see the same prose twice.fix(#393)Homepage hero polish — "Built by me2resh" moved from between tagline and subhead to below the CTAs; version line dimmed further (14px→13px, opacity 0.7→0.55, also bumped to v2.0.1); hero inline link shortened to "See a day with it →" +white-space:nowrapso it doesn't wrap mid-phrase.fix(#393)Subtitles trimmed on/architectureand/skillsso they don't wrap awkwardly on mobile.Compatibility
No breaking changes. No framework code touched. Adopters see no changes to hooks, skills, rules, agents, templates, or workflows — only
site/files were modified.Release strategy
Branched
release/v2.0.1fromupstream/main(NOT from dev) to sidestep the v1.4→v2.0 squash divergence carried on dev. Cherry-picked the squashed #394 fix (0b51465) onto the release branch and bumped the site version link tov2.0.1while resolving the (trivial) conflict on the version pill.The main→dev sync is a known follow-up — same shape as the v2.0.0 release; will be folded into the next release cut or done as a standalone sync PR.
AgDR note
<!-- agdr: not-applicable -->marker present at the top: release PRs are pure version-cuts with no architectural decisions (the underlying #393 fix was already shipped as a cosmetic CSS+HTML hotfix to dev with the same skip marker on PR #394).Testing
[2.0.0]with parallel structuregit log upstream/main..HEADshows exactly 2 commits (cherry-picked fix + CHANGELOG)v2.0.0→v2.0.1(will resolve to a real tag after merge)v2.0.1tag pushed toupstream/mainGlossary
fix:commits are included since the last tag — no new features, no breaking changes.dev;mainonly receives release PRs fromdev(tagged with semver). Adopted in AgDR-0007.git cherry-pick <sha>— replay a single commit's diff onto the current branch. Used here so the v2.0.1 release contains exactly the #394 fix and nothing else.<!-- multi-close: approved -->Closes-per-PR hook. Release PRs legitimately close many tickets at once and are the canonical use case for the marker.<!-- agdr: not-applicable -->Closes #395
Closes #393