Skip to content

Update Flutter logo image sources in README#186195

Merged
johnpryan merged 2 commits intoflutter:masterfrom
johnpryan:update-logo-in-readme
May 7, 2026
Merged

Update Flutter logo image sources in README#186195
johnpryan merged 2 commits intoflutter:masterfrom
johnpryan:update-logo-in-readme

Conversation

@johnpryan
Copy link
Copy Markdown
Contributor

@johnpryan johnpryan commented May 7, 2026

This updates the light and dark mode images in the README.

The Cloud Storage bucket that was being used was turned down. This is the same image that was used before (courtesy of Wayback Machine

@johnpryan johnpryan requested a review from jtmcdole May 7, 2026 16:39
@flutter-dashboard flutter-dashboard Bot added the CICD Run CI/CD label May 7, 2026
@johnpryan
Copy link
Copy Markdown
Contributor Author

Something doesn't look right, the text is white in the image. Investigating...

Copy link
Copy Markdown
Contributor

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

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 updates the Flutter logo image source in README.md. Feedback indicates that the dark mode source URL remains pointed at the old storage bucket and requires an update to prevent a broken image for dark mode users.

Comment thread README.md
@johnpryan
Copy link
Copy Markdown
Contributor Author

I published the wrong image. flutter/website#13372 should fix this.

@johnpryan
Copy link
Copy Markdown
Contributor Author

Okay, there's actually two images required here. one for dark mode and one for light mode. I will fix this in flutter/website#13372

@github-actions github-actions Bot removed the CICD Run CI/CD label May 7, 2026
@johnpryan
Copy link
Copy Markdown
Contributor Author

This is ready for review, flutter/website#13372 will ensure that the dark mode image is working too.

@johnpryan johnpryan changed the title Update Flutter logo image source in README Update Flutter logo image sources in README May 7, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

CICD Run CI/CD

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants