jiebi/CodeConvo huggingface dataset:
Change:
bsStyle='primary' className='btn-link' onClick={closeDonationModal} <ins> tabIndex='0' </ins> > {closeLabel ? 'Close.' : 'Ask me later.'} </Button>

Discussion:
Describe the bug The popup modal for becoming a sponsor cannot be closed by just using the keyboard. To Reproduce Steps to reproduce the behavior: some challenges which triggers the modal Expected behavior The modal closes when pressing the Esc key The modal closes when clicking somewhere outside of the modal I can navigate to the "Ask me later" button with the tab key to close the modal Screenshots ! Desktop (please complete the following information): OS: Ubuntu Browser: chrome and firefox Version: Latest Smartphone (please complete the following information): Unknown
This is by design to the best of my knowledge. I would defer to Quincy for a confirmation.
Thanks for reporting that the "ask me later" link should be accessible by tab but you're correct - it isn't. We will fix this. We decided not to have escape key / clicking outside of the modal to dismiss this modal (like we do with other modals) since the goal is to get people to actually slow down and consider donating to our nonprofit - not to just mash the escape key.
Change:
isOpen: true }); <del> if (files) { </del> <ins> if (files?.length) { </ins> return ( <button className='project-link-button-override'

Discussion:
I'm not sure if this was deliberately at some point, I'm guessing not - but viewing a cert solution for any cert brings up the JS code viewer with the URL in it when a URL is used to submit the project... ! Go View a solution See modal Possibly from this recent PR: - I swear I tested it
My bad. This line probably needs to change:
I just tried implementing your solution, but then realized I didn't know how to test if it worked without sign in/user data. How should that be done?
You should be able to log in - assuming the is set to true in your .env file. Then you can submit some projects, many of them can just be a URL and don't have any requirements. I believe the responsive web design, front end libraries, and data visualization don't require passing tests. So you should be able to get a cert like that and then go view it.
Turns out that login/the dev user doesn't work on my machine for some reason. Works fine on GitPod though, so it's been tested there.
Change:
In this section, you'll see how adding CSS styles to the elements of your CatPhotoApp can change it from simple text to something more. <del> <div role="alert" class="fade alert alert-info show flash-message-enter-active"> </del> <ins> <div role="alert" class="alert alert-info"> </ins> <p> Note: Some browser extensions may interfere with elements on the page. If the tests fail, try disabling your extensions for the most reliable experience. </p>

Discussion:
On , there should be a note displayed between the description and the buttons, but we are seeing an empty div instead: <img width="792" alt="Screenshot 2024-05-24 at 22 53 18" src=""The issue is due to the class to the . In Remove the class to make the note show up Remove the class as it is unused
This issue is open for contribution. Please make sure you read our as well as 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:
That worked, but there is still a little space on the right of the price. <del> You could keep trying various percentages for the widths. Instead, use the back space key on your keyboard to move the `p` element with the class `price` next to the `p` element with the class `flavor` so that they are on the same line in the editor. Make sure there is no space between them. </del> <ins> You could keep trying various percentages for the widths. Instead, use the back space key on your keyboard to move the `p` element with the class `price` next to the `p` element with the class `flavor` so that they are on the same line in the editor. Make sure there is no space between the two elements. </ins> # --hints--

Discussion:
In the directions for the challenge it states "Instead, use the back space key on your keyboard to move the p element with the class price next to the p element with the class flavor so that they are on the same line in the editor. Make sure there is no space between them". But here's the thing, to some campers it might be unclear that the 'them' in the second sentence refers to the elements, and might assume it means the class attribute or something. While this is addressed in the hint, it probably should be clearer in the challenge directions. In this challenge, it should be clear that the last sentence refers to the spacing between the two elements. No response Device: Desktop OS: Windows 10 Browser: Firefox Version: 123.0.1 I've seen at least two threads on the forums covering this challenge.
Change:
"descriptionEs": [ [ "http://i.imgur.com/tP2ccTE.gif", <del> "Un gif mostrandore como puedes pulsar en tu imagen de perfil en la esquina superior derecha conectar tu portafolio y GitHub.", "Dale un vistazo de tu portafolio de código. Da click en tu imagen en la esquina superior derecha. Para activar tu portafolio de código, necesitaras conectar tu cuenta de GitHub con Free Code Camp. Tu portafolio de código muestra tu progreso y cuantos Brownie Points tienes. Puedes obtener Brownie Points completando desafíos y ayudando a otros campistas en nuestras salas de chat. Si obtienes Brownie Points varios días consecutivos, obtendrás una racha.", </del> <ins> "Un gif mostrándote como puedes pulsar en tu imagen de perfil en la esquina superior derecha conectar tu portafolio y GitHub.", "Dale un vistazo de tu portafolio de código. Pulsa en tu imagen en la esquina superior derecha. Para activar tu portafolio de código, necesitaras conectar tu cuenta de GitHub con Free Code Camp. Tu portafolio de código muestra tu progreso y cuantos Puntos Cafés tienes. Puedes obtener Puntos Cafés completando desafíos y ayudando a otros campistas en nuestras salas de chat. Si obtienes Puntos Cafés varios días consecutivos, obtendrás una racha.", </ins> "" ] ]

Discussion:
[X] Learn how Free Code Camp Works [X] Create a GitHub Account and Join our Chat Rooms [X] Configure your Code Portfolio [X] Join a Campsite in Your City [X] Learn What to Do If You Get Stuck
Change:
(getUserInput) => $.get(getUserInput('url') + '/_api/get-tests?type=functional&n=4').then( (data) => { <del> assert.equal(data.assertions[3].method, 'browser.element'); </del> <ins> assert.equal(data.assertions[3].method, 'browser.elements'); </ins> assert.match(data.assertions[3].args[0], /('|")span#dates1/); assert.equal(data.assertions[3].args[1], 1); },

Discussion:
<!-- NOTE: If you're reporting a security issue, don't create a GitHub issue. Instead, email We will look into it immediately. --At the affected page the following code snippet is given: But, according to the documentation of the function, the second argument is supposed to be a message and not the number of elements found. To check that an element exists n number of times, assert.elements() should be used. Affected page System (please complete the following information): Device: Laptop OS: Windows 10 Browser: Firefox Version: 86.0 Additional context Add any other context about the problem here.
Hello there, Thank you, for pointing this out. I can confirm this is incorrect. I am opening this up to first timers, and expect the best change to be: 1) Change the lesson to include () 2) Change the test to look for instead of (test is located in same file as lesson) An issue with the help wanted or first timers only 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
Change:
var challengeSeed = !{JSON.stringify(challengeSeed)}; var challengeEntryPoint = !{JSON.stringify(challengeEntryPoint)}; var passedBonfireHash = !{JSON.stringify(bonfireHash)}; <ins> var bonfireName = !{JSON.stringify(name)}; var started = Math.floor(Date.now() / 1000); </ins> .col-xs-12.col-sm-12.col-md-8 #mainEditorPanel form.code

Discussion:
<!-- NOTE: If you're reporting a security issue, don't create a GitHub issue. Instead, email We will look into it immediately. --In the challenge CSS Flexbox: Use the flex-wrap Property to Wrap a Row or Column, the value wrap-reverse is defined: wrap-reverse: "wraps items from right-to-left if they are in a row, or bottom-to-top if they are in a column." This definition is not correct. Actually wrap-reverse wrap content changing de cross-axis direction, bottom to top in a row, and left to right in a column. Browser Name: Browser Version: Operating System:
I'd like to take on this issue please
Since the curriculum content is in the repo, I submitted the PR there.
Ok, thanks Simon.
Change:
# --description-- <del> In the Present Perfect Tense, y`for` and `since` are used to discuss time duration and starting points. </del> <ins> In the Present Perfect Tense, `for` and `since` are used to discuss time duration and starting points. </ins> `For` tells how long something has been happening. `Since` tells when something started. # --fillInTheBlank--

Discussion:
Describe the bug The first sentence of description contains rogue letter 'y', which needs to be removed: This is 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: Additional context Please make sure you read , we prioritize contributors following the instructions in our guides. Join us in or if you need help contributing, our moderators will guide you through this. Sometimes we may get more than one pull requests. We typically accept the most quality contribution followed by the one that is made first. Happy contributing.
Change:
<SearchBar innerRef={searchBarRef} /> </div> <div className='universal-nav-middle'> <del> <Link id='universal-nav-logo' to='/'> </del> <ins> <Link id='universal-nav-logo' to='/learn'> </ins> <NavLogo /> <span className='sr-only'>freeCodeCamp.org</span> </Link>

Discussion:
Currently when clicking the freeCodeCamp logo in the navbar while on /learn it takes me to the landing page: Instead, clicking the logo on the navbar should take me to /learn. Aside from your first time arriving at / there is never a reason to be taken back there.
I made a PR here, although I'm not sold on the function. I agree that nobody will need to go back to the home page, but it seems a little unintuitive to have it go to when a home page exists.
Change:
You should add the `flavor` class to your `p` element. ```js <del> assert(code.match(/<ps*class=('|")flavor1s*>/i)); </del> <ins> assert(code.match(/<ps*classs*=s*('|")flavor1s*>/i)); </ins> ``` You should only have one element with the `flavor` class.

Discussion:
This test is whitespace sensitive when it shouldn't be. Fails: Passes: Valid HTML should pass. No response All No response
Change:
); ``` <del> `insertionSort` should return an array that is unchanged except for order. </del> <ins> `insertionSort([1,4,2,8,345,123,43,32,5643,63,123,43,2,55,1,234,92])` should return an array that is unchanged except for order. </ins> ```js assert.sameMembers(

Discussion:
The following challenges have an example function call in the seed code. These function calls should be removed and the related tests should be clarified to reflect the call instead. This sorting challenge is correct and could be used for reference.
Hi, nhcarrigan. I'd like to try to help and solve it. I'm still learning about the project so I would like to ask a question so I can understand the issue better. There is a function call in the file.. bubleSort([1,2,3....]) (example: implement-bubble-) ! And it should be removed from the seed section and transferred to here, similarly to implement-quick-sort: ! ! Is it correct or are there other files that I should be concerned about?
That is correct, yes - this same change should be completed on the four files I've linked. I would, however, recommend holding off on working on this until we receive additional consensus that this is the best way to proceed. If you run in to any questions, feel free to review our or ask in our or the .
Sounds great!! Thanks for the fast response. I will wait until further notice.
Hi MuriloMarquesSantos, are you still interested in fixing this issue?
Change:
assert.equal(locations[6].name, 'win'); ``` <del> Your seventh `locations` value should have a `button text` property with the value an array with three `REPLAY?` strings. </del> <ins> Your seventh `locations` value should have a `button text` array with three `REPLAY?` strings. </ins> ```js assert.deepEqual(locations[6]["button text"], ['REPLAY?', 'REPLAY?', 'REPLAY?']); ``` <del> Your seventh `locations` value should have a `button functions` property with the value an array with three `restart` variables. </del> <ins> Your seventh `locations` value should have a `button functions` array with three `restart` variables. </ins> ```js assert.deepEqual(locations[6]["button functions"], [restart, restart, restart]);

Discussion:
reported sentence that looks weird, and I agree. it is used in 4 files here is the sentence I noticed in the last file, we are using deepEquel, should we mention that array's variables' respective position matter?
Can I be assigned to this? We can simply improve it as "property with the value of an array".
contribution isn't open for issues with label , you are more than welcome to discuss how to improve it
I would shorten them to just use ... e.g. "Your fifth value should have a array with three strings."
there is no one has provided a better option yet, so I am opening this for contribution, and when the JS is published, we can come back to this when campers are taking the course. You should follow Tom comment here because no one has answered this, I think it doesn't matter If you would like to fix this issue, please make sure you read , we prioritize contributors following the instructions in our guides. Join us in or if you need help contributing, our moderators will guide you through this.
Hello Sir, can you assign me this issue? hacktoberfest
how does this sounds? "Your fifth locations value should have a with the value being an array consisting of three strings." I for one think we don't need to over explain this, as the values are described in order most people will put them in that exact order.
Although, we are trying to include everyone, you are right this is me trying to overtune something simple. It's good, but we should keep Tom suggestion. In HTML/CSS projects steps, we are using the same sentence structure that Tom mentioned, so there will be comfort, and subtle hints that is an object that carry variables.
you can open a PR, after reading our guideline. more info in this comment
Your fifth locations value should have a property and an array value with three strings. kept most of the original wordings.
Opened a PR for this issue!
Change:
<del> --- title: Create a dummy file with a specific size --- ## How to create dummy files with a specific size using the "dd" command: The "dd" command can be used to create a file of a specific size. This is useful if you would like to test download speeds, or any other tests, and need a file of a specific size. ``` dd if=/dev/zero of=file_name.txt bs=1024k count=10 ``` This will create a file of 1MB called file_name.txt. bs is your byte size and count represent the number of blocks. An easy way to look at is 1024K X 10. Here is an even simpler way to create a 1MB file: ``` dd if=/dev/zero of=file_name.txt bs=1MB count=1 ``` </del>

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 directory below is meant to say 'create-file-with-specific-size'. Specific is misspelled as 'sepcific'. Add a Link to the page with the problem: Tell us about your browser and operating system: Browser Name: Browser Version: Operating System: If possible, add a screenshot here (you can drag and drop, png, jpg, gif, etc. in this box):
i am taking this issue
Change:
] ], "type": "hike", <del> "challengeType": 6 </del> <ins> "challengeType": 6, "nameEs": "Herramientas de desarrollo de Chrome: red", "descriptionEs": [ "La pestaña network (red) ayuda a responder preguntas como '¿Qué elemento o parte de la página fue el más demorado?' o '¿Qué inició una petición?'", "El panel de red registra información detallada sobre cuánto tarda en cargar cada elemento de su página o aplicación.", "Cada solicitud se registra en la bitácora, que se puede encontrar en la parte inferior del panel de red.", "Si te fijas en la línea al lado derecho, te darás cuenta de un patrón de colores. Cada color representa un tipo diferente de contenido en la solicitud. ", "Cuanto más larga sea la línea de un color determinado, tanto más tiempo tomó ese tipo de contenido en la solicitud. En últimas quieres líneas cortas, y si cualquier solicitud tiene largas líneas, sabes que puede estar ralentizando la página web.", "Si pulsas sobre una solicitud en la bitácora, podrás obtener información más detallada.", "Consulta más documentación detallada aquí: developers.google.com/web/tools/profile-performance/network-performance/resource-loading" ] </ins> }, { "id": "56b15f15632298c12f315182",

Discussion:
[x] Chrome Dev Tools: Elements [x] Chrome Dev Tools: Network [x] Chrome Dev Tools: Sources [x] Chrome Dev Tools: Timeline [x] Chrome Dev Tools: Profiles [x] Chrome Dev Tools: Resources [x] Chrome Dev Tools: Audits [x] Chrome Dev Tools: Console [x] Chrome Dev Tools: Summary
Working on it
Change:
```js <del> assert.match(code, /authorContainer.innerHTMLs*=s*(`|"|')<p.*>.*</p>1/) </del> <ins> assert.match(code, /authorContainer.innerHTMLs*=s*(`|"|')s*<p.*>.*</p>s*1/) </ins> ``` Your `p` element should have the class `error-msg`. ```js <del> assert.match(code, /(`|"|')<ps+classs*=s*("|')error-msg2>/) </del> <ins> assert.match(code, /(`|"|')s*<ps+classs*=s*("|')error-msg2>/) </ins> ``` Your `p` element should have the text `There was an error loading the authors`. ```js <del> assert.match(code, /(`|"|')<ps+classs*=s*("|')error-msg2>Theres+wass+ans+errors+loadings+thes+authors</p>1s*;?/) </del> <ins> assert.match(code, /(`|"|')<ps+classs*=s*("|')error-msg2>Theres+wass+ans+errors+loadings+thes+authors</p>s*1s*;?/) </ins> ```

Discussion:
The code provided below does not pass, even though it should. The test does not allow for an optional whitespace between the backticks/single or double quotation marks and the element - in this case a paragraph element. The hint message does not suggest that campers keep the code in one line. In fact, throughout all the curricula, I've been under the impression that creating several lines for setting improves readability, and is, therefore, good practice. So, I don't see any reasons why the tests shouldn't allow this formatting. Since I was not sure whether my code generates an essential difference on the UI/Webpage, I tested it. There was no difference. So, I thought this might be a bug in testing. I filed a PR on this issue. I am quite new to contributing and still learning. Please let me know if there is anything I've done that did not follow good practice. The test does not allow for an optional whitespace between the backticks and the element. It should. ! Device: Laptop OS: Windows 11 Browser: FireFox Version: 125.0.2 (64-bit) No response
Change:
</li> ))} </ul> <ins> <Spacer /> <h2 className='medium-heading'>Additional Learning:</h2> <ul> <li> <Link state={{ superBlock: interviewPrep }} to={`/learn`}> <h2 className='medium-heading'>{interviewPrep}</h2> </Link> </li> </ul> </ins> </Col> </Row> <Spacer />

