Skip to content
This repository was archived by the owner on Sep 25, 2019. It is now read-only.

fix(challenges): fix description in css variable fallback challenge#264

Merged
scissorsneedfoodtoo merged 1 commit intofreeCodeCamp:devfrom
joshalling:fix/css-variable-fallback-description
Aug 26, 2018
Merged

fix(challenges): fix description in css variable fallback challenge#264
scissorsneedfoodtoo merged 1 commit intofreeCodeCamp:devfrom
joshalling:fix/css-variable-fallback-description

Conversation

@joshalling
Copy link
Copy Markdown
Member

Description

I fixed the description in this challenge to point out that the variable fallback is not used to work with browsers that don't support variables.

I also updated the instruction to call out exactly what is going on rather than explaining it as a note at the bottom.

Pre-Submission Checklist

  • Your pull request targets the dev branch.
  • Branch starts with either fix/, feature/, or translate/ (e.g. fix/challenge-tests)
  • All new and existing tests pass the command npm test.
  • Use npm run commit to generate a conventional commit message.
    Learn more here: https://conventionalcommits.org/#why-use-conventional-commits
  • The changes were done locally on your machine and NOT GitHub web interface.
    If they were done on the web interface you have ensured that you are creating conventional commit messages.

Checklist:

  • Tested changes locally.
  • Addressed currently open issue (replace XXXXX with an issue no in next line)

PARTIALLY RESOLVES: freeCodeCamp/freeCodeCamp#17546

@scissorsneedfoodtoo scissorsneedfoodtoo merged commit bc33a03 into freeCodeCamp:dev Aug 26, 2018
@scissorsneedfoodtoo
Copy link
Copy Markdown
Contributor

@joshalling, thank you for improving this challenge! It's much more clear what a fallback is and what the user is meant to do in this challenge.

@joshalling joshalling deleted the fix/css-variable-fallback-description branch September 17, 2018 18:18
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

CSS variable doesn't working on IE.

2 participants