* fix: general bugs
- Ensure editable region at end of code-block is followed by a blank
line.
- Remove apparent duplicate step.
- Address spelling issues.
* feat: some tests
* feat: tests 6-20
Add tests for steps 6-20.
* feat: tests 21-30
Add tests for steps 21 through 30.
* feat: tests 31-40
Add tests for steps 31 through 40.
* feat: tests 41-50
Add tests for steps 41 through 50.
* feat: tests 51-60
Add tests for steps 51 through 60.
* feat: tests 61-70
Add tests for steps 61 through 70.
* feat: tests 71-80
Add tests for steps 71 through 80.
* feat: steps 81-92
Add tests for steps 81 through 92.
* feat: align dashed names
Align the dashed names with the titles and file names.
* fix: apply gikf's suggestions
Co-authored-by: gikf <60067306+gikf@users.noreply.github.com>
* fix: manual review suggestions
Apply gikf's other suggestions.
* feat: more thorough testing
When a test asserts that a camper adds a new element relative to
an existing element, also test that the existing element is not
malformed.
* feat: optional semi colons
Make EOL semi-colons optional in the CSS. If they're missing and the
CSS doesn't render correctly, other tests will catch that. If they
are on the last property of the ruleset, they technically aren't
required so we shouldn't require them?
* chore: apply review suggestions
Apply review suggestions from gikf
Co-authored-by: gikf <60067306+gikf@users.noreply.github.com>
* chore: apply suggestions from code review
Co-authored-by: gikf <60067306+gikf@users.noreply.github.com>
* feat: review suggestions
Apply Ahmad's review suggestions.
* chore: additional suggestions
Manual application of Ahmad's review suggestions.
Co-authored-by: Ahmad Abdolsaheb <ahmad.abdolsaheb@gmail.com>
* chore: apply suggestions from code review
Co-authored-by: Ahmad Abdolsaheb <ahmad.abdolsaheb@gmail.com>
* chore: apply shaun's review suggestions
Co-authored-by: Shaun Hamilton <51722130+ShaunSHamilton@users.noreply.github.com>
* feat: add clarification
Add a bit of text clarifying the file switch in the editor.
* chore: replace URLs
Replace the tinyurls with our self-hosted CDN images.
* feat: new CSS tests
Re-write CSS tests with new parser.
* fix: grammar
Apply Shaun's grammar reviews.
Co-authored-by: Shaun Hamilton <51722130+ShaunSHamilton@users.noreply.github.com>
* fix: optional chaining
* chore: apply suggestions from code review
Co-authored-by: Shaun Hamilton <shauhami020@gmail.com>
* chore: apply reviews manually
* chore: getStyle update
Replace getStyleDeclaration with getStyle.
* fix: resolve test issue
* chore: remove jQuery :(
Co-authored-by: Shaun Hamilton <shauhami020@gmail.com>
* chore: apply suggestions from code review
Co-authored-by: Shaun Hamilton <shauhami020@gmail.com>
* fiux: Update curriculum/challenges/english/01-responsive-web-design/basic-css-cafe-menu/part-073.md
Co-authored-by: Tom <20648924+moT01@users.noreply.github.com>
Co-authored-by: gikf <60067306+gikf@users.noreply.github.com>
Co-authored-by: Ahmad Abdolsaheb <ahmad.abdolsaheb@gmail.com>
Co-authored-by: Shaun Hamilton <51722130+ShaunSHamilton@users.noreply.github.com>
Co-authored-by: Shaun Hamilton <shauhami020@gmail.com>
Co-authored-by: Tom <20648924+moT01@users.noreply.github.com>
* Added initial version of spy helper
* add missing backslashes
* update to check for line start
* change __fixtures file extensions to .ts from .tsx
* add to exports and add documentation
* add tests
* update event.preventDefault() test
* fix: recognize background-image as correct (#42738)
The page mentioned in #42738 before would not accept background-image and background, now it should.
* fix: CI build error
Co-authored-by: Ilenia <nethleen@gmail.com>
Co-authored-by: Ilenia <nethleen@gmail.com>
* Update referenced object keys for clarity
Because each movie object has both `Rated` and `imdbRating` key/value pairs, simply referencing `rating` could be confusing to campers. IMO explicitly referencing the keys by name (`Title` and `imdbRating`) removes ambiguity in the non-important part of this challenge and helps campers focus on learning about `.map()`.
* other references and tests
* Update curriculum/challenges/english/02-javascript-algorithms-and-data-structures/functional-programming/use-the-map-method-to-extract-data-from-an-array.md
* reverse previous changes
* Update curriculum/challenges/english/02-javascript-algorithms-and-data-structures/functional-programming/use-the-map-method-to-extract-data-from-an-array.md
Co-authored-by: Shaun Hamilton <shauhami020@gmail.com>
Co-authored-by: Shaun Hamilton <shauhami020@gmail.com>
* fix: replace html with markdown and MathJax
* fix: replace html with markdown and MathJax
* fix: replace example
* fix: clarify expected input and output
* fix: add another test
* fix: change image in seed code
* fix: grammar
Co-authored-by: Nicholas Carrigan (he/him) <nhcarrigan@gmail.com>
Co-authored-by: Nicholas Carrigan (he/him) <nhcarrigan@gmail.com>
* Ensure the test only checks the state name variable is present, it doesn't have to be the only word
* Emphasise in the test the wording that the instructions use
* Added note detail and event.preventDefault() test
* Replaced lookbehinds with matches
* Reverted to using \s instead of .
Now only matches if event.preventDefault() call is the only thing in the comment. This avoids matching note comments like "need to call event.preventDefault() here"
* Added optional semicolons for both comment checks
* Added trailing whitespace for line comment check
* Switched to match length checks
This should be equivalent to the two original lookbehinds
* Removed unnecessary .length check
Co-authored-by: Oliver Eyton-Williams <ojeytonwilliams@gmail.com>
Co-authored-by: Oliver Eyton-Williams <ojeytonwilliams@gmail.com>
