Replaced single quotes with double quotes to avoid linter error#5335
Replaced single quotes with double quotes to avoid linter error#5335christinaor merged 1 commit intohackforla:gh-pagesfrom
Conversation
|
Want to review this pull request? Take a look at this documentation for a step by step guide! From your project repository, check out a new branch and test the changes. |
|
Review ETA: EOD Tues |
allison-truhlar
left a comment
There was a problem hiding this comment.
Hi @ajb176! Great job with this pull request! The branches are set up correctly and there is a correctly formatted linked issue. You made only the requested change to the code and no visual changes resulted on the website, as expected. Keep up the good work!
|
I can see the change to double quotes, the branch name is descriptive, and branch itself is set up as it should be. Only thing is the formatting of the bullets in the issue is missing a space, so the dashes are appearing as themselves instead of bullets. This is what you have: vs. what it should be between the quotes to get the bullet points for next time: Otherwise, great job @ajb176 ! |
Fixes #5281
What changes did you make?
-Replaced single quotes around class in HTML tag with double quotes
Why did you make the changes (we will use this info to test)?
-To avoid a linter error
Screenshots of Proposed Changes Of The Website (if any, please do not screen shot code changes)
Minor formatting issue, no visual changes to website.