AI sub-brand fixes, Cohort logos, agent-panel + workflow doc#29
Merged
Conversation
…ces panel Header.tsx: the /ai route has its own isolated sub-brand skin and its own "back to main brand" link, so the shared site header (main-brand colors, nav) no longer renders there. Checked via usePathname() after all hooks run, per Rules of Hooks. page.tsx: the "AI sub-brand ->" text link under Quick Resources is the only place on the homepage that links to /ai. Replaced it with a full card matching the three Quick Resources panels above (same border/padding/radius, own row, same column width), so it reads as a fourth resource rather than an afterthought. Hover color uses the AI sub-brand's teal (#2FD09A) instead of the main brand's orange primary, since this panel is the doorway into that different palette. Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com>
Contributor
|
Deployment failed with the following error: View Documentation: https://vercel.com/docs/accounts/team-members-and-roles |
|
Important Review skippedAuto reviews are disabled on base/target branches other than the default branch. Please check the settings in the CodeRabbit UI or the ⚙️ Run configurationConfiguration used: defaults Review profile: CHILL Plan: Pro Run ID: You can disable this status message by setting the Use the checkbox below for a quick retry:
✨ Finishing Touches🧪 Generate unit tests (beta)
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
Documents the suedeN-per-work-period branching convention: one branch per period (not per task), branched from main, PRs always target deploy, deleted once merged. deploy -> main promotion is Sam's responsibility. Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com>
Matches the target="topw" pattern used by the For Archers/For Warriors internal page links, rather than the target="_blank" used for external resources (PDF/Figma/GitHub). Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com>
Four logos used on raidguild.org/join (moloch-500, moloch-800, scroll-100, scroll-700), copied from RaidGuildWebsite's public/images and renamed to the established short-code convention (cohort-m500.svg etc.), following the existing "Floating" logotype pattern on this page exactly. Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com>
Half-width cards, two rows of two, instead of a single stacked column. Reordered to m800, m500, s100, s700 to match the Logotype section's floating logo sequence (m800, m500, s100), with s700 appended since Logotype has no floating s700 variant to reference. Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com>
The old ai-mark-*.svg files were a recolor of the legacy RaidGuild sword icon and were wrong. Replaced with the correct symbol files (green, pink, white, black) supplied directly. Also replaced the "wordmark not cut yet" placeholder (live text next to the mark) with the three actual lockup versions now available, grouped by version with both color variants shown together: - Wordmark: stacked display type, no mark - Full Lockup: wordmark + mark, same stacked display type - Modern Lockup: single-line sans-serif wordmark + mark, matching this page's own typography (Space Grotesk, not the stacked serif style used in the other two) Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com>
Suede0619
changed the title
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.
Work-period follow-ups on the brand site, batched into one PR against
deploy./aisub-brand pageai-mark-*.svgmarks (a recolor of the legacy sword icon) were wrong and are removed. Replaced with the correct supplied assets: four symbol variants (green / pink / white / black) and three lockup treatments (Wordmark, Full Lockup, Modern Lockup), each in green + pink, grouped by version./ai— the route has its own isolated dark skin and its own back-link, so the main-brand header was off-brand there.Homepage
target=topw) as the other internal page links./logosRepo
BRANCHING.mddocumenting the suedeN-per-work-period branching convention (branch from main, PR to deploy, Sam promotes deploy to main).Build verified clean. Not merging — that is done manually.