Skip to content

feat(marketing): refresh trusted-by logo strip#2857

Merged
atharvadeosthale merged 2 commits intomainfrom
feat/trusted-by-logo-strip
Apr 24, 2026
Merged

feat(marketing): refresh trusted-by logo strip#2857
atharvadeosthale merged 2 commits intomainfrom
feat/trusted-by-logo-strip

Conversation

@eldadfux
Copy link
Copy Markdown
Member

@eldadfux eldadfux commented Apr 3, 2026

Replace several homepage and shared LogoList assets with grey SVGs for Times of India, First Media, Acer, DevKind, LangX, and K-collect, and remove unused Apple, TikTok, and Intel logos. Update the homepage headline copy, keep the marketing logo row title on one line, and scale logos down slightly with wider grid gaps for clearer spacing.

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

Replace several homepage and shared LogoList assets with grey SVGs for
Times of India, First Media, Acer, DevKind, LangX, and K-collect, and
remove unused Apple, TikTok, and Intel logos. Update the homepage
headline copy, keep the marketing logo row title on one line, and scale
logos down slightly with wider grid gaps for clearer spacing.

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

greptile-apps Bot commented Apr 3, 2026

Greptile Summary

This PR refreshes the "trusted by" logo strip across the homepage and shared LogoList components, swapping in grey SVGs for Times of India, First Media, Acer, DevKind, LangX, and K-collect while removing Apple, TikTok, and Intel assets. It also updates the headline copy and tightens logo sizing with wider grid gaps.

All new SVG files are present, self-contained (no broken url(#…) references), and render correctly within their declared viewBoxes. The only loose end is that deloitte.svg, nestle.svg, and oracle.svg were removed from the logo arrays but their static files were not deleted alongside apple.svg, intel.svg, and tiktok.svg.

Confidence Score: 5/5

Safe to merge — purely visual/asset changes with no logic regressions.

All findings are P2; no correctness, data, or security concerns. New SVGs are valid and all referenced files exist.

No files require special attention; the leftover SVG assets (deloitte, nestle, oracle) are cosmetic cleanup only.

Important Files Changed

Filename Overview
src/lib/components/LogoList.svelte Logo array refreshed (apple/oracle/tiktok/intel/deloitte/nestle → times-of-india/devkind/first-media/acer/langx/k-collect); grid gap widths increased; all new SVGs present in repo.
src/routes/(marketing)/(components)/logo-list.svelte Default title updated; same logo array refresh as shared component; mobile max-w cap on images tightened from 80px to 68px; md:whitespace-nowrap replaces previous max-width constraint on heading.
src/routes/(marketing)/+page.svelte Headline copy updated to "Loved by hackers, startups and enterprises alike"; minor formatting only.
static/images/logos/trusted-by/acer.svg New grey (#ADADB0) SVG; large coordinate system (viewBox 0 0 4151.1 1000, intrinsic 415.11×100) but renders correctly when displayed at target dimensions via <img>.
static/images/logos/trusted-by/devkind.svg New grey SVG using luminance masks; clipPath and all referenced IDs are self-contained in the file.
static/images/logos/trusted-by/first-media.svg New grey SVG; uses id="first-media-clip" for its clipPath — no ID collision risk since these are loaded as <img> (separate documents, not inline SVG).
static/images/logos/trusted-by/k-collect.svg New grey SVG; references url(#pattern0_1827_96721) which is defined in the file's <defs> — no broken reference.
static/images/logos/trusted-by/langx.svg New grey SVG; uses a uniquely-named id="langx-trusted-by-clip" clipPath to avoid cross-document collisions.
static/images/logos/trusted-by/times-of-india.svg New grey SVG using large absolute transforms that cancel out correctly; all paths render within the declared 144.965×52.53 viewBox.
static/images/logos/trusted-by/apple.svg Deleted — no longer referenced in either LogoList component.
static/images/logos/trusted-by/tiktok.svg Deleted — no longer referenced in either LogoList component.
static/images/logos/trusted-by/intel.svg Deleted — no longer referenced in either LogoList component.

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

Comment thread src/routes/(marketing)/(components)/logo-list.svelte Outdated
@atharvadeosthale atharvadeosthale merged commit 33e6d7e into main Apr 24, 2026
6 checks passed
@atharvadeosthale atharvadeosthale deleted the feat/trusted-by-logo-strip branch April 24, 2026 20:16
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.

2 participants