Skip to content

Copy: replace logo strip headline with grounded developer messaging#2945

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

Copy: replace logo strip headline with grounded developer messaging#2945
eldadfux merged 2 commits intomainfrom
chore/homepage-logo-strip-copy

Conversation

@eldadfux
Copy link
Copy Markdown
Member

Use consistent, accurate wording for the customer logo section on the marketing homepage and homepage variations.

Made-with: Cursor

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.)

Use consistent, accurate wording for the customer logo section on the
marketing homepage and homepage variations.

Made-with: Cursor
@greptile-apps
Copy link
Copy Markdown
Contributor

greptile-apps Bot commented Apr 30, 2026

Greptile Summary

This PR updates marketing copy (logo strip headline unified to "Trusted by developer teams worldwide" across the main and variation pages), removes the BETA badge from the messaging product page, and redesigns the GitHub stat button across all layouts — replacing the star icon + "Star on GitHub" text label with the GitHub logo icon + star count only, paired with accessible aria-label attributes and a new .is-github-stat-link CSS variant.

Confidence Score: 5/5

Safe to merge — only P2 findings present.

All changes are copy updates, icon swaps, and CSS additions. The only finding is a missing trailing newline in a SCSS file, which is a P2 style issue and does not affect runtime behavior.

src/scss/6-elements/_button.scss (missing trailing newline)

Important Files Changed

Filename Overview
src/lib/components/shared/github-stats.svelte Replaces star icon + 'Star on GitHub' text with GitHub icon + aria-label; adds is-github-stat-link class with safe class-list merge.
src/lib/layouts/Docs.svelte Applies same GitHub button redesign (github icon, aria-label, new CSS class) as other layouts.
src/lib/layouts/Main.svelte GitHub button update; conditional aria-label correctly omits star count when isOfferPage disables the link.
src/lib/layouts/Sidebar.svelte GitHub button update; also incidentally fixes a pre-existing bug where class="star" was used instead of name="github" on the Icon component.
src/routes/(marketing)/+page.svelte Logo list headline changed from 'Loved by startups and world leaders' to 'Trusted by developer teams worldwide'.
src/routes/[variation]/+page.svelte Logo list headline updated to match the main marketing page copy.
src/routes/community/+page.svelte GitHub button updated to icon-only style with accessible aria-label.
src/routes/products/messaging/+page.svelte Removes the BETA badge span from the messaging product page hero.
src/scss/6-elements/_button.scss Adds .is-github-stat-link variant styles for the compact icon+count GitHub button; missing trailing newline.
src/scss/7-components/_main-header.scss Removes the rule that was hiding InlineTag in the header, now that the star count is meant to be visible alongside the GitHub icon.

Reviews (2): Last reviewed commit: "Refactor GitHub stats links across compo..." | Re-trigger Greptile

- Updated GitHub stat links in `github-stats.svelte`, `Docs.svelte`, `Main.svelte`, `Sidebar.svelte`, and `community/+page.svelte` to use the GitHub icon instead of the star icon.
- Added `is-github-stat-link` class and improved accessibility with `aria-label` attributes for better user experience.
- Cleaned up unused code in `Main.svelte` and adjusted styles in `_button.scss` for GitHub stat links.

These changes enhance the visual consistency and accessibility of GitHub links throughout the application.
@eldadfux eldadfux merged commit 3a4573c into main Apr 30, 2026
6 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