Skip to content

Chore/homepage logo strip copy#2950

Merged
eldadfux merged 4 commits intomainfrom
chore/homepage-logo-strip-copy
Apr 30, 2026
Merged

Chore/homepage logo strip copy#2950
eldadfux merged 4 commits intomainfrom
chore/homepage-logo-strip-copy

Conversation

@eldadfux
Copy link
Copy Markdown
Member

What does this PR do?

(Provide a description of what this PR does.)

Test Plan

(Write your test plan here. If you changed any code, please provide us with clear instructions on how you verified your changes work.)

Related PRs and Issues

(If this PR is related to any other PR or resolves any issue or related to any issue link all related PR and issues here.)

Have you read the Contributing Guidelines on issues?

(Write your answer here.)

- Replaced 'Claude' with 'Claude Code' and added 'Codex' in various locations including AI tools list and documentation.
- Updated styles and structure in Svelte components for better consistency and appearance.
- Removed references to 'Windsurf' and adjusted the layout for improved clarity in the AI tooling section.
- Enhanced tooltip content and visual elements for better user experience.
- Introduced new SVG icons for Astro, Java, Nuxt, Qwik, and Solid.
- Updated the dashboard component to include a new top navigation SVG.
- Enhanced the platforms component with improved heading handling and animation effects.
- Added a new TypeScript file for quick-start strip items, consolidating framework icons and links for better organization.
@greptile-apps
Copy link
Copy Markdown
Contributor

greptile-apps Bot commented Apr 30, 2026

Greptile Summary

This PR promotes Claude Code and Codex to the top of featured AI tooling across the homepage, docs landing page, sidebar nav, and documentation copy, while removing Windsurf Editor from the featured list. It also substantially rewrites the homepage platforms.svelte to display a paginated framework strip alongside a separate AI-tooling row, and extracts the framework data into a new quick-start-strip-items.ts module with deterministic first-page ordering.

Confidence Score: 5/5

Safe to merge — all findings are P2 only; the missing-div layout issue was already flagged in a previous review thread.

All new findings are P2 (dead hover classes on a mobile-only pointer-events-none element; one nav href cross-section link). No P0 or P1 issues were identified beyond the previously flagged unclosed div.

src/routes/(marketing)/(components)/platforms.svelte — the unclosed border-y border-dashed div (flagged in a prior review thread) and the dead mobile-marquee hover gradient.

Important Files Changed

Filename Overview
src/routes/(marketing)/(components)/platforms.svelte Major rewrite: introduces paginated framework strip, AI tooling row, and mobile marquee; a missing closing </div> for the border-y border-dashed wrapper causes the nav-links section to render inside the border (previously flagged).
src/routes/(marketing)/(components)/quick-start-strip-items.ts New module exporting ordered framework strip data and constants; buildOrderedQuickStartStrip correctly hoists MARKETING_HOME_FIRST_NAMES to the front and appends remaining entries in source order.
src/routes/(marketing)/(components)/ai.svelte Tool list reordered to promote Claude Code and Codex to the top; Windsurf removed; tooltip styling aligned with the updated design system.
src/partials/mcp-add-ides-tools.md Tool ordering updated in both light/dark variants: Claude Code first, Codex added, Windsurf removed; Codex href is consistent with other files.
src/routes/docs/tooling/mcp/+layout.svelte Sidebar nav reordered; Codex entry links to /docs/tooling/ai/ai-dev-tools/codex while every other entry in this MCP layout links to /docs/tooling/mcp/… — may be intentional but worth confirming.

Reviews (2): Last reviewed commit: "format" | Re-trigger Greptile

Comment thread src/routes/(marketing)/(components)/platforms.svelte
- Introduced new SVG icons for Astro, Java, Nuxt, Qwik, and Solid.
- Updated the dashboard component to include a new top navigation SVG.
- Enhanced the platforms component with improved heading handling and animation effects.
- Added a new TypeScript file for quick start strip items, consolidating framework data for better management.
@eldadfux eldadfux merged commit bbba5fc into main Apr 30, 2026
5 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant