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

New cold colour #136

Open
wants to merge 4 commits into
base: master
Choose a base branch
from
Open

New cold colour #136

wants to merge 4 commits into from

Conversation

owodunni
Copy link
Member

Tryed to make it less red ich when really cold and over ice.

@Rovanion Rovanion temporarily deployed to weather-front-pr-136 December 13, 2016 11:41 Inactive
@owodunni
Copy link
Member Author

Please check so that it is not confused with heat

@Rovanion Rovanion temporarily deployed to weather-front-pr-136 December 13, 2016 18:54 Inactive
@Rovanion Rovanion temporarily deployed to weather-front-pr-136 December 14, 2016 17:10 Inactive
@majil334
Copy link
Contributor

Jag tycker att det är bättre som det är nu på master, din förändring gjorde det mer rosa vilket är väldigt likt rött

@owodunni
Copy link
Member Author

Sant, det jag vill komma ifrån var att få en distinkt slinnad mot rött på dåliga skärmar. Man kanske skulle gå mot vitt eller någon helt annat färg?

@majil334
Copy link
Contributor

Ja, jag tycker också den lila vi har nu kan kännas röd så vore nice om vi kom på någon bra färg att byta mot. Men jag vet inte vilken som skulle passa bättre än lila.. kanske ngn lila med mindre rött i?

@MangeI89
Copy link
Member

MangeI89 commented Dec 14, 2016 via email

@owodunni
Copy link
Member Author

owodunni commented Dec 14, 2016 via email

@Rovanion
Copy link
Member

Alltså uttryck i RGB är violett, per wikipedia (127, 0, 255). Om vi tänker oss att den röda kanalen ska nå maximalt 127 och vi vet att basfärgen maximalt bidrar med 0.35*255=89, per shadern som den är skriven idag. Med andra ord kan färgen t.ex. vara (38, 0, 255).

@owodunni
Copy link
Member Author

owodunni commented Dec 19, 2016 via email

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.

4 participants