jiebi/CodeConvo huggingface dataset:
Change:
# --description-- <del> Inside your `select` element, add the following five `option` elements with these corresponding values for the option text and `value` attribute: </del> <ins> Inside your `select` element, add the following five `option` elements with these corresponding values for the `option` text and `value` attribute: </ins> **Value Attribute:**

Discussion:
There are some small issues that need to be fixed in the following files: optionfor elementsinline"Check your code" String` This looks like something that can be fixed by "first-time" code contributors to this repository. The files you should look at to work on a fix are those linked above. Please make sure you read our , we prioritize contributors following the instructions in our guides. Join us in our or our if you need help contributing; our moderators will guide you through this. Sometimes we may get more than one pull request. We typically accept the most quality contribution followed by the one that is made first. Happy contributing.
Hi I'd like to work on this issue. Thanks
Hey We typically do not assign issues. Instead, we accept the first pull request that comprehensively solves the issue. Issues labelled with or are open for contributions. Please make sure you read . We prioritize contributors following the instructions in our guide. Join us in or if you need help contributing - our community will be happy to assist you.
Change:
# --description-- <del> Call the `square_root_bisection` function with the `N` variable as the argument and print the result. </del> <ins> Call the `square_root_bisection` function with the `N` variable as the argument. This will print the result to the console. </ins> Experiment with larger values.

Discussion:
this makes look like you need to use , but the function call is enough to cause an output to the terminal. Possible solutions: NA NA NA No response
I'd update the instructions removing the part.
Hi, I'm looking to contribute to open source, could I take this ticket on as a way to onboard to this codebase? And also what is the process usually for grabbing a ticket and pushing a PR?
We typically do not assign issues. Instead, we accept the first pull request that comprehensively solves the issue. Issues labelled with or are open for contributions. Please make sure you read . We prioritize contributors following the instructions in our guide. Join us in or if you need help contributing - our community will be happy to assist you.
Change:
letter-spacing: 0.02em; } <ins> .monaco-aria-container { top: -999em; } </ins> #challenge-page-tabs .nav-tabs { margin-left: 2px; display: flex;

Discussion:
<!-- NOTE: If you're reporting a security issue, don't create a GitHub issue. Instead, email We will look into it immediately. --Extra padding seems to appear at the bottom of the page after finishing a challenge and moving on to the next. The extra padding disappears when the page is refreshed. Occurs and reproducible on some challenges. Used screen resolution is 1366x768. Browser Name: Firefox Nightly Browser Version: 62.0a1 Operating System: Kubuntu 18.04 ! !
Is this still a problem
Yep, still present. Tested on Manjaro, Firefox Nightly 67.0a1 and Firefox 65.0 in safe mode and on different profiles.
Can you checkout our which has many changes which may have resolved this issue. If it is still present, then we can label it as such. Also, just to make it clear, can you maybe highlight or circle where you are seeing this extra padding on a screenshot? Thank you.
can confirm this still happens on Seems like the completion modal causes the text in the description to shift by a pixel or two.
Steps to reproduce: Use Firefox Go to Currently you can scroll to the bottom and see no white padding Click on line 2 of the editor and hit Ctrl-Space to get suggestions and then Ctrl-Enter to run the tests Now scroll to the bottom and see that the padding is visible ! This is caused by the div which should be visually hidden, but is actually sitting at the bottom of the page. I'll submit a PR fixing this shortly.
Change:
const { t } = useTranslation(); return ( <del> <Panel data-cy='c-sharp-survey-alert' bsStyle='info'> </del> <ins> <Panel variant='info' data-cy='c-sharp-survey-alert'> </ins> <Panel.Heading>{t('survey.foundational-c-sharp.title')}</Panel.Heading> <Panel.Body className='text-center'> <p>{t('survey.misc.two-questions')}</p> <Spacer size='small' /> <Button block={true} <del> bsSize='md' bsStyle='info' </del> <ins> variant='info' </ins> data-cy='start-csharp-survey-btn' <del> className='btn-invert' </del> onClick={openSurveyModal} type='button' >

Discussion:
<img width="810" alt="image" src="" all the trophy challenges night theme to the exam page UI to be legible. No response Browser: Chrome <details<summaryLight theme is fine</summary<img width="742" alt="image" src=""</details
An issue with the label is open for contribution. The first comprehensive PR created will be reviewed and merged. We typically do not assign issues to anyone other than long-time contributors. If you would like to contribute, and have not read the contributors docs, please do so here: If you have any issues with contributing, be sure to join us on the , or on the
Hey I would like to work on this. Can you tell me the file that I will be working with?
Hey how can we complete the trophy challenges when working in the dev environment?
To test locally, I just alter the database to have the relevant challenges in the array. That is, I , but I now see we have a flag.
Taking a look at this, I noticed that the flag doesn't work. should fix that. Also, it was that the surveys and exams get seeded when you run the seed command. So to see that alert - you need to seed the trophy challenges, and then delete the collection.
The The fix is to import from instead of and give the component a prop and you should be able to remove the prop. I only played with it for a short time, but I think this will fix it all up.
Change:
You should set the `font-family` to `Impact`. ```js <del> const hasFontFamily = new __helpers.CSSHelp(document).getCSSRules().some(x => x.style['font-family'] === 'Impact'); </del> <ins> const hasFontFamily = new __helpers.CSSHelp(document).getCSSRules().some(x => x.style['font-family'].toLowerCase() === 'impact'); </ins> assert(hasFontFamily); ```

Discussion:
The instructions ask to learner to use the Impact font. One learner on the forum posted that the tests fails use of using lower case first letter for the font name. In the screenshots, one picture is the rendering of the unaltered code, the other using impact (lower case) font name. The tests should pass use of either impact or Impact. ! ! NA No response
The test right now is: which constrict to a specific casing it could be transformed into Thanks for opening this issue. This looks like something that can be fixed by "first-time" code contributors to this repository. Here are the files that you should be looking at to work on a fix: List of files: /challenges/english/14-responsive-web-design-22/learn-basic-css-by-building-a-cafe- Please make sure you read our , we prioritize contributors following the instructions in our guides. Join us in our or our if you need help contributing; our moderators will guide you through this. Sometimes we may get more than one pull request. We typically accept the most quality contribution followed by the one that is made first. Happy contributing.
Change:
testString: assert(typeof findXmasSunday === 'function'); - text: <code>findXmasSunday(2000, 2100)</code> should return an array. testString: assert(typeof findXmasSunday(2000, 2100) === 'object'); <del> - text: <code>findXmasSunday(2008, 2121)</code> should return <code>[1977, 1983, 1988, 1994, 2005, 2011, 2016]</code> </del> <ins> - text: <code>findXmasSunday(1970, 2017)</code> should return <code>[1977, 1983, 1988, 1994, 2005, 2011, 2016]</code> </ins> testString: assert.deepEqual(findXmasSunday(1970, 2017), firstSolution); - text: <code>findXmasSunday(2008, 2121)</code> should return <code>[2011, 2016, 2022, 2033, 2039, 2044, 2050, 2061, 2067, 2072, 2078, 2089, 2095, 2101, 2107, 2112, 2118]</code> testString: assert.deepEqual(findXmasSunday(2008, 2121), secondSolution);

Discussion:
<!-- NOTE: If you're reporting a security issue, don't create a GitHub issue. Instead, email We will look into it immediately. --Describe your problem and how to reproduce it: The test statement should return doesn’t make sense, since 2008 is after 1977. Add a Link to the page with the problem: Tell us about your browser and operating system: Browser Name: Firefox Browser Version: 81.0.2 Operating System: Linux Mint
there seems to be just an issue in the test text: There is just to make so that the two functions, the one executed and the one written match. I will mark for first timers this is the file:
Change:
- When prompted in the terminal, select your keyboard layout by language/area Now, [Cypress can be run](how-to-add-cypress-tests.md#_2-run-the-cypress-tests) <ins> ## Troubleshooting ### Unable to Connect to Port 8000 If Cypress fails to run with the following error: ``` CypressError: `cy.visit()` failed trying to load: http://localhost:3000/signin We attempted to make an http request to this URL but the request failed without a response. We received this error at the network level: > Error: connect ECONNREFUSED 127.0.0.1:8000 Common situations why this would fail: - you don't have internet access - you forgot to run / boot your web server - your web server isn't accessible - you have weird network configuration settings on your computer This error occurred while creating the session. Because the session setup failed, we failed the test. ``` You can resolve the issue by: - Going to the root `package.json` file and adding `--host 0.0.0.0` to the `develop:client` command: ```json "scripts": { "develop:client": "cd ./client && pnpm run develop --host 0.0.0.0" } ``` - Then, re-running `pnpm run develop` </ins>

