Skip to content

Update README.md#49078

Merged
timsneath merged 2 commits intomasterfrom
timsneath-logo-fix
Jan 17, 2020
Merged

Update README.md#49078
timsneath merged 2 commits intomasterfrom
timsneath-logo-fix

Conversation

@timsneath
Copy link
Copy Markdown
Contributor

Looks like the Flutter logo got deleted from flutter.dev/assets. Redirecting to a new location.

Looks like the Flutter logo got deleted from flutter.dev/assets. Redirecting to a new location.
@timsneath timsneath requested review from Piinks and removed request for Piinks January 17, 2020 20:42
Copy link
Copy Markdown
Contributor

@Piinks Piinks left a comment

Choose a reason for hiding this comment

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

LGTM

@timsneath timsneath merged commit 12e52bd into master Jan 17, 2020
@timsneath timsneath deleted the timsneath-logo-fix branch January 17, 2020 21:43
@egorbelibov
Copy link
Copy Markdown

egorbelibov commented Jan 18, 2020

Not sure if it's intended or not, so just pointing out:
The logo that was deleted was about 37 pixels high. The new one is much bigger. Not sure if, from a design perspective, makes sense having that big of a logo. 🙃

@timsneath
Copy link
Copy Markdown
Contributor Author

timsneath commented Jan 18, 2020 via email

@github-actions github-actions Bot locked as resolved and limited conversation to collaborators Aug 2, 2021
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants