Fix missing simpleicons-based windows logo by standalone version #73
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
TO: @Ralim
CC: @gamelaster
TL;DR: for some reason since Window logo did disappear from simpleicons.org database, the easiest and the simplest way seems to just replace it with the local version for the table of supported devices.
Currently vs Patched.
It's strange but you can't find any windows nor microsoft related icons through simpleicons anymore. I did try to utilize
svg
version from fontawesome, like this:But this doesn't work with GitHub'based Markdown flavor (for security reasons to avoid malicious payloads and/or content from 3rd party sources which may be compromised, as far as I could understand).
So the straightforward option is to:
svg
version of fontawesome;svg
;png
;img/
;img src=img/...
.Hence, here is the suggestion in this patch.
For the record, due to such kind of exact nuances I'm not a big fan myself of turning markdown files into html much (because the whole idea of markdown is not to be html much). But since there is a nicely done table already, this is a proposal fix for the broken window logo (sweet irony, punch intended :)
Let me know what you think.