Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

redesign readme.md #496

Merged
merged 7 commits into from
Feb 21, 2021
Merged

redesign readme.md #496

merged 7 commits into from
Feb 21, 2021

Conversation

amacado
Copy link
Member

@amacado amacado commented Feb 21, 2021

So I reworked the README to give it a cleaner, fresher look. You can check it in my branch.

image

Happy to hear your feedback on this! :)

@github-actions

This comment has been minimized.

@devicons devicons deleted a comment from github-actions bot Feb 21, 2021
@github-actions
Copy link
Contributor

Hi!

I'm Devicons' SVG-Checker Bot and it seems we've ran into a problem. I'm supposed to check your svgs but I couldn't do my task due to an issue.

Please let my maintainers know of the issues. They will take a look at my work and try to resolve the problem. Until then, please hang tight and sorry for the inconvenience.

Cheers,
SVG-Checker Bot 😄

Thomas-Boi
Thomas-Boi previously approved these changes Feb 21, 2021
Copy link
Member

@Thomas-Boi Thomas-Boi left a comment

Choose a reason for hiding this comment

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

Wow, I really like the redesign 👍. It looks professional and well-formatted. Great job amacado!

@Thomas-Boi
Copy link
Member

Oh, I just notice one thing:

image

What is this acronym? I am familiar with a similar one called "TL;DR", which stands for "too long; didn't read". Is this a different acronym?

@amacado
Copy link
Member Author

amacado commented Feb 21, 2021

TL;TR = To Long; To Read, but TL;DR; is fine too.. ;-)

@github-actions

This comment has been minimized.

@amacado amacado requested a review from Thomas-Boi February 21, 2021 21:37
@github-actions

This comment has been minimized.

@amacado
Copy link
Member Author

amacado commented Feb 21, 2021

@Thomas-Boi I renamed the headline to TL;DR; this is more familiar.. Thanks for pointing out. Can you review again? :)

Copy link
Member

@Thomas-Boi Thomas-Boi left a comment

Choose a reason for hiding this comment

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

Everything looks 😄

@amacado amacado merged commit f2145c2 into develop Feb 21, 2021
@amacado amacado deleted the amacado/feature/update-readme branch February 21, 2021 22:12
@amacado amacado mentioned this pull request Feb 21, 2021
4 tasks
GCHQDeveloper926 pushed a commit to GCHQDeveloper926/devicon that referenced this pull request Dec 20, 2024
* update readme for a fresher look

* add target="blank" and links to badges

* add more badges

* replace depreceated bower with yarn

* fix typo in headline
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.

2 participants