Closed kevinsmithwebdev-zz closed 5 years ago
@kevinsmithwebdev, thank you again for raising this issue and volunteering to fix it.
I just have a couple of minor suggestions for the note you added. Turns out there's already a note wrapped in strong tags at the bottom of the description, so I think we can remove the word entirely. Also, what do you think about reordering the first sentence a bit? I was thinking something like the following:
"This may be different than the outdated RYB color model that many of us were taught in school, which has different primary and complementary colors. Modern color theory uses the additive RGB model (like on a computer screen) and the subtractive CMY(K) model (like in printing). Read here for more information on this complex subject."
These are just suggestions, though, so let us know if you have another idea.
Sure, I'll take a look at it tomorrow.
Regards, Kevin Smith
kevinsmithwebdev.com 510-277-2006
On Tue, Sep 4, 2018 at 12:15 AM, Kristofer Koishigawa < notifications@github.com> wrote:
@kevinsmithwebdev https://github.com/kevinsmithwebdev, thank you again for raising this issue and volunteering to fix it.
I just have a couple of minor suggestions for the note you added. Turns out there's already a note wrapped in strong tags at the bottom of the description, so I think we can remove the word entirely. Also, what do you think about reordering the first sentence a bit? I was thinking something like the following:
"This may be different than the outdated RYB color model that many of us were taught in school, which has different primary and complementary colors. Modern color theory uses the additive RGB model (like on a computer screen) and the subtractive CMY(K) model (like in printing). Read here https://en.wikipedia.org/wiki/Color_model for more information on this complex subject."
These are just suggestions, though, so let us know if you have another idea.
— You are receiving this because you were mentioned. Reply to this email directly, view it on GitHub https://github.com/freeCodeCamp/curriculum/pull/299#issuecomment-418265421, or mute the thread https://github.com/notifications/unsubscribe-auth/AYqZyCniIiUxRiRBRZqBJbrw0a9OWVZfks5uXiiYgaJpZM4WYDOg .
@scissorsneedfoodtoo That all sounds good. The only change I will make is because of the reordering of the first sentence I will change "may be" to be "is". In the old sentence "may be" makes sense because the predicate was "what we were taught..." But now the predicate is "the outdated RYB..." and "this" (the RGB model) is different, by definition. There is some conditionallity left in the sentence by the later qualification of "many of us" - leaving the door open that it may be what the camper learned and may not and making the "may be" at the beginning of the sentence redundant.
So, I will have "This is different than the outdated RYB color model that many of us were taught in school, which has different primary and complementary colors."
I'll push that up now but if it still needs adjustment, let me know.
OK, I pushed the local changes up and it looks like they are automatically being received here (without having to do an additional PR.) I'm still learning this, so please let me know if there is something else I need to do.
@kevinsmithwebdev, you're absolutely right, the sentence reads much better that way.
Everything LGTM! Thank you for all of your hard work on this issue, and congratulations on your first contribution to freeCodeCamp :sparkle: :sparkle:
The old (correct) definition of complementary colors was edited to change it from the correct RBB model to the outdated RYB description (that many of us were taught and school.) The challenge is updated to use "yellow" instead of "orange".
ISSUES CLOSED: #221
Description
The complementary color definition had been changed to the old RYB model (where blue and orange are complementary) from the modern RGB (where yellow is blue's complement). I've changed the definition back to the RGB model, changed the challenges back to use yellow as blue's complement, and added a note in the challenge text that this may be different that what we were taught and provided a link to a wikipedia article about color theory.
Pre-Submission Checklist
dev
branch.fix/
,feature/
, ortranslate/
(e.g.fix/challenge-tests
)npm test
.npm run commit
to generate a conventional commit message. Learn more here: https://conventionalcommits.org/#why-use-conventional-commitsChecklist:
Closes #221