jiebi/CodeConvo huggingface dataset:
Change:
"id": "bad87fee1248bd9aedf08824", "title": "Add Different Margins to Each Side of an Element", "description": [ <del> "Give the green box a <code>margin</code> of <code>40px</code> on its top and left side, but only <code>20px</code> on its bottom and right side.", </del> "Sometimes you will want to customize an element so that it has a different <code>margin</code> on each of its sides.", <del> "CSS allows you to control the <code>margin</code> of an element on all four sides with <code>margin-top</code>, <code>margin-right</code>, <code>margin-bottom</code>, and <code>margin-left</code> properties." </del> <ins> "CSS allows you to control the <code>margin</code> of an element on all four sides with <code>margin-top</code>, <code>margin-right</code>, <code>margin-bottom</code>, and <code>margin-left</code> properties.", "Give the green box a <code>margin</code> of <code>40px</code> on its top and left side, but only <code>20px</code> on its bottom and right side." </ins> ], "tests": [ "assert($(".green-box").css("margin-top") === "40px", 'Your <code>green-box</code> class should give the top of elements <code>40px</code> of <code>margin</code>.')",

Discussion:
The Waypoint description says: It seems to me that the first paragraph should be the last, i.e., the assignment was given before the Waypoint's description.
Change:
.price { text-align: right; <del> width: 25% </del> <ins> width: 25%; </ins> } ```

Discussion:
.price { text-align: right; width: 25% } Line 156 of the file within the freeCodeCamp curriculum. Impact: While functionally harmless in modern browsers, adding the semicolon improves code consistency and maintainability. Discussion: The comments on the issue highlight: It's likely a typo since other CSS rules in the challenge have semicolons. While not strictly an error, adding the semicolon for consistency is recommended. to '' on 'Code Edit' down to 'Line 156' ; at last character The default code has not ; sign In css part It does look like a typo: ! ! Device: Laptop OS: Windows 11 Browser: edge Current Status: The issue is labeled as help wanted and type: bug. Multiple pull requests have been linked that might fix this issue: Update by sharabshrestha99 fix(curriculum): typo error on css file by brunades Additional Notes: A comment offers a reference for further details on semicolons in CSS:
it's not necessarily an error, the last rule doesn't need semicolon, but it could be for consistency yeah The list of files in which this is present is (44 files): I will open this for contribution.
Some additional context on last-rule semi-colons for those who might be interested: ! But I agree that omitting it is inconsistent with other steps and so it was likely a typo and we should fix it for consistency.
is this still open? I would like to contribute
hey We typically do not assign issues. Instead, we accept the first pull request that comprehensively solves the issue. Issues labeled 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:
"set-certs-public": "Set your certification settings to public", "set-profile-public": "Set your profile settings to public", "set-claim": "Claim and view your certification" <ins> }, "forum-help": { "browser-info": "**Your browser information:**", "user-agent": "User Agent is: <code>{{userAgent}}</code>", "challenge": "**Challenge:** {{challengeTitle}}", "challenge-link": "**Link to the challenge:**", "whats-happening": "**Tell us what's happening:**", "describe": "Describe your issue in detail here.", "camper-project": "**Your project link(s)**", "camper-code": "**Your code so far**", "warning": "WARNING", "too-long-one": "The challenge seed code and/or your solution exceeded the maximum length we can port over from the challenge.", "too-long-two": "You will need to take an additional step here so the code you wrote presents in an easy to read format.", "too-long-three": "Please copy/paste all the editor code showing in the challenge from where you just linked.", "add-code-one": "Replace these two sentences with your copied code.", "add-code-two": "Please leave the ``` line above and the ``` line below,", "add-code-three": "because they allow your code to properly format in the post." </ins> } }

Discussion:
We want to localize the Ask-To-Help template that pre-fills the post on the forum. ! The above button which takes you to the forum should render this template: ! In the localized version instead.
Should be able to tackle this today, I think.
Sure. Not a high priority, so take your time.
Change:
</Modal.Title> </Modal.Header> <Modal.Body className='help-modal-body text-center'> <del> <h3> </del> <ins> <h3 className='help-modal-heading'> </ins> <Trans i18nKey='learn.tried-rsa'> <a href={RSA}

Discussion:
I was looking at and I wonder if we should improve the legibility of that heading text as well. The default Bootstrap isn't great here. The font used with makes it harder to read in my opinion. Here is the Modal with and the default for the heading. ! And here is the original ! Any challenge with "Get Help" Open the Modal Better legibility of the heading text. 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
! Isn't it already correct?
I don't know what you mean by correct. I'm suggesting changing the and to make it easier to read. I'm talking about the with the Read-Search-Ask link in it to be clear. It is less of an issue in light mode than dark mode but I still think the legibility can be improved by the suggested changes.
I don't think it's hard to read, but it does feel a little condensed and I do think it looks better with a little more space. With the space, we may not need to make the adjustments to the link underlines.
If the text "looks better" chances are it "reads better". And yes, the line height should also help with the underline issue.
Change:
import React from 'react'; <del> import { faExternalLinkAlt } from '@fortawesome/free-solid-svg-icons'; </del> <ins> import { faWindowRestore } from '@fortawesome/free-solid-svg-icons'; </ins> import { FontAwesomeIcon } from '@fortawesome/react-fontawesome'; import { useTranslation } from 'react-i18next'; import EditorTabs from './editor-tabs';

Discussion:
A camper mentioned in his comment, that this icon below indicate to them that it's an external link not a popup. ! We can change it to clearer icon that show, that it's a popup. The original comment for context:
I found it interesting, that say you should use ellipsis (…) when there is some additional action required after the action (popup, dialog, confirmation) and it isn't clear by the action itself. This doesn't mean you should use an ellipsis whenever an action displays another window only when additional information is required to perform the action. Consequently, any command button whose implicit verb is to "show another window" doesn't take an ellipsis, such as with the commands About, Advanced, Help (or any other command linking to a Help topic), Options, Properties, or Settings. So maybe an ellipsis of some kind: in button's label or in the icon? More so, it's a part of too. If we were to stick with keeping an icon regardless, this regular feels like a better choice, with the arrow denoting there would be some kind of external interface but the enclosed square representing that it is not an external redirection, thus denoting some kind of popup or modal.
I think these guides are saying that the ellipses should be when the button opens a window that requires the user to enter more information or perform another action. This isn't the case here, so I don't think they are warranted in this instance. If we want to use a new icon then I would suggest we use one that has . I'm not suggesting that we use this exact icon. This is just an example of what I'm suggesting.
I second the use of some kind of variation of that icon. There's a small issue though, there's no such icon in the font awesome library for this so we'll have to find a different way to implement this.
Font awesome has an icon that could have the same meaning: Alternatively Material Icons has a specific preview icon if these can be used?
can you assign me this task?
issues with label, aren't open for contribution yet. You can however browse the label help wanted for open issues
allright sir
Both and both are plausible.
I'd suggest we should move forward to replace the icon with the icon. Should I move ahead and send over a PR? This is the output - !
Sure. Just make sure we don't need to add other dependencies to get that icon.
Change:
assert.equal(locations[7].name, "easter egg"); ``` <del> Your eighth `locations` value should have a `button text` property with the value an array with the strings `2`, `8`, and `Go to town square?`. </del> <ins> Your eighth `locations` value should have a `button text` array with the strings `2`, `8`, and `Go to town square?`. </ins> ```js assert.deepEqual(locations[7]["button text"], ["2", "8", "Go to town square?"]); ``` <del> Your eighth `locations` value should have a `button functions` property with the value an array with the variables `pickTwo`, `pickEight`, and `goTown`. </del> <ins> Your eighth `locations` value should have a `button functions` an array with the variables `pickTwo`, `pickEight`, and `goTown`. </ins> ```js assert.deepEqual(locations[7]["button functions"], [pickTwo, pickEight, goTown]);

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:
#supporter-progress-wrapper .progress { height: 38px; margin-bottom: 0px; <ins> background-color: var(--quaternary-background); </ins> } #supporter-progress-wrapper .progress-bar { padding-top: 8px; <ins> background-color: var(--secondary-color); </ins> }

Discussion:
When you sign in, you get sent to the "welcome" page where it shows the number of donators. The progress bar there is still green. I'm assuming we want to make it some shade of blue or something. <img width="652" alt="Screen Shot 2019-08-21 at 9 17 21 AM" src="https://user-"
Change:
Your `sum` function should return the result of calling `.reduce()` on `nums`. ```js <del> assert.match(code, /consts*sums*=(?s*numss*)?s*=>s*nums.reduce(/); </del> <ins> assert.match(code, /consts*sums*=s*(?s*numss*)?s*=>s*nums.reduce(/); </ins> ``` Your `sum` function should return the sum of all numbers in `nums`.

Discussion:
The solution requires that parenthesis do not surround the parameter for the sum function. This is a bit inconsistent since previous projects and steps have either required the parenthesis or has accepted both. Both code answers should be correct. No response Device: [Laptop] OS: [ Windows 11] Browser: [Edge] Version: [120.0.2210.91] No response
Issue is actually with optional space, that's not in regex, as this passes:
Change:
<td><a href="/docs/arabic/CONTRIBUTING.md"> عربى </a></td> <td><a href="/docs/chinese/CONTRIBUTING.md"> 中文 </a></td> <td><a href="/docs/portuguese/CONTRIBUTING.md"> Português </a></td> <del> <td><a href="/docs/russian/CONTRIBUTING.md"> русский </a></td> </del> <ins> <td><a href="/docs/russian/CONTRIBUTING.md"> Русский </a></td> </ins> <td><a href="/docs/spanish/CONTRIBUTING.md"> Español </a></td> <td><a href="/docs/greek/CONTRIBUTING.md"> Ελληνικά </a></td> </tr>

Discussion:
Describe your problem and how to reproduce it: When you try to change contibution guidelines language - you'll see how the languages table's order is desynchronized and in several languages you cannot find new Greek language. Add a Link to the page with the problem: Tell us about your browser and operating system: Browser Name: Chrome Browser Version: Version 74.0.3729.169 (Official Build) (64-bit) Operating System: Windows 10 Pro 1809 If possible, add a screenshot here (you can drag and drop, png, jpg, gif, etc. in this box): !
Change:
- text: Your code should set the <code>font-size</code> property for the <code>h5</code> tag to 21 pixels. testString: assert($('h5').css('font-size') == '21px'); - text: Your code should set the <code>font-size</code> property for the <code>h6</code> tag to 14 pixels. <del> testString: assert($('h6').css('font-size') == '14px'); </del> <ins> testString: const regex = /h6s*{s*font-sizes*:s*14pxs*(;s*}|})/i; assert.strictEqual(true, regex.test(code)); </ins> ```

Discussion:
<!-- NOTE: If you're reporting a security issue, don't create a GitHub issue. Instead, email We will look into it immediately. --This is a problem I found in the "Applied Visual Design: Set the font-size for Multiple Heading Elements".Even though the styling of headers in this challenge may be correct,the console gives an error stating that "h6" is not properly styled. I figured this may be due to the minimum-font-size issue of the browsers usually set to 15 or 16px(In Google Chrome and Firefox). Just wondered if we can put a note on the left pane where the question is present. Link Browser Name: Google Chrome Browser Version: 80.0.3987.116 Operating System: Linux
Yes, I believe this has come up before. I would rather see the use regex I think.
But this issue comes up every now and then in the forum. How is it possible for a beginner to use regex in this challenge ocnsidering that they may be only a few days into the course
I meant to use regex to test that users entered "14px". Not that users should use regex at all. Right now the test is checking the value of the as it is rendered by the browser. If we check the input from the user, it wouldn't matter what size the browser gave the element.
I am new to Open Source Contributions,can you guide me on fixing this issue?
I would recommend following the . After that you need to fix the file I linked above and make a pull request. You can also reach out to us in our
Hi How would you target the element inside the tag, as it's inside HTML?
I don't understand. You want to check the CSS of the . Right?
The first part, , isolates the styles for the class. Then you can more easily find if the correct styles are in there.
Thanks It was the variable that I was missing
Change:
const oldLog = self.console.log.bind(self.console); self.console.log = function proxyConsole(...args) { <del> logs.push(args.map(arg => JSON.stringify(arg)).join(' ')); </del> <ins> logs.push(args.map(arg => '' + JSON.stringify(arg)).join(' ')); </ins> if (logs.join('n').length > MAX_LOGS_SIZE) { flushLogs(); }

Discussion:
Describe the bug Because the log calls and that returns , just outputs an empty line. This is in contrast to which outputs . To Reproduce Steps to reproduce the behavior: to into the editor. the tests Expected behavior
Change:
Your `userData` object should have a `songs` key set to `[...allSongs]`. ```js <del> assert.match(code, /songss*:s*[...allSongs],?/); </del> <ins> assert.match(code, /songss*:s*[s*...allSongss*],?/); </ins> ``` # --seed--

Discussion:
While doing the exercise I observed an inconsistency in the code test of the spread operator when applied to the array. An error occurs when attempting to spread on the next line within the object literal while the test passes when the spread operation is conducted inline. Also in the exercise statement, it is not being mentioned to spread inline Error Received: Spreading both inline and on next line is correct. So, We should probably make test pass for both the cases We should probably make test pass for spreading both inline or on next line. <img width="774" alt="Screenshot 2024-01-07 at 9 05 09 PM" src=""- 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
I am Working on this issue
Sorry, I forgot to mention that I was working on this. Also, I didn't see that you were working on this
Change:
BackEnd.displayName = 'BackEnd'; BackEnd.propTypes = propTypes; <del> export default reduxForm( { form: 'BackEndChallenge', fields, validate: createFormValidator(fieldValidators) }, </del> <ins> export default connect( </ins> mapStateToProps, mapDispatchToActions )(BackEnd);

Discussion:
Describe the bug Submitting the solution link as "" produces an error. Submitting "" does not. Whenever we submit a solution link "" it is appended with "/api/" so resultant solution gives "Not Found/Timeout error" and the same link submitted without "/" like "" gives solution as accepted. To Reproduce Steps to reproduce the behavior: to on 'solution input box' "https://freecodecamp-" solution gets accepted. "https://freecodecamp-" error Expected behavior This should be tackled as Chrome appends "/" by default to any domain URL you copy, so people who would copy URL from search bars will get URLs appended with "/" hence producing wrong results. This file shows the testString is directly appended with "/api" without checking if the input URL contains "/" at the end or not. Screenshots ! Desktop (please complete the following information): OS: Ubuntu 18 Browser: Google Chrome Version 73.0.3683.75
I am not having this problem - my project will pass the tests with or without the trailing https://fcc-manage-npm- and https://fcc-manage-npm- both pass for me in chrome Can you share a share a screen shot of the error?
Updated
hmm, it's strange that my project will give a good response when that string is appended but not yours https://fcc-manage-npm- - package as response https://freecodecamp- - not found as response not sure what is causing this - the only difference seems to be that I'm using glitch and you're using heroku in fact mine will work like this... https://fcc-manage-npm- so maybe how the two websites process a route or something? can they do that? glitch strips out the extra or something? weird Either way - I think a solution like you suggested would probably work - check if the submitted URL ends in a and remove it if it's there. This same problem probably occurs throughout the whole backend area all over the place.
Weird, would u check with an alternative hosting if the problem persists? If the problem persists the last solution would be stripping the las from input url
Did you try this Above is the actual link processed in the test with an appended , It gives a response with the same as it is shown in the screenshot I posted. So I guess the problem is with the routing of Heroku.
Change:
"Many HTML elements have <code>children</code> which <code>inherit</code> their properties from their parent HTML elements.", "For example, every HTML element is a child of your <code>body</code> element, and your "jQuery Playground" <code>h3</code> element is a child of your <code>&#60;div class="container-fluid"&#62</code> element.", "jQuery has a function called <code>children()</code> that allows you to access the children of whichever element you've selected.", <del> "Here's an example of how you would use the <code>children()</code> function to give the children of your <code>left-well</code> element the background color of blue: <code>$("#left-well").children().css("color", "blue")</code>", </del> <ins> "Here's an example of how you would use the <code>children()</code> function to give the children of your <code>left-well</code> element the color of blue: <code>$("#left-well").children().css("color", "blue")</code>", </ins> "Give all the children of your <code>#right-well</code> element a color of green." ], "tests": [

Discussion:
On "Here's an example of how you would use the children() function to give the children of your left-well element the background color of blue : $("#left-well").children().css("color", "blue")" Shouldn't it just be color of blue instead of background color of blue?
thank you for pointing this out!!
Change:
); ``` <del> All elements from the first array should be added to the second array in their original order. </del> <ins> All elements from the first array should be added to the second array in their original order. `frankenSplice([1, 2, 3, 4], [], 0)` should return `[1, 2, 3, 4]`. </ins> ```js assert.deepEqual(frankenSplice([1, 2, 3, 4], [], 0), [1, 2, 3, 4]);

Discussion:
Continuing from This is a challenges with the function call used in the test not being visible in the test text. There may be others. The test text should include the function call used.
I'm not sure I understand here Are you referring to this test: How it doesn't say what exactly is being tested? The text should be more specific? edit: This challenge should definitely be renamed to
Sorry, it's this one: All elements from the first array should be to the second array in their original order. assert.deepEqual(frankenSplice([1, 2, 3, 4], [], 0), [1, 2, 3, 4]); it happens that users use ways that don't agree with an empty array, but fail to understand why it is failing, and the only way for them to see that it was an empty array was to use inside the function and rely on the test logs
So to add, I think the test text should be something like The original text could be changed to be this, or the two could be combined in some way
A suggestion to try to make the description a little clearer: "You are given two arrays and an index. Copy each element of the first array into the second array, in order. Write a function called <codefrankenSplice</codethat receives the two arrays and an integer as the index <coden</codeon the second array where you start inserting the elements of the first array. The <codefrankenSplice</codefunction will return another array with the elements of the first array inserted in the second array starting on the index <coden</code. The input arrays should remain the same after the function runs." Also, to add to the idea expressed on "The first array should remain the same after the function runs.", I would suggest adding the following to that test text saying "In case the second array is empty and n is 0, frankenSplice would return a copy of the first array."