Discussion:
I consistently got the following error when running Cypress locally: Run Run either or Cypress attempts to run, but will fail with the above errorOS: MacOS Sonoma I was able to resolve the issue by adding to the command and re-run . I've been having some strange issues lately with localhost so I think this is due to my local setup, but I'm reporting still, to check if anyone was/is experiencing the same problem. We can add a note to the Cypress guide if I'm not the only one ran into it.
Solution: Remove Cypress
So the victim count is 2 now as Sem joined the club. I'll go ahead and add a note to the doc. I've also created to kick off the Cypress removal.
Change:
); }); }); <ins> describe('Custom output for JavaScript objects', function () { beforeEach(() => { cy.visit(locations.js); cy.get(selectors.editor, { timeout: 15000 }) .first() .click() .focused() .type('{ctrl}a') .clear(); }); it('Set object', () => { cy.get(selectors.editor) .first() .click() .focused() .type( 'const set = new Set();{enter}set.add(1);{enter}set.add("set");{enter}set.add(10);{enter}console.log(set);' ); cy.get(selectors.defaultOutput).should('contain', 'Set(3) {1, set, 10}'); }); it('Map object', () => { cy.get(selectors.editor) .first() .click() .focused() .type( 'const map = new Map();{enter}map.set("first", 1);{enter}map.set("second", 2);{enter}map.set("other", "map");{enter}console.log(map);' ); cy.get(selectors.defaultOutput).should( 'contain', 'Map(3) {first => 1, second => 2, other => map})' ); }); }); </ins>

