Update Together AI logo#268
Merged
Merged
Conversation
Contributor
Reviewer's GuideUpdates all Together AI SVG logo components (brand, text, mono, and color icons) to the new rebranded artwork and primary color, including adjusting viewBox, vector paths, and primary color constant. Updated class diagram for Together icon components and style constantsclassDiagram
class TogetherStyle {
+string TITLE
+number COMBINE_TEXT_MULTIPLE
+number COMBINE_SPACE_MULTIPLE
+string COLOR_PRIMARY
+string AVATAR_BACKGROUND
}
class BrandColorIcon {
+Icon(size, style, rest)
+svg viewBox_0_0_105_24
+group_scaled_logo_paths
}
class BrandMonoIcon {
+Icon(size, style, rest)
+svg viewBox_0_0_105_24
+group_scaled_logo_paths_mono
}
class TextIcon {
+Icon(size, style, rest)
+svg viewBox_0_0_105_24
+group_scaled_text_logo_paths
}
class ColorIcon {
+Icon(size, style, rest)
+svg viewBox_0_0_24_24
+group_scaled_mark_color_paths
}
class MonoIcon {
+Icon(size, style, rest)
+svg viewBox_0_0_24_24
+group_scaled_mark_mono_paths
}
TogetherStyle <.. BrandColorIcon : uses TITLE
TogetherStyle <.. BrandMonoIcon : uses TITLE
TogetherStyle <.. TextIcon : uses TITLE
TogetherStyle <.. ColorIcon : uses TITLE
TogetherStyle <.. MonoIcon : uses TITLE
TogetherStyle <.. ColorIcon : uses COLOR_PRIMARY
TogetherStyle <.. MonoIcon : uses COLOR_PRIMARY
TogetherStyle <.. BrandColorIcon : uses COLOR_PRIMARY
TogetherStyle <.. BrandMonoIcon : uses COLOR_PRIMARY
TogetherStyle <.. TextIcon : uses COLOR_PRIMARY
File-Level Changes
Tips and commandsInteracting with Sourcery
Customizing Your ExperienceAccess your dashboard to:
Getting Help
|
Member
|
👍 @Nutlope |
Contributor
There was a problem hiding this comment.
Hey - I've left some high level feedback:
- In
Color.tsxyou removedfill="currentColor", which means the icon will no longer inherit color from the surrounding text/theme; consider whether this is intentional or if the new SVG should still be themeable viacurrentColor. - The new SVGs use hard-coded
transformscales (e.g.,scale(0.12486)andscale(0.04949)); it may be worth extracting these into named constants or comments so future maintainers understand how these values were derived and can adjust them more safely. - The
viewBoxwidth change from116to105in the brand components can affect layout for consumers relying on the previous intrinsic aspect ratio—if the intent is to preserve sizing behavior, double-check that this new width matches how the logo is expected to render in existing UI contexts.
Prompt for AI Agents
Please address the comments from this code review:
## Overall Comments
- In `Color.tsx` you removed `fill="currentColor"`, which means the icon will no longer inherit color from the surrounding text/theme; consider whether this is intentional or if the new SVG should still be themeable via `currentColor`.
- The new SVGs use hard-coded `transform` scales (e.g., `scale(0.12486)` and `scale(0.04949)`); it may be worth extracting these into named constants or comments so future maintainers understand how these values were derived and can adjust them more safely.
- The `viewBox` width change from `116` to `105` in the brand components can affect layout for consumers relying on the previous intrinsic aspect ratio—if the intent is to preserve sizing behavior, double-check that this new width matches how the logo is expected to render in existing UI contexts.Help me be more useful! Please click 👍 or 👎 on each comment and I'll use the feedback to improve your reviews.
Member
|
❤️ Great PR @Nutlope ❤️ |
canisminor1990
added a commit
that referenced
this pull request
Mar 26, 2026
* ✨ feat(icons): add Qoder brand icon (#270) * ✨ feat(icons): add Qoder brand icon - Add Qoder icon components (Avatar, Color, Mono, Text) - Add Qoder icon documentation - Export Qoder icon in icons.ts - Add Qoder entry in table of contents new file: src/Qoder/components/Avatar.tsx new file: src/Qoder/components/Color.tsx new file: src/Qoder/components/Mono.tsx new file: src/Qoder/components/Text.tsx new file: src/Qoder/index.md new file: src/Qoder/index.ts new file: src/Qoder/style.ts modified: src/icons.ts modified: src/toc.ts * ✨ feat: add Qoder icon - Add Qoder icon with Avatar, Color, Mono, and Text variants - Export Qoder icon from react-native package * ✨ feat: add Coding Plan provider icons support (#272) * ✨ feat: add Coding Plan provider support * ✨ feat: add Coding Plan provider support for React Native * feat: add Apertis icon (#269) * feat: add Apertis icon Add Apertis (apertis.ai) — a unified API platform for 500+ AI models across 30+ providers. Includes Mono, Text, Avatar, and Combine components following the standard icon package structure. Primary color: #0d9488 (Apertis Teal) * feat: export Apertis icon and add react-native support Add Apertis to src/icons.ts barrel export and create the full react-native icon set (Mono, Text, Avatar, Combine) under packages/react-native/src/icons/Apertis. * Update Together AI logo (#268) * wip * wip * Revert "wip" This reverts commit 76dfa33. * wip * feat: add AskVerdict AI brand icons (#265) Add AskVerdict AI (askverdict.ai) — multi-agent AI debate engine. Components: Mono, Color (gold gradient), Text, Combine, Avatar Shared SVG paths extracted to paths.ts to prevent divergence. Brand color: #E8A317 (gold) Website: https://askverdict.ai * ✨ feat: add AgentVoice icon (#257) Co-authored-by: RRyanHoward <ryan@websitellm.com> Co-authored-by: Claude Opus 4.6 <noreply@anthropic.com> * feat: merge PR #259 (LLM API) and PR #262 (Xpay) into dev (#276) * ✨ feat: Add LLM API logo * ✨ feat: add xpay icon * fix: set hasBrandColor to true in toc entry --------- Co-authored-by: Markos Basilio <markosbasilio@MacBook-Pro-Markos.local> Co-authored-by: xPay <xpaysh@gmail.com> Co-authored-by: copilot-swe-agent[bot] <198982749+Copilot@users.noreply.github.com> * 💄 style: Fix some icon style * ✨ feat: Update rn --------- Co-authored-by: zhi <yantze@126.com> Co-authored-by: Hardy <yimingmail@yandex.com> Co-authored-by: Quert <a97041304@gmail.com> Co-authored-by: Hassan El Mghari <hassan4709@gmail.com> Co-authored-by: GDS K S <39922405+thegdsks@users.noreply.github.com> Co-authored-by: Ryan Howard <ryan@agentvoice.com> Co-authored-by: RRyanHoward <ryan@websitellm.com> Co-authored-by: Claude Opus 4.6 <noreply@anthropic.com> Co-authored-by: Copilot <198982749+Copilot@users.noreply.github.com> Co-authored-by: Markos Basilio <markosbasilio@MacBook-Pro-Markos.local> Co-authored-by: xPay <xpaysh@gmail.com>
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
3 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.
💻 变更类型 | Change Type
🔀 变更说明 | Description of Change
Hi, my name is Hassan from Together AI. We recently went through a rebrand and I submitted this PR to update our logo. You can find more details about the rebrand and our latest brand assets at together.ai/brand. Thank you!
📝 补充信息 | Additional Information
Summary by Sourcery
Update Together AI branding assets across logo components to match the latest rebrand.
Bug Fixes:
Enhancements: