Skip to content

fix: update showcase company data#13535

Merged
abdallahshaban557 merged 3 commits into
mainfrom
fix-images-showcases
Jun 24, 2026
Merged

fix: update showcase company data#13535
abdallahshaban557 merged 3 commits into
mainfrom
fix-images-showcases

Conversation

@abdallahshaban557

@abdallahshaban557 abdallahshaban557 commented Jun 24, 2026

Copy link
Copy Markdown
Contributor

Description of what this PR is changing or adding, and why:

Update the showcase company logos in the main /showcase page.

Issues fixed by this PR (if any):

PRs or commits this PR depends on (if any):

Presubmit checklist

  • If you are unwilling, or unable, to sign the CLA, even for a tiny, one-word PR, please file an issue instead of a PR.
  • If this PR is not meant to land until a future stable release, mark it as draft with an explanation.
  • This PR follows the Google Developer Documentation Style Guidelines—for example, it doesn't use i.e. or e.g., and it avoids I and we (first-person pronouns).
  • This PR uses semantic line breaks
    of 80 characters or fewer.

@abdallahshaban557 abdallahshaban557 requested a review from a team as a code owner June 24, 2026 21:54

@gemini-code-assist gemini-code-assist Bot left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Code Review

This pull request removes two SVG files, deletes Bytedance from the showcase companies list, and updates the card image path for the Karaca showcase to use the company logo. The reviewer correctly pointed out that using the company logo as the card image might cause layout or scaling issues on the showcase page due to aspect ratio requirements, and recommended using a properly proportioned card image instead.

Comment thread sites/www/content/showcase/karaca.md
@flutter-website-bot

flutter-website-bot commented Jun 24, 2026

Copy link
Copy Markdown
Collaborator

Staged preview of the updated docs.flutter.dev site (updated for commit 31c4c30):

https://flutter-docs-prod--docs-pr13535-fix-images-showcases-5pgqpf8z.web.app

@flutter-website-bot

Copy link
Copy Markdown
Collaborator

Staged preview of the updated flutter.dev site (updated for commit 31c4c30):

https://flutter-dev-230821--www-pr13535-fix-images-showcases-fewk5xu3.web.app

@lamek lamek left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

LGTM

@abdallahshaban557 abdallahshaban557 merged commit 01be764 into main Jun 24, 2026
15 checks passed
@abdallahshaban557 abdallahshaban557 deleted the fix-images-showcases branch June 24, 2026 23:02
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.

3 participants