Discussion:
<!-- NOTE: If you're reporting a security issue, don't create a GitHub issue. Instead, email We will look into it immediately. --Javascript Code Editor Steps to reproduce the behavior: the following code; ! Device: Laptop OS: Windows 10 Browser: Edge Version: 96.0.1054.34
Same goes for Map. I don't believe we teach Set or Map anywhere so it's maybe not super relevant. But the output should be as close to the browser console as possible I would think.
cc
My best guess is it has to do with the on the log here: But debugging has been a pain because that file does not hot-reload in the client.
Set is used in this under the coding interview prep section. While using is not specifically required for the task, the example code shown does have with an expected output underneath which with this bug will not be displayed in the output. This would be quite misleading for someone who pastes in the example code to the editor.
I dug into this a little bit: the problem is that relies on a fallback when running in the browser. This fallback does not know how to handle sets or maps and just returns . If anyone wants to work on this, the obvious approach is to check if the input is an or and then write some custom code to format them.
I take it we are leaving this open, as tests should probably be ?
Change:
Back in your event listener, you need to update the text of the `result` element. <del> Use a ternary operator to update the text of the `result` element after the `if` statement. Check the truthiness of calling `isSpam()` on the value of the `messageInput` element. If true, set the text to `Oh no! This looks like a spam message.`. Otherwise, set the text to `This message does not seem to contain any spam.` </del> <ins> After the `if` statement, use a ternary operator to check the truthiness of calling `isSpam()` with `messageInput.value` as the argument. If true, set the `textContent` property on the `result` element to `Oh no! This looks like a spam message.`. Otherwise, set it to `This message does not seem to contain any spam.` </ins> <del> Then set the `messageInput` element's value to an empty string. </del> <ins> Then set the `messageInput` element's `value` property to an empty string. </ins> # --hints--

