Skip to content
This repository has been archived by the owner on Apr 10, 2023. It is now read-only.

Add whereby logo as svg file in readme #6

Merged
merged 1 commit into from
Aug 27, 2021

Conversation

br1anchen
Copy link
Contributor

Since github markdown does not support inline svg, change it to whereby-logo.svg file.

@br1anchen br1anchen requested a review from estebanwb August 27, 2021 08:07
Copy link
Contributor

@estebanwb estebanwb left a comment

Choose a reason for hiding this comment

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

LGTM, but see suggestion.

<path d="m51.2641 6.28927c-1.1643 0-2.4356 1.2466-3.0282 2.43006v-2.55632h-.1691c-.7202 1.03538-2.4149 1.64835-4.0027 1.62654v.16874c.8687.25368 1.2921.95043 1.2921 2.17641v5.0288c0 .9722-.4234 1.7321-1.2702 2.1132v.1688h5.4638v-.1688c-.8686-.3799-1.2494-1.141-1.2494-2.1132v-5.91615c.3175-.40176.6776-.82418 1.101-.82418.7835 0 1.1011.95044 2.0756.95044.6776 0 1.3771-.54983 1.3771-1.47961-.0023-.92863-.7432-1.60473-1.59-1.60473z"></path><path d="m43.0884 9.28982c0-1.73214-1.7788-3.00054-4.0453-3.00054-3.4309 0-5.7607 2.95807-5.7607 6.06422 0 3.3805 2.5841 5.4099 5.1463 5.4099 2.0549 0 4.0453-1.2466 4.6597-3.4229l-.1485-.1263c-.6983 1.0354-1.9064 1.6059-3.1133 1.6059-1.5889 0-2.8809-.9091-3.4941-2.5357 3.2824-.5291 6.7559-1.4176 6.7559-3.99458zm-6.9043 3.48608c-.1266-.5073-.191-1.056-.191-1.6265 0-2.74688 1.1643-4.28963 2.6048-4.28963 1.1229 0 1.483.95044 1.483 1.96516 0 2.11207-1.9271 3.52857-3.8968 3.95097z"></path>
</g>
</svg>
<img src="./whereby-logo.svg" />
Copy link
Contributor

Choose a reason for hiding this comment

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

Nice! I think we can actually simplify this whole block by just using the code below:

![Whereby logo](./whereby-logo.svg)

# Whereby Embedded examples

Haven't tested it so it might need tweaking, and the heading won't be centered... but it's a lot less code and looks more like what most people expect from a README, I think.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Yeah, but the reason to use html is to apply center style in README.md though...
I would not say most people expect no styling in README in my opinion.

@br1anchen br1anchen merged commit 9b04d4b into main Aug 27, 2021
@br1anchen br1anchen deleted the brianchen/add-whereby-logo-readme branch August 27, 2021 11:25
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.

2 participants