Discussion:
On the home page, it lists the section as a certification.... ! Maybe remove that since there's no cert for it?
Yes, you are correct. I will defer to for the correct categorization for this.
We could change the font size of this. Then it would be visually distinct from the other items in this list.
We can indent the list belonging to same category. And make change in font style Like this way:- Certifications: &nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;<a href="#"Responsive Web Design</a&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;<a href="#"JavaScript Algorithms and Data Structures</a&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;<a href="#"Front End Libraries</a&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;<a href="#"Data Visualization</a&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;<a href="#"APIs and Microservices</a&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;<a href="#"Information Security and Quality Assurance</a<a href="#"Coding Interview Prep</a
I'm not sure I like either of these solutions - it seems like it might just make a user ask themselves something like: "what's with that last one?" - And not be quite clear enough that it is not a cert. What about something a little more specific... Certifications: &nbsp;&nbsp;<a href="#"Responsive Web Design</a&nbsp;&nbsp;<a href="#"JavaScript Algorithms and Data Structures</a&nbsp;&nbsp;<a href="#"Front End Libraries</a&nbsp;&nbsp;<a href="#"Data Visualization</a&nbsp;&nbsp;<a href="#"APIs and Microservices</a&nbsp;&nbsp;<a href="#"Information Security and Quality Assurance</aAdditional Challenges: &nbsp;&nbsp;<a href="#"Coding Interview Prep</a
Change:
<ins> --- title: Create a dummy file with a specific size --- ## How to create dummy files with a specific size using the "dd" command: The "dd" command can be used to create a file of a specific size. This is useful if you would like to test download speeds, or any other tests, and need a file of a specific size. ``` dd if=/dev/zero of=file_name.txt bs=1024k count=10 ``` This will create a file of 1MB called file_name.txt. bs is your byte size and count represent the number of blocks. An easy way to look at is 1024K X 10. Here is an even simpler way to create a 1MB file: ``` dd if=/dev/zero of=file_name.txt bs=1MB count=1 ``` </ins>

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 directory below is meant to say 'create-file-with-specific-size'. Specific is misspelled as 'sepcific'. Add a Link to the page with the problem: Tell us about your browser and operating system: Browser Name: Browser Version: Operating System: If possible, add a screenshot here (you can drag and drop, png, jpg, gif, etc. in this box):
i am taking this issue