Discussion:
We are not assigning the function call to anything. Not 100% sure what the expected error in the code is that we are testing for here. It might be its position. Also, if you forget to use the property on the you are passing to you will get this assert message, which is confusing. I don't think this test should care about the argument to , only the function call position (if that is what we are testing for here). tests: As an aside. We have all the elements and event listeners ready to be used, can't the test use them for at least the last three tests? Give the correct assert messages. No response Device: [e.g. iPhone 6, Laptop] OS: [e.g. iOS 14, Windows 10, Ubuntu 20.04] Browser: [e.g. Chrome, Safari] Version: [e.g. 22] No response
We could use this wording for the hint text I believe so. It looks we are testing for and then the next few tests are for the ternary I feel like we still need to check for the actual user message here because if the camper does something like this then they might be missing the point of the challenge. So I think we should update the tests to better check for that. But also, I think the description could be updated to better explain why they need to make these changes. Something like this could work Maybe? It would be nice to cut down on the regex here
"the result of the function call" makes it sound like the element will be assigned the function return value. We are not assigning the result of the function call. We are assigning the expression the ternary condition evaluates to based on what returns. For test 2, if we only care about the position of the function then: Because the regex includes the argument it will cause a false positive where the position can be correct but if you do not use the correct argument you get this assert message. The assert message should be about the function position without making it sound like its return value is being assigned to the element. We check the argument in test 3 and the hint shows the function call with the expected argument.
Change:
For example: ```console <del> pnpm run cypress -- run --spec=cypress/e2e/default/landing.js </del> <ins> pnpm run cypress -- run --spec=cypress/e2e/default/landing.ts </ins> ``` - To create a development build, start the development server, and run all existing cypress end-to-end tests:

Discussion:
instead of the example command should be: docshow-to-add-cypress- project dependencies to run will see error: instead of the example command should be: No response
Change:
Next, you will need to update the playlist in your HTML document to display the songs. <del> Add `songsHTML` to `playlistSongs`, by using the `innerHTML` property. This will insert the `li` element you just created into the `ul` element in the already provided `HTML` file. </del> <ins> Assign `songsHTML` to the `innerHTML` property of the `playlistSongs` element. This will insert the `li` element you just created into the `ul` element in the already provided `HTML` file. </ins> # --hints--

Discussion:
The question is worded super horribly. It just isn't written in an easy to understand way. The wording should be something like No response Device: Desktop OS: Windows 10 Browser: Firefox Version: 121.0 First reported in this thread.
A PR has already been made here
Change:
# --description-- <del> One of the most popular shapes in the world is the heart shape, and in this challenge you'll create one using pure CSS. But first, you need to understand the `::before` and `::after` pseudo-elements. These pseudo-elements are used to add something before or after a selected element. In the following example, a `::before` pseudo-element is used to add a rectangle to an element with the class `heart`: </del> <ins> One of the most popular shapes in the world is the heart shape, and in this challenge you'll create one using pure CSS. But first, you need to understand the `::before` and `::after` pseudo-elements. ::before creates a pseudo-element that is the first child of the selected element; ::after creates a pseudo-element that is the last child of the selected element. In the following example, a `::before` pseudo-element is used to add a rectangle to an element with the class `heart`: </ins> ```css .heart::before {

Discussion:
Hey everyone! On the lesson , with regards to and , it says: I think this is not actually correct. A more appropriate definition could be: Otherwise it may lead people to believe it's getting before/after the actual element. Another definition could be (taken from the MDN):
i'll do it (i like MDN definition)
Change:
} return showWarning ? ( <del> <div className='offline-warning'> </del> <ins> <div className='offline-warning alert-info'> </ins> You appear to be offline, your progress may not be being saved. </div> ) : null;

