Skip to content

feat(docs): Update README.md#8446

Closed
iamdarshshah wants to merge 3 commits intofacebook:mainfrom
iamdarshshah:patch-1
Closed

feat(docs): Update README.md#8446
iamdarshshah wants to merge 3 commits intofacebook:mainfrom
iamdarshshah:patch-1

Conversation

@iamdarshshah
Copy link
Copy Markdown

Changes

  • Added Flow logo (image source: https://opensource.facebook.com/projects)
  • Aligned header elements to the center

This will make it look a lot cleaner. It's good to include a logo in the docs, thus added it to the top.

### Changes
- [x] Added Flow logo
- [x] Aligned header elements to the center

> This will make it look a lot cleaner.
Copy link
Copy Markdown
Contributor

@facebook-github-bot facebook-github-bot left a comment

Choose a reason for hiding this comment

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

@mroch has imported this pull request. If you are a Facebook employee, you can view this diff on Phabricator.

@mroch
Copy link
Copy Markdown
Contributor

mroch commented Jul 27, 2020

indeed, looks nice. thanks!

@iamdarshshah
Copy link
Copy Markdown
Author

Thanks @mroch 👍

@Brianzchen
Copy link
Copy Markdown
Contributor

@iamdarshshah can you update this PR? It has a conflict since #8454 got merged recently

@iamdarshshah
Copy link
Copy Markdown
Author

iamdarshshah commented Aug 26, 2020

@Brianzchen Thanks for letting me know. PR is now Updated 👍

@Brianzchen
Copy link
Copy Markdown
Contributor

@iamdarshshah the image you've used doesn't seem very stable, looks like the url expires over time?

@gkz
Copy link
Copy Markdown
Member

gkz commented Jan 12, 2023

Close in favor of other PR

@gkz gkz closed this Jan 12, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants