Skip to content

feat: add The Lawyers Global icon with 8 variants#39

Merged
thegdsks merged 3 commits intomainfrom
feat/the-lawyers-global-icon
Mar 17, 2026
Merged

feat: add The Lawyers Global icon with 8 variants#39
thegdsks merged 3 commits intomainfrom
feat/the-lawyers-global-icon

Conversation

@thegdsks
Copy link
Member

Summary

  • Adds The Lawyers Global (TLG) brand icon with 8 SVG variants
  • Variants: default (blue), light (white bg), dark (black bg), mono (grey), color (rose), gold, wordmark, wordmarkDark
  • Brand hex: #031F3D

Closes #36

Copilot AI review requested due to automatic review settings March 17, 2026 18:33
@vercel
Copy link

vercel bot commented Mar 17, 2026

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated (UTC)
thesvg Ready Ready Preview, Comment Mar 17, 2026 6:55pm

Request Review

@thegdsks thegdsks self-assigned this Mar 17, 2026
@greptile-apps
Copy link

greptile-apps bot commented Mar 17, 2026

Greptile Summary

This PR adds The Lawyers Global (TLG) brand icon to the repository with 8 SVG variants (default, light, dark, mono, color, gold, wordmark, wordmarkDark) and a corresponding entry in src/data/icons.json. All 8 SVGs are well-formed and render correctly for their intended use cases. The main issue is a filename convention violation in the wordmarkDark variant.

Changes:

  • 8 new SVG files added under public/icons/the-lawyers-global/
  • New entry added to src/data/icons.json with correct slug, brand hex (031F3D), alias (TLG), license (custom), URL, and guidelines link

Issues found:

  • wordmarkDark.svg uses camelCase instead of the repository's kebab-case convention for multi-word filenames. Every other icon with a dark wordmark variant uses wordmark-dark.svg (e.g., ahooks, ahrefs, anthropic). The file should be renamed to wordmark-dark.svg and the icons.json path updated accordingly.

Confidence Score: 4/5

  • Safe to merge after renaming wordmarkDark.svg to wordmark-dark.svg and updating the JSON path reference.
  • All SVGs are valid and render correctly. The icons.json entry is well-structured. The only issue is a filename convention violation (wordmarkDark.svg vs. the expected wordmark-dark.svg) that is easy to fix.
  • public/icons/the-lawyers-global/wordmarkDark.svg (needs rename to wordmark-dark.svg) and the corresponding path in src/data/icons.json.

Important Files Changed

Filename Overview
public/icons/the-lawyers-global/default.svg Valid square SVG (1400×1400) with brand navy blue (#031f3d) background and white paths; uses Illustrator-generated UUID class names scoped within the file.
public/icons/the-lawyers-global/wordmark.svg Valid wide-format wordmark SVG (5200×925) with navy blue (#031f3d) text on transparent background; correct wordmark structure.
public/icons/the-lawyers-global/wordmarkDark.svg Valid wide-format wordmark SVG (5200×925) with white text on transparent background, but the filename uses camelCase (wordmarkDark.svg) instead of the repository-wide kebab-case convention (wordmark-dark.svg).
src/data/icons.json New TLG entry is well-structured with correct slug, hex, aliases, license, and guidelines; however, the wordmarkDark variant path references a camelCase filename (wordmarkDark.svg) that diverges from the kebab-case convention used by every other icon with a wordmarkDark variant.

Flowchart

%%{init: {'theme': 'neutral'}}%%
flowchart TD
    A[icons.json entry\nslug: the-lawyers-global] --> B[Square variants\n1400×1400 viewBox]
    A --> C[Wordmark variants\n5200×925 viewBox]

    B --> D[default.svg\n#031f3d bg / white text]
    B --> E[light.svg\nwhite bg / #031f3d text]
    B --> F[dark.svg\nblack bg / white text]
    B --> G[mono.svg\n#3b4757 bg / white text]
    B --> H[color.svg\n#fa0340 bg / white text]
    B --> I[gold.svg\n#b4a168 bg / white text]

    C --> J[wordmark.svg\n#031f3d text / transparent]
    C --> K[wordmarkDark.svg ⚠️\nwhite text / transparent\nshould be wordmark-dark.svg]
Loading

Last reviewed commit: 56f2f42

Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

Adds a new brand entry (“The Lawyers Global”) to the icon registry and introduces the corresponding SVG assets under public/icons/ so the icon can be served via the site/API and packaged downstream.

Changes:

  • Added the-lawyers-global metadata entry in src/data/icons.json with multiple variants and brand guidelines URL
  • Added new SVG variant files for the icon (default/light/dark/mono/color/gold/wordmark/wordmarkDark)

Reviewed changes

Copilot reviewed 1 out of 9 changed files in this pull request and generated 2 comments.

Show a summary per file
File Description
src/data/icons.json Registers the new icon slug, metadata, and variant paths
public/icons/the-lawyers-global/default.svg Adds default square logo SVG
public/icons/the-lawyers-global/light.svg Adds light-background square logo SVG
public/icons/the-lawyers-global/dark.svg Adds dark-background square logo SVG
public/icons/the-lawyers-global/mono.svg Adds mono variant SVG (currently not themeable as “mono”)
public/icons/the-lawyers-global/color.svg Adds additional color variant SVG
public/icons/the-lawyers-global/gold.svg Adds additional gold variant SVG
public/icons/the-lawyers-global/wordmark.svg Adds inline wordmark SVG
public/icons/the-lawyers-global/wordmarkDark.svg Adds inline wordmark SVG (currently appears to be the white version)

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

@thegdsks thegdsks merged commit 4540925 into main Mar 17, 2026
2 of 3 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.

[Icon Request] TLG LOGO SQUARE BLUE WHITE

2 participants