Discussion:
To Reproduce Steps to reproduce the behavior: to /learn or /settings with the theme set to dark off internet connection and wait Actual behavior The offline warning message shows up, but the text is quite hard to read. Screenshots <img width="1680" alt="Screen Shot 2020-11-09 at 12 22 16 AM" src="https://user-"
WCAG does say the contrast is only 1.41:
The notification background is hardcoded and the font color is being handled based on theme. We have let bootstrap handle the styling for the notifications since they handle the font color and the background and look well on both themes. So doing the followings should do the trick. Add as a class here: and remove the from here:
I think the best way is to implement luma-corrected calculations. The first obvious implication of going with a luma-corrected approach is that the notification background color needs to switch to an RBG declaration for the backgrounds, calculate the luma from whatever method we choose, and use it on the foreground color declaration, which can (and will) still be HSL color. I am saying this because this would allow the font color to switch based on the contrast of the notification background contrast from to and vice versa.
can I work on this? I might need help. please guide me.
that is a cool trick. Thanks for sharing the link. Since are planning to move off of bootstrap, we should consider luma-corrected calculations in how we implement our dark/light themes during the rewrite process. Meanwhile, I think we could move forward with making the simple change that resolves this issue. sure. Make sure you read this:
Change:
Your `setAttribute` method should have ``song?.title ? `Play ${song.title}` : "Play"`` as the second argument. ```js <del> assert.match(code, /playButton.setAttribute(s*('|")aria-label1,s*song?.titles*?s*`Plays*${song.title}`s*:s*1Play1s*);?s*/) </del> <ins> assert.match(code, /playButton.setAttribute(s*('|")aria-label1,s*song?.titles*?s*`Plays*${song.title}`s*:s*('|")Play2s*);?s*/) </ins> ``` # --seed--

Discussion:
In Step 59 of the Music Player project, the following code is valid as JavaScript code, but our test doesn't accept it. Notice that and are wrapped with different quotes. or The test is here: Reported in these forum posts: If we want to allow the mixing of single and double quotes, we should fix the test. If not, we should add a hint to guide users to use either single or double quotes. ! Device: [e.g. iPhone 6, Laptop] OS: [e.g. iOS 14, Windows 10, Ubuntu 20.04] Browser: [e.g. Chrome, Safari] Version: [e.g. 22] No response
Yeah, I agree that campers should be able to pass with either types of quotes because technically they did get the question right. I have opened this up for contribution
Change:
</head> <body> <header> <del> <img id="logo" src="https://cdn.freecodecamp.org/platform/universal/fcc_primary.svg"> </del> <ins> <img id="logo" alt="freeCodeCamp" src="https://cdn.freecodecamp.org/platform/universal/fcc_primary.svg"> </ins> <h1>HTML/CSS Quiz</h1> <nav> <ul>

Discussion:
[X] The "CSS" and the have the same value. This is affecting screen reader users since this involves an input and associated label. [ ] The logo in the upper left needs alt text. I would suggest it just be "freeCodeCamp". A lot of them. N/A Resolved Add alt text to the logo. Do we need to add an extra step for this or can we just add it to ? No response Device: [e.g. iPhone 6, Laptop] OS: [e.g. iOS 14, Windows 10, Ubuntu 20.04] Browser: [e.g. Chrome, Safari] Version: [e.g. 22] No response
Seeing as the logo is non-functional, do we want the alt to be null? ? If it does require an , then this should get its own step, and a clear description as to why it needs one should be given. I suggest changing the of the (and associated ) to .
Per , the logo does require alt text. Functionality isn't the deciding factor here, it's if the image conveys information (i.e. is not purely decorative), which this image clearly does. Do we want to split this off into its own issue if we are going to create a new step?
I would say yes, but it is not fun to deal with the merge conflict, when two PRs are opened at the same time.
The logo still needs an alt text. The logo is introduced in step 7: That step should be updated to instruct campers to give the an alt text of . The seed code in all subsequent steps should be updated to include that new attribute.
hey may i work on this issue? if so please assign it to me
Hey We typically do not assign issues. Instead, we accept the first pull request that comprehensively solves the issue. Issues labelled with or are open for contributions. Please make sure you read . We prioritize contributors following the instructions in our guide. Join us in or if you need help contributing - our community will be happy to assist you.
Change:
```js assert( $('source').attr('src') === <del> 'https://s3.amazonaws.com/freecodecamp/screen-reader.mp3' </del> <ins> 'https://cdn.freecodecamp.org/curriculum/applied-accessibility/screen-reader.mp3' </ins> ); ```

Discussion:
All links in the english curriculum need to be replaced. This is the last of the S3 links Here are the new CDN links that should be used instead multiple see explanation above see explanation above No response Device: [e.g. iPhone 6, Laptop] OS: [e.g. iOS 14, Windows 10, Ubuntu 20.04] Browser: [e.g. Chrome, Safari] Version: [e.g. 22] No response