Skip to content

Feature/logos#24

Open
Dirk (d-rk) wants to merge 2 commits intoduolingo:masterfrom
d-rk:feature/logos
Open

Feature/logos#24
Dirk (d-rk) wants to merge 2 commits intoduolingo:masterfrom
d-rk:feature/logos

Conversation

@d-rk
Copy link
Copy Markdown
Contributor

Hey,

this is the final thing I modified and wanted to contribute back.
I added logos for all the engines:
Screenshot from 2022-02-04 09-57-32
Screenshot from 2022-02-04 09-57-49

@jwpjrdev
Copy link
Copy Markdown

Lingo is https://lingoapp.com, not Duolingo

@d-rk
Copy link
Copy Markdown
Contributor Author

Joshua Price (@jwpjrdev) thank you for the hint. I updated the logo

@artnc
Copy link
Copy Markdown
Member

Wow! I'm curious, where did you get the images from? One reason why I hadn't done this being that I was never sure about committing to also providing images for all new engines going forward

@d-rk
Copy link
Copy Markdown
Contributor Author

Art Chaidarun (@artnc) actually that was quite cumbersome 🙈 I first just added it for the services we are using internally. But for this PR I had to find all logos, which took some time. I mainly checked the brand/marketing pages of the companies for the logos. But for some of them I had to grab them elsewhere.

I think going forward this shouldn't be a problem, because for a new engine that needs to be added only a single logo has to be found.

Copy link
Copy Markdown

@YuzeHao2023 Halamji (YuzeHao2023) left a comment

Choose a reason for hiding this comment

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

lgtm

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Development

Successfully merging this pull request may close these issues.

4 participants