The Wayback Machine - https://web.archive.org/web/20210210194453/https://github.com/github/linguist/pull/4978
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

Abolish restrictions governing colour choices #4978

Merged
merged 1 commit into from Sep 1, 2020
Merged

Abolish restrictions governing colour choices #4978

merged 1 commit into from Sep 1, 2020

Conversation

@Alhadis
Copy link
Collaborator

@Alhadis Alhadis commented Aug 30, 2020

This is the first of three PRs to fix #4291 and #4506, both of which are blocked on this change. I won't repeat what those threads already do a good job of explaining.

Status of related tasks:

  1. Abolish restrictions governing colour choices (this)
  2. Add/fix incorrect or missing language colours
  3. Ungroup major languages

The problem

We're still enforcing colour-difference thresholds, but nobody can explain why. _"Still waiting on GitHub's designers" isn't a sound justification when they've had 1.5 years to raise an objection to the colour-test's removal.

The solution

Stop giving a crap and just go for it. Chances are the test appears 10 times more important to us than it does GitHub's designers, because they haven't been the ones dealing with it for the past 5 years.

@lildude
lildude approved these changes Sep 1, 2020
Copy link
Member

@lildude lildude left a comment

Lets do this. 🙇

@Alhadis Alhadis merged commit a945819 into master Sep 1, 2020
6 checks passed
6 checks passed
Ruby 2.6.x
Details
Ruby 2.6.x
Details
Ruby 2.5.x
Details
Ruby 2.5.x
Details
Ruby 2.4.x
Details
Ruby 2.4.x
Details
@Alhadis Alhadis deleted the liberation branch Sep 1, 2020
sambacha added a commit to freight-trust/linguist that referenced this pull request Sep 4, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
2 participants