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

feat(languages): use github language names and add new icons #2270

Open
wants to merge 1 commit into
base: develop
Choose a base branch
from

Conversation

ReenigneArcher
Copy link

@ReenigneArcher ReenigneArcher commented Sep 15, 2024

Double check these details before you open a PR

  • PR does not match another non-stale PR currently opened

Features

This PR adds language icons and alias that match the naming GitHub uses in the GitHub API. GitHub uses "linguist" for the languages API. Their list of languages can be found here: https://github.com/github-linguist/linguist/blob/master/lib/linguist/languages.yml

The advantage of this is it makes it possible to use icons directly, without creating a custom mapping.

  • Many new icons, from https://github.com/LizardByte/uno/tree/master/gh-pages-template/language-icons
  • powershell icon updated
  • current nginx icon moved to wordmark icon and added new original icon
  • jest-plain renamed to jest-original
  • fixed graphql "plain" icons -> original
  • re-ordered a couple of out of order icons, like windows8 and reactrouter
  • some new altnames added (taken from the linguist list)
  • some tags added or changed from existing icons (taken from the linguist list)

This PR closes NONE

Notes

I understand that typically new icons are probably submitted one per PR; however since this is somewhat of a new feature (names to match github API), I'd really rather avoid 230 PRs.

The colors for new icons were selected using a color picker. If there was a more prominent color, I selected that, otherwise it may be a random color in some cases (where an icon may have many colors).

I have a question on "graphql". There is no "original" icon, and no alias to it. Is that correct? Edit: I updated the graphql icons based on the comment below.

@ReenigneArcher ReenigneArcher force-pushed the feat/languages/use-github-language-names-and-add-new-icons branch 4 times, most recently from f65dd52 to 011bf7c Compare September 15, 2024 02:21
@Snailedlt
Copy link
Collaborator

Snailedlt commented Sep 15, 2024

@ReenigneArcher all icons should have an original version, that includes graphql. So that's not intended

Edit: The graphql icons should have been named "graphql-original" and "graphql-original-wordmark", and had aliases to "plain" and "plain-wordmark" respectively.

@ReenigneArcher ReenigneArcher force-pushed the feat/languages/use-github-language-names-and-add-new-icons branch 16 times, most recently from 6cdf663 to bc73c6f Compare September 16, 2024 02:48
@ReenigneArcher ReenigneArcher force-pushed the feat/languages/use-github-language-names-and-add-new-icons branch from bc73c6f to 2b6372c Compare September 16, 2024 20:12
@ReenigneArcher ReenigneArcher marked this pull request as ready for review September 16, 2024 20:16
@ReenigneArcher
Copy link
Author

This PR is ready for review.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants