Template talk:GitHub link

From OpenStreetMap Wiki
Jump to navigation Jump to search

Icon size (black cat)

Currently there is a setup for a fix size of 20px Font Awesome 5 brands github.svg. I propose to reduce this a little bit, e.g. to 17px Font Awesome 5 brands github.svg

Or somebody could modify the template to use a variable for the size:
the current no size-variable:
{{GitHub link|openstreetmap/openstreetmap-website|OpenStreetMap Website}}
e.g. to a new version with size-variable:
{{GitHub link|openstreetmap/openstreetmap-website|OpenStreetMap Website|10px}} --MalgiK (talk) 13:36, 20 February 2020 (UTC)

Agree, and OOjs UI icon check-constructive.svg reduced, to 16px even. —M!dgard [ talk ] 10:32, 21 February 2020 (UTC)
Thank you - especially for the text-bottom alignment. --MalgiK (talk) 20:12, 21 February 2020 (UTC)

What is the point of the template?

It adds ugly icon, it replaces standard link by unusual template... Why it is preferable at all over regular links? Mateusz Konieczny (talk) 22:28, 11 February 2021 (UTC)

I removed the github branding - it is not like we are paid by Microsoft for putting their branding on OSM wiki and there was benefit to making github links so prominent. Again, what is the benefit of templatefying normal links? Mateusz Konieczny (talk) 12:46, 17 February 2023 (UTC)

@Mateusz Konieczny: I agree that this template is generally unnecessary, but it can be convenient when making a long list or table of GitHub links, to avoid redundant external link icons everywhere and ensure consistent link text.

It would be useful to have some kind of indicator that the link goes to GitHub. Otherwise, by default, it's just the name of the repository without any additional context. It would get especially confusing in an "External links" section, where a GitHub link typically goes right beneath the link to an ordinary website representing the organization or project, now with identical link text. We shouldn't put branding here, but a simple "on GitHub" would be fine.

 – Minh Nguyễn 💬 23:32, 3 June 2023 (UTC)