Password form button does not respond to click. Additionally the jumbotron div that is meant to have a white background for the form is too short (only approx 96px and form is 317px).
Lui8906 can you confirm the issue is resolved so we can close it?
Still when I am at the reset form after recieving the email and following the link, I put in a matching password and the button does not do anything or provide any feedback, are there password requirements? etc..
Ok, thanks. To confirm, this is on password reset, after requesting to reset password? I'll look into it, thanks.
Yep On Tue Jan 13 2015 at 8:57:53 AM Nathan wrote:
We've fixed this with our recent pull request.


On the bonfire page the title should have Learn to code with bonfire | Free Code Camp. Learn to code is our SEO keyword, so we want to implement that when we can.
I just a title and did a pr instead of an issue :)


Code expects, in one case, for you to only be pulling the key-value pair and treating it as a new individual object, and in the second case, for you to pull the full object that possesses the key-value pair. And output / tests: Fails assert.deepEqual(where([{ 'a': 1 }, { 'a': 1 }, { 'a': 1, 'b': 2 }], { 'a': 1 }), [{ 'a': 1 }, { 'a': 1 }, { 'a': 1 }], 'should return with multiples');AssertionError:should return with multiples: expected [ { a: 1 }, { a: 1 }, { a: 1, b: 2 } ] to deeply equal [ { a: 1 }, { a: 1 }, { a: 1 } ] But Succeeds assert.deepEqual(where([{ first: 'Romeo', last: 'Montague' }, { first: 'Mercutio', last: null }, { first: 'Tybalt', last: 'Capulet' }], { last: 'Capulet' }), [{ first: 'Tybalt', last: 'Capulet' }], 'should return an array of objects');
Same problem. Going to fix it now.
Can you please explain for(var key in source)
that may be a better question to ask in gitter. See: and


source: Currently Reads: "Remove all values (last argument(s)) from an array (first argument) and return as a new array." Issue: From my understanding, the user is all the values from an array. However, the information in the parentheses (e.g "last argument(s) and "first argument") seem confusing. Can you please ellaborate further?
Here's another way of saying it: You will be provided with an array (the first argument in the function) and one or more values (the remaining arguments in the function) that must be removed from that array. If you look at the current tests, the arguments are similar: 1 array and 2 values. The instructions are asking you to remove as many values as are provided, but you will pass the tests if you only account for two (this will most likely be updated in the future). Does that help?
In the more information section we should mention about the object of JavaScript, and also perhaps link to the MDN page of it.
See updated copy: and pull request


Seeing this in production here: solution: Use and/or check for user existence and end early. cc:


Typo on the first test "your text field have the property of being required". Also, the second test passes even if the placeholder text field is empty.
Thanks for the report, the copy has been updated.


Challenge has an issue. Please describe how to reproduce it, and include links to screen shots if possible. What I see: You can apply a class to an HTML element like this: <h2 class="blue-text"CatPhotoApp<h2What I expected to see: You can apply a class to an HTML element like this: <h2 class="blue-text"CatPhotoApp</h2
Markdown's getting in the way, see screenshot. !


Challenge has an issue. Please describe how to reproduce it, and include links to screen shots if possible. Step : "Remember to use RSAP if you get stuck. Try using jQuery's $.getJSON() to consume APIs." incorrectly to this url: It should link to: Unless that misdirection is meant to be tricky for the student ;)
This is a duplicate of
Kind of. This one is referring to a different page (challenge/zipline), but I did notice that all of the Ziplines have the same bug.
This was fixed ages ago, right? Please close.
This isn't actually fixed. The erroneous link still exists in these ziplines: Maybe it's not an issue because we'll be switching over to the content at soon? But I thought you should know that the issue hasn't actually been fixed yet. Actually, the only reason that it isn't an issue on the beta site is that none of the ziplines seem to link directly to the jquery json page.
This was fixed in prior to the branch being available through Now they seem to have changed how the ziplines look, so it wouldn't be an issue regardless. Since all changes seem to happen on the branch before ending up in production, issues that are fixed in should probably be considered fixed, even though it might be better to merge small fixes like this into production at a quicker rate than what is the current reality.
Thanks for the explanation! That makes sense. I'm not privy to staging, so I didn't realize it had been fixed there.


In this lesson: In the second requirement the is included in the code tag markup which indicates it should be part of the href. It's minor but wrong and confusing for people new to html.


One of the instructions has the following: start this tutorial by running npm install how-to-npm it says to run the npm tutorial by typing 'npm install how-to-npm but you are actually supposed to just type 'how-to-npm'


Challenge has an issue. In JavaScript whole numbers (called integers) can be easily used to preform mathematical functions <-- Let's try a few of the most commonly used ones now <-- We use - for subtraction <---
merged to


! Same with the other waypoints that ask for a hex code that includes alpha chars.
Green worked fine.. blue has case issue.


Challenge has an issue. Please describe how to reproduce it, and include links to screenshots if possible. "Another type of JavaScript loop is called a "while loop" because it runs "while" something is true, and stops once that something is no longer true." Should probably read as: "Another type of JavaScript loop is called a "while loop", because it runs "while" something is true and stops once that something is no longer true." Commas separate complete sentences, or act as a pause.


In waypoint description there is a word with missing letter "r" and wrong letter - "a" instead "e" . I don't know English well, therefore this typo a slightly confused me:) !


Challenge has an issue. Please describe how to reproduce it, and include links to screenshots if possible. There seems to be a conflict between the last paragraph above the "Go to Next Challenge" button, and the first test-item below the "Reset/Help/Bug" buttons (see screenshot, below). It is not clear if the <div class="well"'s should go inside the <div class="col-xs-6"'s, or vice versa. If I were to follow ALL the directions to the letter, I'd have two <div class="col-xs-6"'s nested inside each of two <div class="well"'s, each of which is inside an outer <div class="col-xs-6". I think this is just a text-editing problem, because my code passes when I did what I expected the waypoint to be looking for: an empty <div class="well"inside each of the previously-placed <div class="col-xs-6"'s; this is included in the screenshot, as well. Also, I am fairly the text is missing a "<", where I've sloppily drawn it in on my screenshot. !
I ran into this also. The challenge validates when you follow the instructions in the explanatory text. It does not validate when you only follow the instructions in the checkpoints.


Challenge has an issue. This is backwards and should read: "Add two div elements inside your "col-xs-6" elements both with the class "well" - or something similar. !


Challenge has an issue. Please describe how to reproduce it, and include links to screenshots if possible. step 9 says Install how-to-npm with this command: npm install -g git-it. it looks like we are installing something called git-it instead of how-to-npm. As before, if I am incorrect please disregard this issue and just close. Just trying to do what I can for you guys. Love this site!
Hey thanks for posting an issue! It looks like you are right! I believe the step should say
That's what it looked like to me to. It seems that the command was copied from the next page when you do actually have to install how-to-npm. Btw, now that I have done a lot of the free code camp challenges, how would I go about starting to contribute to the project? I would love to help out with the code or issues or anything really.
Awesome! Just jump right in :smile:! Maybe start with putting in a pull request for this issue? Feel free to ping me in gitter if you want to chat about this further (so as to not pollute this issue with conversation)
Dear camper, your issue is related to: Please see .
I checked when I was creating the pull request to fix this issue and it seems that the how-to-npm part of it is already fixed.


Challenge has an issue. Please describe how to reproduce it, and include links to screenshots if possible. s - captures much more than spaces as the lesson implies, a better description that talks about what characters white space, as the title hints at, includes should be used.
Thanks for pointing this out. I agree, feel free to for this.


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!!


! Challenge has an issue. User Agent is: <codeMozilla/5.0 (X11; Linux x86_64) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/45.0.2454.101 Safari/537.36</code. Please describe how to reproduce this issue, and include links to screenshots if possible. My code:
This challenge is asking to you wrap the existing tag that is right below the , not create a new one at the bottom. The test expects to find your up there with the original , so adding it down at the bottom causes the test to fail. However, this is the third time in a week or so that I've seen this same issue reported, so I propose adding some clarifying language to the waypoint that lets the user know they should not be moving existing tag, only wrapping it.
It worked. Indeed the language should be more clarifying.
Great, just reading your your response. Unsure if you have clarifying language yet but I have run into the same of two days ago, 10/21...
The change has been pulled into the staging branch but hasn't been pushed into the production code yet. It shouldn't be too long before it is.


Challenge has an issue. User Agent is: <codeMozilla/5.0 (Windows NT 10.0; WOW64; rv:41.0) Gecko/ Firefox/41.0</code. Please describe how to reproduce this issue, and include links to screenshots if possible. My code: Issue: Properties box says: "Let's delete the "tails" property from myDog." Test success statement: "Delete the property "bark" from myDog." Test succeeds on deleting bark property, not tails.
I noticed that too !


Should be "become" I would fix it but I'm not sure where this file is located. !
closed. Fixed with


[x] Trigger Click Events with jQuery [x] Change Text with Click Events [x] Get JSON with the jQuery getJSON Method [x] Convert JSON Data to HTML [x] Render Images from Data Sources [x] Prefilter JSON [x] Get Geo-location Data
Trabajando en esta


Challenge [Bonfire: Validate US Telephone Numbers] Apparently the test case "telephoneCheck("555-555-5555") should return true." runs twice.


Challenge has an issue. User Agent is: <codeMozilla/5.0 (Macintosh; Intel Mac OS X ) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/47.0.2526.106 Safari/537.36</code. Please describe how to reproduce this issue, and include links to screenshots if possible. My code: This is a very minor issue, almost trivial, but this Waypoint initializes the editor with an unnecessary space right under . The space can be found if you look back at the code It should be instead of .
Easy enough. I'll grab this one.
So maybe I'm doing something wrong but I just fetched the latest from "staging" and there is no space between the quotation marks in the latest version. This appears to have already been corrected.
Hmmm, I'm not sure why you're not seeing it on your end. I still see it on github and in the most recent staging I just pulled down.
My mistake, I hadn't pushed to my forked repo and I guess that was keeping the latest fetched files from showing. Should be fixed with


and have text in code tags within a blockquote already. The code tagged strings need to be removed. You can see them because they are highlighted again within the code block. Access MultiDimensional Arrays with Indexes: ! Understand String Immutability: !
Agreed, these just just be in the block quote.


[x] Learn D3 Challenges
what should we do with these ones? I guess they are not finished, so there's no point in translating that one line, or am I wrong?


[ ] JavaScript Lingo: MDN and Documentation [ ] JavaScript Lingo: Value Types [ ] JavaScript Lingo: Variables & camelCase [ ] JavaScript Lingo: Arrays & Objects [ ] JavaScript Lingo: Finding and Indexing Data in Arays [ ] JavaScript Lingo: Manipulating Data [ ] JavaScript Lingo: Math [ ] JavaScript Lingo: Loops [ ] JavaScript Lingo: Regular Expressions
Working on this one


Challenge has an issue. User Agent is: <codeMozilla/5.0 (Windows NT 6.3; WOW64) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/43.0.2357.124 Safari/537.36</code. Please describe how to reproduce this issue, and include links to screenshots if possible. My code:
If you scroll down on the left section of the page below the Run Tests button you can see the results of the tests for your code. In this instance, the test fails because you have a capitalization error. For help with writing code, please consult the . Issues are for reporting bugs with the challenges or website. Happy coding!
thanks for help :) i have found the error it was on the coding i was placing another anchor tag while i supposed to use the existing one thank you :+1:
thanks for the help diagnosing the issue! Closing this issue as it looks like the issue has been resolved. Happy coding!


My challenge is makes sure all p elements have closing tag
Thanks for reporting this. But your code is incorrect. GitHub Issues are for reporting bugs on the website only. Please use the for getting challenge related help. Happy Coding.


Challenge has an issue. User Agent is: <codeMozilla/5.0 (Windows NT 10.0; Win64; x64) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/58.0.3029.81 Safari/537.36</code. Please describe how to reproduce this issue, and include links to screenshots if possible. I feel like while the above code works it simply fails to communicate the purpose of the exercise by delivering a visible change in the resulting page at the right side of the screen. For comparison's sake, this is how it looks like when the code is executed in JSFiddle Given that the page display is narrow, my proposal is to change the required positioning entirely, so that the student will be able to see more clearly the results of the absolute positioning.
Hi - thanks for pointing this out. Yeah, the narrow output screen unfortunately doesn't show the change at all. We could change the requirements to use to make it more obvious. You titled the issue "Replace the example..." though, so if you had ideas to re-work this even more, let us know your thoughts!


According to Google Analytics, the is way too hard, and is taking campers an average of 6 minutes and 35 seconds to finish (if they finish - a lot of them seem to give up). According to this table, campers spend more time on it than they do on the algorithm challenges. ! We need to either reword this or simplify the challenge so it can be completed in closer to 3 minutes. Otherwise, we should move it back to the algorithm section.
This challenge does have a large amount of constant strings which could cause users to take a longer time than a typical challenge. Maybe we could provide an array of the responses in the seed code. Make the seed code: This could help in two ways. The first is the users can save time by not having to type out each response. The second is this approach could minimize errors from typos in the response phrases (i.e. Hole-in-one! vs. Hole in one!). We could remove the number of different responses. Maybe remove Eagle and Double Bogey.
yes - that is a great suggestion. I think leaving all the names is fine but we should pre-populate the array with the names. Since this was your suggestion, would you be interested in tackling this and submitting a PR?
Yes, I would like to tackle this and submit a PR.
The Golf Code challenge did take me longer that the other challenges. I did not initially realize that golfScore() had BOTH strokes and par as parameters. With the other IF ELSE challenges we were only dealing with one argument to pass into the IF ELSE condition. I found myself in that "flow" and completely overlooked that two parameters were available to work with in the IF ELSE condition. I also found myself looking at the Strokes/Par table thinking, "How will IF( <= par-2) work?" After a little head scratching, I realized the values under the Strokes column were what you were supposed to compare Strokes to (i.e. IF (strokes <= par-2)). Hopefully, this feedback is helpful. I think the confusion is coming from the wording of the challenge and suddenly going from using one passed in parameter to two in your IF ELSE condition.
Awesome. has a PR open already. Could you take a look at that first and let me know what you think of his proposed changes?
I agree with MiloATH's proposed changes. Would definitely reduce the potential for typos when working the problem. As a side note, feel free to let me know if I am breaking any GitHub etiquette at any point. Zero experience providing input on collaborative projects. Looking for to learning and have really enjoyed FreeCodeCamp thus far. You guys and gals are doing awesome work. :)
Sure - we will let you know in a polite way if you break GitHub etiquette. Thanks for being receptive to feedback :)


<!-- NOTE: If you're reporting a security issue, don't create a GitHub issue. Instead, email We will look into it immediately. --Tests are not passing with correct code. Curiously, the "The fifth div should have a height of 25 pixels" test is passing, but none of the others. DOM elements are rendering correctly with appropriate inline style. Browser Name: Chrome Browser Version: 68 Operating System: Chrome OS !
it passed for me on firefox and chrome with the code in the picture - were you able to get it to pass?
Oddly enough, no. I find it very strange that I can get the fifth test to pass, but none of the others, no matter what I do.
yea, im not sure - did you try resetting the code and doing it again? is there any errors in the console? Try another browser maybe
Yes to reset, no to errors. If it's a browser problem, that's an issue for people using Chromebooks like myself.
I got the same scenario as . Only the fifth test is passing. Browser Name: Chrome Browser Version: Version 71.0.3578.98 (Official Build) (64-bit) Operating System: MacOS Mojave Version 10.14.2 I tried completing the challenge in Firefox and it passed. Similar users are having problems:
does not work on Chromium - Version 75.0.3770.100 (Official Build) Arch Linux (64-bit) works on Firefox 67.0.3
my browser: Versione 77.0.3865.120 (Build ufficiale) (a 32 bit) on Windows 10 Home v1803 no test passes with callback
Pretty sure the problem is similiar to the problem on
Yep looks like yet another zoom bug with Chrome. We can check the inline styles set instead for the test. I will make a PR.


from its english version at from its english version at from its english version at from its english version at from its english version at from its english version at  When you make a PR for this particular issue, do not tag it with instead just reference it with
Hey ! How could I claim so we don't double work? Thanks
Hi ! Actually we just merge in the first PR for open issues. And your comment on this thread is enough. So, its yours. Happy translating.
I just open a PR for
thanks a lot that was fast! Would you be interested in translating these in future as well? If yes, please keep a watch on this thread. We will be updating on the progress often Happy contributing.
Done translating ! My PR is available at ! Thanks for all you do, freeCodeCamp
I have sent a translation proposal for /docs/how-to-work-on-guide- PR Hope it will help!
There is still work pending on this. When you make a PR for this particular issue, do not tag it with instead just reference it with
Hello, I'm starting to help on the translation of the document /how-to-set-up-freecodecamp-
Working on /docs/spanish/how-to-work-on-guide-
Hello, I've improved the file
Hello, Improving docs/spanish/how-to-catch-outgoing-emails-
Hi, I'm improving docs/spanish/how-to-work-on-coding-
Hi, can I help with /docs/spanish/how-to-setup-freecodecamp-
Sorry, I opened a bad PR regarding but already closed it. I found there was still an English part to be translated
Hi! Is there any remaining documentation where I can help translate?
you can improve or fix redaction and spelling errors.
Hi there, It would be good if at the top are checked so we can see what is left to be translated. Thanks
Where I can start helping?
sorry to put it in this issue, but is there any way somebody could help with the pull requests? I mean, there are 5K+ opened already, people are sending them in troves, and I suspect a lot of them are a total crap (Like ) . I guess you opened a Pandora Box with your articule about HacktoberFest.
Hey all, not a translation, but fixes for the language bar of HTML links on the README file for Spanish, and markdown links to the How To guides that everyone has been translating. . or anyone else looking to do translations in Spanish, you could make a copy of the 2 style guides to translate into Spanish from the English version in the doc folder. It looks like those weren't in to the Spanish folder originally or included on this issue. Just leave the title the same, but put it in the Spanish folder
Hola! to all. I believe we should have guidelines for Spanish translations, which include what treatment (formal or informal) to use when referring to actions or the user. For example, Agregue (formal) or Agrega (informal). - I don't know if this kind of things are found in other languages. In the last articles I've read I've seen a mix between formal and informal, and I think we should standardize (I have done it with some articles). I propose to always use informal. Another thing that could be specified in the guidelines is: Challenges translations. Should they be translated? Referrals to types and attributes (which need to be in English for proper coding). For example, type "date" not type "fecha". If there is a guideline already, please let me know, if not, I would be happy to start one (just let me know where to put it). Hey I would like to know your thoughts about this. Thanks to you all and happy coding!
I agree that we should standardize around informal. I agree with you that we should look out for words like "fecha" that have been accidentally translated during our automated translation process.
Hi all I'm not sure if this is the right issue to discuss this, or if I should open a new one. I'm happy to open a new one if that makes things easier to track. I'll keep it here for the moment. What do people think about translating "array" as "arreglo"? I think this is a big mistake that has been repeated since somebody made that bad translation many years ago. As any Spanish speaker know, "arreglo" means "repair" (), and it makes any programming related text even harder to understand. This is especially true for programming texts because it's full of abstract nouns and things that are not easy to represent in a tangible way. I learned to code in English (mainly FCC and Coursera), and after a while, I stumbled upon a programming text written in Spanish. I couldn't understand what it was talking about until I found out that "arreglo" was the (really bad) translation of "array". Then I could read the text as if I was reading an encoded message that had to be deciphered. I propose that the Spanish version of FreeCodeCamp doesn't carry on with that translation, which is just based on the similarity of the spelling but has nothing to do with the meaning of the words. I think the right term in Spanish is "ristra". According to the : Which basically means: "A group of things placed one after another" They also share all the phonems in "array", with the adittion of /r/ (rolling r) and /t/ in "ristra". I'm open to other candidates, but please, let's take "arreglo" out of the Spanish version. The word "array" has been used in Spanish texts before and even though it's not a Spanish word, I think it's still better than a word that exists with a different meaning. And to be honest, anyone learning how to code will have to become familiar with certain English terms like "list", "function", "import", "scope", etc. [1]: [2]:
Hello guy, I vote for yes, let’s do the change and I think will create a huge impact around of the world! Please let me know if you have more question questions or concerns. Andres Haro Cell: (619) 534-3190
Thanks - I agree that these machine translations need improvement. I encourage you to dig in and start fixing them. Be sure to join if you need help getting set up
There is still work pending on this. When you make a PR for this particular issue, do not tag it with instead just reference it with
Thanks for working on the translations for the "Contributing to freeCodeCamp". Here on we are making contributing to translations for contribution docs ad hoc. You are free to make pull-requests for improving these translations. We are closing the issue as these do not need tracking per se. You can raise a fresh issue if you find something incorrect in the existing docs.


from its english version at from its english version at from its english version at from its english version at from its english version at from its english version at
Please, review
[x] [x] /docs/russian/how-to-work-on-coding- [] ()
Fixes the html links in the top language bar for all language README docs. Not a translation, but just link fixes. Once the Russian README files get merged, we can add the markdown style for links to all the articles in Russian too!
started to review PRs. ping me if more help needed reviewed and requested changed at LGTM from native russian speaker view LGTM Resolve conflict is needed
Hi, guys! Please review my PR
Thanks for working on the translations for the "Contributing to freeCodeCamp". Here on we are making contributing to translations for contribution docs ad hoc. You are free to make pull-requests for improving these translations. We are closing the issue as these do not need tracking per se. You can raise a fresh issue if you find something incorrect in the existing docs.


<!-- 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: Link to the guide on the challenge Data Visualization: Data Visualization with D3: Use the and Functions to Find Minimum and Maximum Values in a Dataset is pointing to the wrong place Incidentally, the challenge page uses the d3-max and d3-min in the url while the guide page uses and Add a Link to the page with the problem: Get a Hint button brings the camper to which is 404 - Page not found Correct link is Tell us about your browser and operating system: Browser Name: Mozilla Firefox for Ubuntu canonical - 1.0 Browser Version: 62.0.3 Operating System: Ubuntu 18.04 If possible, add a screenshot here (you can drag and drop, png, jpg, gif, etc. in this box):
/ If you can guide me to where this file might be located. I can check....
Solution <body<scriptconst positionData = [[1, 7, -4],[6, 3, 8],[2, 9, 3]] // Add your code below this line const maxX = (positionData, (d) =d[2]); const output = maxX ; // Change this line // Add your code above this line ("body") .append("h2") .text(output) </script</body


<!-- 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 formatting of the header-tag page in the guide is wrong. I checked the codebase and there are missing back-ticks for some of the html tags in the markdown page. Add a Link to the page with the problem: Tell us about your browser and operating system: Browser Name: Chrome Browser Version: Version 70.0.3538.102 (Official Build) (64-bit) Operating System: MacOS 10.14.1 If possible, add a screenshot here (you can drag and drop, png, jpg, gif, etc. in this box): !


Expected behavior Please see attached screenshot for more details. Basically: [x] "Contact email" should be "Email Us" [x] We should get rid of the outbound link icons [ ] The footer should always go below the tallest element. Currently on the Guide, it seems to go under the least tall element. Screenshots <img width="1246" alt="echoandprint_freecodecamp_guide" src="https://user-"
Side note (perhaps a separate issue): the footer on the page is difficult to get to due to the loading of content on a scroll down. So you have to keep scrolling and keep loading content until there's no more to load to reach the footer. I suppose there would be a similar issue on the forum or anywhere else that uses the loading on a scroll.
I'll give it a try.
I have fixed the first two issues but couldn't quite reproduce the third one either in Chrome or Firefox(In Windows 10). cc - !
There is a UX trade off for doing that for implementing the third one. In order for the footer to go under the tallest element, both elements should have the height of 100% (no-scroll). In that case, when user clicks on a side nav that is taller than the content, the page should either go to the top to show the top of the content (loses the location of the scroll), or keep its scroll which means the user will scroll up and down just to change the tabs. here is a demo: ! with no footer, our layout would have behaved similar to the default Gatsby sidebar example. With the footer, the optimal solution would be to add some javascript and have a floating sidebar that stops at the footer. We could also have a fixed sidebar that the footer would overlap at the end of the page. imho, the current setup does the job, but please let me know which solution you would like me to explore.
Hi I agree with I think the key here is to use the site as an end user, and then decide on the tradeoff. I agree that the current sidebar is far better UX for the user, who are more likely browsing through an article and have it in view rather than have them scroll a long way to switch to the next topic of their interest. That will definitely increase a session time and reduce bounce rate.
That is fine. People will scroll. I think you all are dramatically overestimating the inconvenience of scrolling. On mobile, this will all be a single column layout anyway. This is what people thought was the optimal solution back in the 1990s. In practice, it's confusing to users. Users would much rather scroll up and down a page then have to move their mouse around to different "frames" and scroll up and down within those in a more confined space. Just look at this page here on GitHub. The footer is always at the bottom of the tallest column, which in this case is the middle one. If I want to see who this issue is assigned to, yes - I have to scroll back up to the top of the page. That's no big deal - it takes less than a second. We should use a similar approach with footers everywhere on freeCodeCamp. Separate horizontal scrolling portions are bad enough, and I'd like to move away from them - let alone separate vertical portions. Here's an example of how having separate scrolling areas is hurting usability on learn: <img width="1415" alt="learnbasichtmlandhtml5fillintheblankwithplaceholdertextfreecodecamporg" src="https://user-"A new user might not even realize that there are buttons and tests underneath the "run test" button. The footer should always be at the bottom of whichever column is tallest, and we should not have independently scrolling columns. At some point in the past year or two, the independently scrolling columns were introduced. could you start working on a version that puts the footer at the bottom of the tallest column?
I strongly recommend testing the app as user would to derive the best UX for our app's use case. For the Guide, I think the tallest element principle does not work in practice, you can test that locally yourself, it's just weird to scroll through whitespace vs content. People will scroll through content, not whitespace? For Learn I had given my feedback in a comment here: But, we quickly realized that the tallest element won't work either. Here is what we should do though is to use the buttons as a fixed component on learn. This gives us the ability to scroll the "content" as much as we want, while keeping the most used component buttons in our case in view. ! Also, I would like to bring light to the fact that several feedback (fairly frustrated) threads that we have had here on GitHub and forum is the ability to use the challenge map about scrolling. It works when you are casually browsing, not when you are looking for something specific. But that's a different problem unrelated to this thread. Anyways, let's start working on a version that does put the footer at the bottom of the tallest element and do a A/B test.
I will make a MVP and have you test it ASAP. Here is how Khan academy does it when the scroll meets the footer. Notice the shadow indicating that side nav is continuing and scrollable <img width="504" alt="screen shot 2019-02-12 at 9 29 49 pm" src="https://user-".
Please let me know if you wanna move forward with this design, and if you want the user to go to the top of the page after clicking the side nav or not so I could adjust it accordingly.
Please take a look at this document: Also consider this: very few people are actively looking for the footer. And if they are, they're just going to swipe down really fast and get there immediately, regardless of how tall the page is. The main purpose of the footer is SEO and for the 0.001% of people who are actively looking for it. (how many times a day do you find yourself trying to scroll down to content in a footer?) The Guide's sidebar is only 109 entries tall. It's not going to be that much space. If they have one of the bigger categories expanded, yes, it could be more. But I don't think the footer is important enough to warp the experience of the guide.
A good example of how to handle a documentation page with expandable side menu items, and a footer, without extra scrolling is the MDN: When you expand the menu items on the left, the footer just gets pushed further down the page.
with these changes, this issue will back Or am I missing something?
so I think wants the page scroll to go to zero only when a node with no children is clicked on the side nav. I cannot think of a way that I could implemented it without recording and resetting the scroll with JS, so if anything comes to your mind, let me know.
Nodes with children also have articles.
very good point, so we would be reverting to the bug, then. if the height of both elements are set to 100% our footer would be pushed down when an element is expanded; however, in the mozilla doc, Nodes with children do not have articles. So users can click on an end node on the side nav and then it page goes to the top. in our case, if a user wants to visit for example . She would be scrolling the whole side nav 3 times. If we really need to go in that direction, the other option would be not take the users to the top of the page regardless of where they click on the nav. but when they reach their destination in the sidenav tree, they will have to scroll up to see the article's beginning.
Hi IMHO, and all honesty I really do not care about the footer. Meaning we are on the same page about its priority. The idea is the best usability of the site itself. I am not advocating that we should not make it so that people do not have to scroll. All I am interested in is making the UX intuitive and not painful from a user's viewpoint. I completely agree on your well researched thoughts about scrolling. But I keep re-iterating to test the site ourselves in A/B strategy and see what works best for us. In my tests, I found it weird and awkward to scroll when I am working through challenges, or looking through guide articles (not casually browsing like on a news site). So, from my tests: On the guide, the footer should be below the content element (the right guide article itself). The content element will expand in height depending on its content. This will keep the footer out of the view at all times. That is The footer can be hidden out of view under the viewport. Your MDN example, does something same. At all times, the content block is on the top of the footer. On the learn, I think we should get rid of the internal scrolls. But this also means, we should keep three things always in view: editor (labeled 2 on my screenshot) buttons preview panel (labeled 4 on my screenshot) These are the most interacted elements on the learn app. Only the instructions (without the buttons, will scroll) The footer can be hidden out of view under the viewport.
Maybe for the challenges and guide page we could make a smaller footer similar to the one github uses.
!
Re-opening. GitHub closed it automatically. This issue will be taken care of in repeated iterations.
I believe the decision is to make the sidebar more concise so there would not be a need to scroll. Although displaying the sibling nav items were discussed, I think it would be best to move forward with something similar to the following convention where only children and parent nodes are in the sidebar. !
Yes, let's get a variation that you propose in, and test it out.
I have a couple of suggestions on the footer: I feel the links are too cramped and need a little more padding between them Headings can be simplified There should be substantial space between the block of text on the left and the links At the end, it could look somewhat like this: !
I really like your design, it definitely looks better than the current one. Here are a couple of considerations. The logo looks great, but it makes the footer area taller. As a general rule we would like to save as much screen real state as possible. To be accessible we need a way other than color to establish visual hierarchy between headings and footer items. I encourage you to take screenshot of both the current version and your design so we could compare their sizes and their white space allocation better. After that let's have a pr and merge the changes.
Also I don't think you're allowed to change the colour of logo according to https://design-style-
let us address side-nav related suggestions and improvement in the above-referenced issue so it does not get confused with the footer.
Here are the screenshots for screenshots for comparison. ! ! !
would you be interested in making the footer titles more concise as suggested?
I always have this message always poping up when i run my code "Your Footer do not all have a white color" i have done all i could but cannot detect any fault in my code. this stopped my html from running further. I gave an inline style with white as color yet the problem continued. I just don't know what to do again


Donate modal is rendered/animated twice when invoked
Steps to reproduce. Complete at least 9 challenges. See modal once per session.
alternatively, change this line to true and get the modal after completion of each challenge.
Just in case someone tackle this before I get back to it. it seems like the app gets updated after the donate modal is up. so that might be causing the flashing. redux devtools actions order: <img width="338" alt="screen shot 2019-02-22 at 6 46 38 pm" src="https://user-"devtoolds error: <img width="642" alt="screen shot 2019-02-23 at 12 19 40 pm" src="https://user-"and here is where the modal is being called:


Describe your problem and how to reproduce it: Anchor links such is not working properly in several languages. For sure is not working in: Russian, Spanish, Chinese languages. 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): !
These 3/4 PRs (exlude one - which I PR'd to my own forked-repo) may close this issue. Please review them.


Describe your problem and how to reproduce it: Link to "Guide website" is different from English version. Portuguese version is redirecting to "". English version is redirecting to "". Add a Link to the page with the problem: English version Portuguese version Tell us about your browser and operating system: Browser Name: FireFox Browser Version: 68.0 Operating System: Windows 10 Professional


Challenge: Video: The code is supposed to create a two-column layout but there is a typo. Video has: Should be: It would be nice to have the video fixed if possible.
I would like to help but I need some more information like the download link for the video to edit it and overwrite columns on colums.
Is it possible to get this video updated?
this still needs to be fixed, if possible.


Front end project pages have no margin and hug the window: <img width="370" alt="Screen Shot 2019-08-20 at 9 12 57 AM" src="https://user-"All other pages have a margin I believe: <img width="373" alt="Screen Shot 2019-08-20 at 9 12 48 AM" src="https://user-"I can fix this one
from the screenshots. does adding a padding that way causes a horizontal scroll?
No, I'm pretty sure that's just my firefox being goofy again - that screenshot is from .dev and it shows this at the bottom on my firefox... <img width="319" alt="Screen Shot 2019-08-23 at 7 40 55 AM" src="https://user-"causing the horizontal scroll bar to appear, but it's fine on chrome.
That should get fixed by , unless there's some other stuff overflowing. It just needs
Thinking about it: do you want to make that change in your PR as well, That way you can see if there are any other problems.
Looks like it might not be my firefox - I found that when signed in, it displays the button in the picture above - but when not signed in, the button says "I've completed this challenge" So it's not the margin causing the scrollbar.
looks like that property does fix the scrollbar issue - <img width="292" alt="Screen Shot 2019-08-23 at 8 23 25 AM" src="https://user-"but it's strange that the button displays different text based on authentication


I did not see anyone raising this issue, but I don't think the background of the preview in default or night mode should be anything other than white unless the coding exercise specifically changes the background to something else. The preview should be exactly what you would see in a browser. In almost all the challenges, the background would be white. !
I share your concerns The reason we changed the color of the display is that a white display is discomforting to the eye on night mode when everything else is dark. so we injected some styles to harmonize the view in general. That convention was carried over to light mode.
It might be better to default to white in light mode, same as the editor. As for night mode, some of the css challenges might need some changes to look reasonable in both modes. For instance, in night mode, looks like this: ! If we change the seed code to it would become ! It would have to be case-by-case, though.
That is a very good point. I will defer this to
Logically, I have to agree with - displaying anything other than what the code in the editor would produce might confuse campers. But it is a little rough on the eyes to have the white background on night mode. And the current theme looks a lot better - so it's a tough call. I scrolled through a number of challenges, and there's quite a few that don't look right with the blue background. So there might be a decent amount of work to go through and fix all of them.
I really like the new theme, but a preview should be a true preview. We want to make this like the real world. Developers working in some kind of night mode on their favorite local editors, still expect to check the "real" result of the code they have written on a browser or in a preview mode in their local editors. The night mode should only apply to the editor or the non-preview part of the FCC site. Anything else is just not realistic.
The preview should be white.
I agree with everyone here on this thread. The preview should remain white. Note that soon the preview won't always be there on the right side "brightening up" the user interface and ruining people's night mode experience. Soon we will have a single column layout where you the preview only shows up when you click a button to show it, or it loads in a separate tab. - could you update the styling on this so that the preview area is unaffected by our styling?
Definitely.


Is there a reason the certifications have an apparent margin above them? ! It's bigger with a taller viewport. Seems like there should be no margin there.


<img width="616" alt="Screen Shot 2019-10-01 at 3 26 24 PM" src="https://user-"This is a picture of my own private profile locally, seems like it should say something like "You have not made your profile public" instead of "mot01 has not made their profile public" And as I'm typing this, I'm wondering if a profile should render completely if it's theirs, even if it's not public - there's no way for a user to see what their profile would look like if they don't have it public.
That does look better. The next sentence should be something like "You need to change your privacy settings in order for other people to view your profile". So, they should be able to preview their public profile, basically? Sounds good. If going that route, it would be good to emphasise (even more than currently) that their preview is not public. It wouldn't be good if people went to their preview, saw that it looked right and didn't notice that they were the only people that could see it.
Yes to this, please open a PR. It seems to have been an oversight. I think it's fair to leave the warning on their own pages while showing this page in that case, which means we may need to update the copy for them.
Can you clarify what you want here I'm not sure I understand - do we want to keep the page how it is, and just change the two sentences? Or do we want to display their private profile for them?
Yes. Sorry about being vague last time. I thought about this a little more and I think we should show the users their own profile only when they are signed in. Here is what we should achieve at the end of the day: profiles which are public are visible by default. settings button is only available if the user is on their own profile page. who are not signed in and are visiting profiles which are not public and are not their own are shown the current notice and warnings: ! who are signed in and are visiting their own private profiles are shown the same notice as the above screenshot in point 3 with the verbiage stating "You" and the settings button available. The rest of the their profile is visible as a preview.
So, there'd be a button on their private profile labelled 'Preview profile' and a label 'preview how your profile would look if you made it public'. Clicking that would take them to, say, which shows the preview (and a notice saying that it's currently private). Something like that?
Nope, we do not need a new route. Their profiles can be rendered just below the notice like so: ! Here is the copy: You have not made your profile public. You need to change your privacy setting in order for your profile to be seen by others. This is a preview of how your profile will look when made public.


Is there any reason not to put the real and in ? I reflexively put in a dummy key when I set this up, but now that I revisit this, that seems like a mistake.
Yes, actually it broke the .dev build unsurprisingly , I think noticed it and reported in the team chat. My bad for not catching this previously. The real fix is to ignore and move ahead when the keys are not available are default. We do this with google application credentials.
Oh, I assume it broke the build because .dev isn't using I didn't think of that... So, we should implement the fix and add the public keys to , right?
I was thinking something more along the lines of (example from api-server): I would try to avoid hundreds of contributors hitting the indexes locally. Ideally people should get their own keys. While the key and index is public, it's still not recommended to hit it?
As for the production keys, it's actually fine because the build should fail. I have since the keys in the build pipeline.
One thing: if we don't want contributors spamming search (fair enough!) I think there should be some docs that let people know what's what. Either that search isn't available in development or the steps they need to take to make it work.
I agree. We should add some docs on getting keys, or perhaps creating a test index and sharing that instead


The font size for the CTA could use similar breakpoints for the h1, h2. and set to 1.2 - 1.3 rem !
I'd like to work on this.
can i?
Where is this button?


<!-- 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: I feel like the description for the code in the is a little confusing and can be further cliarified. For example, the description is this: "... Like strings, arrays use <dfnzero-based</dfnindexing, so the first element in an array is element <code0</code." <strongExample</strongTo be a little more clear and following the index concept, this could possibly be explained like this: "... Like strings, arrays use <dfnzero-based</dfnindexing, so the first element in an array is element at index <code0</code."


Link to the challenge: Adding a global flag to the regular expression has the test fail for the first two assertions. This can be confusing considering also how the challenge is introduced, since the code snippet explaining the concept shows the global flag being used. Update the code snippet introducing the challenge: However, since the global flag should not affect the outcome of the test (to what I can ascertain), a better suggestion would be to update the test itself, to account for its presence.
Thanks for the detailed report! I think it would make sense to do both. The isn't necessary, so it should not be there and it's not wrong, so we should not reject it.
It might be better to remove the global flag from the challenge example unless we also explain how and give some info on . I think in order to really know if we should fail the test when using the global flag we would have to consider how the code is supposed to run. Are we "emulating" that each is being run in isolation or are they executed one after the other.
Agreed. This challenge focuses on teaching the user about groups - the use of over seem incidental to me, as does the presence of the flag. Also, the challenge asks you to make sure so it's only asking you to use against once.
But it is when the test is called multiple times on the same regex. Isn't this kind of the same as when users use global variables which end up causing the user's code to fail for all tests except the first because the the global variable does not get reset each time?
I'm dumb - I'd completely failed to realise it's being run twice, once in the user's code and again in the tests. You're quite right and PR fixes that.
Challenge: Fails with: Passes with:
The example/seed code moves to 14. I guess one easy option would be to change the example string to something like "Fun with regular expressions!", that should let all the tests pass and accept the global flag. The problem is if the camper uses a different string to test the regex it may fail. So we might just have to reset before running the tests.
I assumed it would be the exact same fix as what Tom did:
yep, that's what I concluded. I think we can apply Tom's fix in , rather than modifying the tests. Either works, but that should be more concise.


<img width="1396" alt="Notifications_Twitter" src="https://user-"


<img width="682" alt="Screen Shot 2020-02-03 at 7 30 48 PM" src="https://user-"For some reason, the font changes halfway down this introduction. . I don't see the problem at first glance. Probably just a spacing issue or something.
Hi . It looks like the problem caused by HTML tags in the text. I removed them and it looks better. Please check PR.


Describe your problem and how to reproduce it: Go to: Look for: Test outputs in the bottom-left corner Duplicated test output: Add a Link to the page with the problem: Tell us about your browser and operating system: Browser Name: Chrome Browser Version: 79 Operating System: Mac OS 10.13.6 If possible, add a screenshot here (you can drag and drop, png, jpg, gif, etc. in this box): !
The same duplicated test output exist in the following Challenge: Go to: Look for: Test outputs in the bottom-left corner Duplicated test output: The Presentational component should render an h2, input, button, and ul elements. !
You are correct, the 2nd test should actually state: The <codePresentational</codecomponent should render to the page.
Taking this up.


Describe your problem and how to reproduce it: As discussed at exhaustive length , the username requirements don't technically preclude the username "007". The requirement "Usernames should begin with a letter" should be . Add a Link to the page with the problem: Low-hanging fruit for a new contributor.


The logs for are quite verbose for some reason, making checking CI failures etc very difficult. Currently is logging all the solution, code etc for challenges? We need to avoid this.


When navigating to and selecting a monthly donation, the text below says "Confirm your donation of X / month with PayPal" This text is coming from: There is logic elsewhere in this file that handles the different frequencies, it seems like this logic is needed here as well.
Can I try this?


Describe your problem and how to reproduce it: Declaring a loop using as the declaration to solve the challenge does not pass one test. Add a Link to the page with the problem: Tell us about your browser and operating system: Browser Name: Chrome Browser Version: 80 Operating System: Windows 10 Pointed out in this forum topic:


<!-- 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


Is your feature request related to a problem? Please describe. I noticed that there is a difference between the headers in and . Describe the solution you'd like Everything else is consistent except for the header. A solution would be to have it consistent throughout. Have it display "Learn to code at home" in both links. On the second line. Additional context Add any other context or screenshots about the feature request here. <img width="1095" alt="without home" src="https://user-"<img width="1093" alt="with home" src="https://user-"


Is your feature request related to a problem? Please describe. The activities count on the search bar in both /learn, and /news is different than /forum. Describe the solution you'd like A clear and concise description of what you want to happen. Fix the count on activities. Additional context Add any other context or screenshots about the feature request here. <img width="629" alt="Screen Shot 2020-05-11 at 23 27 55" src="https://user-"<img width="730" alt="Screen Shot 2020-05-11 at 23 27 39" src="https://user-"


Describe your problem and how to reproduce it: Not Major: The sentence grammar should read: What type of HTTP request is usually used to access a website? Currently reads: What type of HTTP is request is usually used to access a website? Add a Link to the page with the problem:


In the challenge, the following line tells users to use and , but the tests do not force the user to use and . Since this is an algorithm challenge, I believe we should let the users write the code any way that solves the problem. My suggestion is to change the line's text to the following: Any thoughts?
I'll try to solve problem
Before creating a PR, let's wait for feedback from at least one of the users I tagged.
sure, that sounds like a good idea, especially since we're not testing for those methods.
I think this sounds like a good improvement
ok
Go for it.
Nice, I'm thinking use on the testString, but I first want to know what's the asset version used on this repository. If you want that I use other way, please tell me. I also want to learn more about JavaScript
Or use keyword?
Hello , There should be no need for either of those. The idea is to change the wording of this line of text: Use the array methods slice and splice to copy each element of the first array into the second array, in order. To this: There is no need to change the tests, as they do not check for the or methods.
Thanks. It's very much easy!


I think the description of the checkfunds method () is faulty. ! It should be: "A method that accepts an amount as an argument. It returns if the amount is greater than the balance of the budget category and returns otherwise. This method should be used by both the method and method." This is more logical, makes the code simpler and actually works with the tests. Browser Version: Firefox 79 on Ubuntu Linux Operating System: Ubuntu 20.04 LTS
this is genuinely a mistake as how can the amount which needs to be withdrawn be more than the balance in the ledger. amount can only be withdrawn if it is less than the balance in ledger. however there can be a probability that a part of that amount can be deducted from balance but the challenge stated nothing for the same. also the test module asked for a False return value which i think is wrong here in the challenge. !
Hello! Has any resolution to this been implemented? Im not entirely sure what to do about submitting this project because this error still exists. Any help is greatly appreciated!
I don't think so, hasn't changed. It doesn't matter though, design the project as if it was modified the way I suggested and it'll be fine.
Thank you, for opening this. For anyone reading, before a fix: Change this: To this in the file: Can I get a sanity check here? What needs to change on the fCC repl account? I am opening this up to first-timers only Here is the repo: The README description makes sense, but the tests do not, in my opinion. So, the README does not need to change.
Thanks for your patience and for catching this Also, thanks for @ mentioning me, Sorry I didn't see this sooner. My first thought is that we don't need to change the test, but just the README like suggested. It seems like is supposed to be used by the other methods to make sure that there's a sufficient balance before calling or . So in the current test: It makes sense that the first condition is false and the second is true -- you shouldn't be able to buy 10. It's just that the current instructions in the README doesn't reflect that. is that what you were thinking for the method? Also, to answer your question once we decide what to do here and fix things in the boilerplate repo, someone on the team can sign into and mirror those changes.
Thanks for that. I was thinking about the use of the method in an odd way.
I'm willing to work on this one if it's agreed that the README needs to be changed instead of the test.
Thanks, for your eagerness and patience. I was a bit quick with approving the change to be made. So, right now, I suggest you leave your current PR open as is. Once we come to a consensus here, we will be sure to let you know which changes (if any) to make on your PR.
no problem. I think it could work the other way around, we would just need to change the description a bit more and maybe change the method name to something like . I heard back from and he agrees that changing the README makes the most sense. The wording that suggested sounds good to me.
Just mirrored the changes from PR in the project: Thanks again everyone for your suggestions, reviews, and for this fix.


<!-- NOTE: If you're reporting a security issue, don't create a GitHub issue. Instead, email We will look into it immediately. --In the exercise, one of the condition is "The globalTitle variable should not change." but it returns an error if you modified the var (for debugging) at the top, even if your function doesn't mutate it. The verification should be that the initial globalTitle variable is the same as the final one, instead of simply looking to match it to a string. Add a Link to the page with the problem:
Hello there, Ideally, I agree, the tests would be robust enough to account for something like this. However, there is usually a reason for the comments, and users should not expect their code to pass, if they ignore this. If others agree to change this, I would not be opposed to the first test being removed. I cannot see it being used in any meaningful way in the seed.
I would argue that, as this lesson comes after the section, users should be somewhat comfortable with using . Changing a global variable's hard-coded value seems like a less practical debugging method than logging a function call with the argument value to debug. Additionally, as mentioned, the is present to indicate which portions of the seed code should not be modified.
for debugging purposes, instead of changing a variable you are told to not change you should change the function call, wiritng there but, yeah, it's a thing that trips many up. But dealing with small details is also something that should be learned
I believe the original purpose of the global variable was to actually pass a variable with a known value () to the function and validate that the function does not mutate it. However, the tests do not even check for this case. Such a test would look like: As states, the user should be writing their own calls to the function to test it, instead of changing the global variable. One way to possibly handle this, would be to remove the global variable declaration and use something similar as I mentioned above: I would argue, this kind of test should be to all the functional programming challenges, since this is one of the main concepts to learn.
I can go ahead and submit a PR if we'd like to go this route.
can a string even be mutated in that way? shouldn't strings be immutable and so that is not possible in any case? the test should be to make sure the user reference function parameter instead of a global variable even if the global variable is present, I think To see if passed in argument is not mutated should be a thing to check with arrays and objects, as those are mutable
you are correct that strings are immutable, but it is possible to reassign the value of the variable within the function (which is bad practise, thus the reason for the test I believe).
I mean if the global variable is removed from the editor you can't change a global string by operating on function parameter, right?
Correct - if we remove the global variable entirely, then it cannot be modified. If we leave it, it can only be modified by assigning the value to that variable name within the function. However, in general it is bad practise to modify the values of a function's arguments directly - with strings, it is less harmful than it is with arrays, but I believe that was the intent behind this particular test case. !
I should not have used the word "mutable". Looking at the challenge again, I think it would be fine to remove the global variable all together as it is not used in any other test (except the first). So, my suggestion is to remove the declaration and remove the first test, unless someone else can find a good reason to keep them.
I have some errands to run, but can submit a PR later this evening. If anyone else would like to tackle this before then, let me know and I'll gladly step back and give you the opportunity!
I would like to work on this issue, if that's okay with you .
By all means, go ahead! As a note, issues labelled with the "help wanted" tag (like this one) don't require permission before submitting a PR - I only made my earlier comment to ensure my interest didn't deter other contributors. Feel free to let me know if you have any questions!


Describe the bug The completion model buttons are not correctly aligned. To Reproduce Steps to reproduce the behavior: to out. to the challenge run tests error Screenshots ! This is only on the development server, probably created by the css changes introduced by the landing page redesign.
I am able to confirm this bug. Looking at the elements: The Go To Next Challenge button has a left margin of 5px. ! The Download My Solution button has a top margin of 5px. !


If there is no name set in the settings, doesn't matter if name is public or private, instead of falling back to username, it gives an error and certificate is not shown
I assume the best UX is for a note to appear, if the Camper tries to set to public, without having first set to something.
the error appears also if name is not set and set to private - certificate does not appear saying that one has to put a name (I'm adding this on top message too)
Picture !
The objective here then is to show the username on the certification if the name is not set ? Maybe some message then must warn the camper that his username will be shown on the certification instead of his real name (I am not sure though that a certification given to a weird username would be ok)
already if the name is private the username is shown, this issue appears if the name is not set
Apologies if this has nothing to do with you, but I vaguely remember you being involved in the discussion I cannot find: Was there any conclusion as to disabling the certification until a Camper has set their name, and made it public?
Sorry for the late response here - The only thing that comes to mind is
Either we would need to remove having usernames as a fallback , or having a public name requirements when displaying certs. I now it might be inconvenient, but I vote for removing the fallback for simplicity and keeping the public name as a requirement.
Discussing in issue triage meeting. a camper sets the to a non-empty string and , the certificate renders with the . .When a camper sets the to an empty string and either or , the certificate does not render and redirects to with a flash message. We should update the behaviour of to mirror the behaviour of , and avoid using the username field as a fallback entirely.
Hello Is this issue solved ?


Describe your problem and how to reproduce it: SVG image in the challenge description is affected by SVG logo in navbar. They share some element s, what results in malformed image in description. Changing s to make them not overlap should deal with the issue. The simplest way seems to be, to change one of the images (probably one in the challenge), for example by adding unique prefix to s. That wouldn't eliminate completely the chance for some other conflict in future, but that's probably not something to worry about much. Add a Link to the page with the problem: If possible, add a screenshot here (you can drag and drop, png, jpg, gif, etc. in this box): Current look ! Proper image !


<!-- NOTE: If you want to become an author on freeCodeCamp, you can find everything here: --The sidebar text on "Standardize Times with the HTML5 datetime Attribute" says "The datetime attribute is an inline element that can wrap a date or time on a page." The attribute cannot be an inline element. Change it to: "The element is an inline element that can wrap a date or time on a page. " or "The is an inline element that can wrap a date or time on a page.". !
Hey good spot there I'll sort this


If I am signed in as a user while on and log out, I am redirected to the english home page. redirects to the chinese page.
wrong endpoint in server code? Thats what I would assume, but am not sure how to locate where the correct endpoint would be placed in the code.
Here is the endpoint: and most likely the issue is with the redirect "predictor" utility: However we would want to confirm the cause before we open this up for contributions.
yes, that's the issue. To fix it we can grab the from and combine that with to get the desired redirect.
Should that be done in the getRedirectParams function or should the path be put together from the data set returned by getRedirectParams in
You need to construct the URL from the values you get from the as a consumer of the function and not change its existing behavior. In simple terms the change is required within the endpoint handler.
That makes sense.
I got the codebase installed locally is there a way to run just
Not sure what you mean by running only one file? The API server is an express based application (Loopback 3). You need to start up at least the api-server app. If you want to do that, you should be able to do that by navigating to directory and using the run-script commands in its
I'm trying to figure out what gets passed in pathPrefix, from getRedirectParams, when one signs out from the Spanish page. But when running the site locally, switching to Espanol leads to a "This site can't be reached" error. Just for clarity's sake, I realize now that is not meant to be run alone and I am using the "npm run develop" command to try and recreate this bug locally. Is this a bad approach?
Ah - right. Sorry about sending you on a goose-chase. When you build locally, we do not have a way yet to develop both and at the same time alongside the API app. I am going to untag this from help-wanted and assign this instead. Oliver, you should be able to test / reproduce and recommend a fix on our staging instance - not sure because this may be needing access for "close to prod" type setup.
I was just in the process of going through the steps, when you commented Mrugesh! I'll leave them here for the curious: Unfortunately testing locally is quite a pain, because we use since is entirely separate from . It's a bit of a slow, memory intensive process, but you can do this: in your .env file Then you can go to


Describe the bug The arrow to the left of Expand courses/Collapse courses buttons points down when the list is in collapsed state and points up when the list is in expanded state. This is opposed to standard behavior of similar features in other apps. To Reproduce Steps to reproduce the behavior: to at the arrow to the left of Collapse courses. It points up while the list is in expanded state. on the above mentioned button to collapse the list. the list is collapsed state but the arrow points down. Expected behavior By convention, the arrow should point to the right or up when the list is in collapsed state and point down when the list is in expanded state. Screenshots ! ! Desktop: OS: Ubuntu 20.04.2 LTS Browser: Opera Version: 76 Additional context Also tested this on the same OS with Firefox 88.0.1 with the same result.
I agree, we should have the arrow pointing to the left when the courses are collapsed and down when they are expanded.
Hi Ahmad, I think you mean we should have the arrow pointing to the right when the courses are collapsed, right?
Hi. I'm new to contributing to FreecodeCamp, and just read the guidelines. With your permission, I will fork the project and try to solve the issue with a pull request.


! there is written stUnes instead of stOnes This looks 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: 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.


!
If possible please assign this issue to me! as I am new to open source
For future reference, it would help to have a link to the challenge when creating an issue It's To clarify, you want to change: to: That's not bad. What about
Yes, that is better! and sorry for any inconvenience as I am new to open source
Please, can you assign me to resolve that issue? That'll be a great start for my open source journey!
We do not assign issues The first PR that fixes the problem will be accepted. If you have not done so already, you should read the . Thanks for your interest in contributing If you have any questions, feel free to ask - or you can reach out in the
Please review the PR
please be patient, people will review when they can.
Alright!


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)


The instruction looks kinda confusing. The word can be replaced by . ! <!-- Please complete the following information. --- Device: Laptop OS: Windows 10 Browser: firefox Version: 98.0.1 (64-bit) <!-- Add any other context about the problem here. --
Yeah - I see what you mean here. While it's implicit and really a nitpick, we can be clear as water in verbiage.


The title for code preview modals is left aligned. The rest of our modals have a centered title. to on view -show code left aligned title it -click view -show project is centered Center the title Code modal: ! Preview modal: ! I think I would just center the title, not the HTML/CSS/JS title
I wonder how should I tackle these, If I can tackle it of course. there is there solution: adding inline CSS to the adding with value to file we can go wild and change value to include
I would look at how the other titles are getting centered. Maybe there's just a class to those we didn't add on the others or something.
I see there is with value , I will start working on it


Even though on macOS it would seem that Ctrl+Enter does not work & Cmd+Enter does; website still tells you to use Ctrl+Enter. I think this might end up being confusing for some - and should be fairly straightforward of a change. All coding pages You must use macOS for this issue. to any coding challenge your changes will see button underneath says "Check your code (Ctrl + Enter)" Ctrl + Enter will happen. Cmd + Enter behaviour will occur. It should tell the user correct shortcut (Cmd + Enter) instead of (Ctrl + Enter) depending on the platform. <img width="647" alt="Screenshot 2022-06-09 at 16 52 28" src="https://user-"- Device: M1 MacBook Pro OS: macOS 12.0.1 Browser: Safari Version: 15.1 No response
I would like to handle this issue as a first contribution. My proposal would be to use (). Use Ctrl + Enter as a fallback and define a special case for macOS cases. Alternatively we could use () however that is not widely supported as opposed to . As far as I can tell on mobile we don't show modifier keys on the button; so I'll preserve that behaviour. I can't change the labels on this issue - I don't think this is severe enough to be classified as a bug based on definition; my apologies regarding that.
Hey thanks for raising this issue and don't worry about the labels, they need to be handled by maintainers. What you've suggested does sound good to me, so if you're happy to take this on, that's awesome.
I've created the PR however; I've updated non English translations as well in here to use dynamic values; however not really sure how to align it with the crowdin. Since updated translations should coincide with this change; any opinions on this?
Do not worry, I think you are putting too much effort into it. Simply drop the changes to those files and as the bot mentioned, they will get updated eventually via our translation workflow. As for the bug itself: I am 100% sure at some point we introduced a regression, because (in addition to the implicit Cmd + Enter) was working since the inception of the code editor. do you happen to know if this was just an accident with the new editor? To be clear it should always work with regardless of the OS in addition to implicitly working with just like it was before.
I can't check this myself, since I don't have a mac, but it should work. This is the code: and could you try and reproduce the issue?
I've shot a short video of the behaviour I am seeing; if that helps.
That does help, yeah. It seems to be ignoring the command button entirely, which is odd. I can't investigate myself, but this will help anyone that has access to a Mac.
Thanks for the video I don't recall if ctrl+enter ever worked on mac - I may or may not have ever used it. I just hit buttons without looking I'm not sure if you are trying to show something specific with those code snippets - that's the code on main. Which results in the current behavior. Edit: I get it now - you were just showing the code and that it should be working. I thought you trying to show new changes or something. Here's the current behavior I see (all on mac) - there's some nuance... New RWD: Execute: only command+enter works on mac Submit: Enter alone works, so any key+enter works Legacy RWD, JS, etc: Execute: Only cmd+enter works Submit: Enter alone does not work - both ctrl+enter or command+enter do work Multiple choice Python video questions: Execute: Enter alone does not work - ctrl+enter and cmd+enter work Submit: Enter alone does not work - both ctrl+enter or command+enter do work Projects with URL inputs - e.g. front end libs cert project, or backend api challenges, etc: Execute: Enter alone works cause it's a form I suppose, which also means any key+enter works Submit: Enter alone does not work - both ctrl+enter or command+enter do work The last three groups there have the same submit modal, so the behavior is the same on all of them. What I see is that the submit modal, and the video question both allow [ctrl or cmd] +enter - we should see what those are doing. Looking into to it...
K, I think I found a fix. ctrl+enter works when the hotkeys package/component is used. So crtl+enter actually does work on the legacy challenges when the instructions are focused. When the editor is focused, those hotkeys don't get used, the actions/hotkeys we add to the editor take over. I made ctrl+enter work when the editor is focused by changing that line 431 mentioned to this: That should work. There's a number of actions in that file that use the same hotkeys - lines 457 and 469 for example. There may be others. I think we should be safe to change all those. Do you mind playing with that a bit and updating the PR to make those changes.
Nice work, Tom. I thought CtrlCmd was "Ctrl or Command", but it must be Ctrl on Windows and Command on Mac :man_facepalming: I guess this has never worked properly.
happy to give it a look and update the PR. Thank you
Here's an I had created sometime back when working on the . Apparently the logic goes like this - the first word is Windows and the second one is Mac button.


Overly strict whitespace tests The space between the input element and closing label tag shouldn't matter, right? No response All No response
Here is the markdown file: we can use regex in these tests Something like , but we have to use regex which is headache.
could we just trim the string?
Sure, I don't mind, because they aren't required to type text in this step, but the description will be altered
I'm not sure I see why the description would be changed. I was thinking of this:
Yes, this line while be unnecessary. And the test will check for `EnterYourLastName:<input' But I maybe misunderstanding something
Trim won't remove spaces in the middle. We are comparing against:


there is a typo in cramers-rule challenge, is used instead of which is wrong these sentences should be changed to Here is the markdown file 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.
I wanted to work on this issue
feel free to open pull request after you have read


In the page above, a question (Which activation function switches values between -1 and 1?) uses 'switches' when the actual word should be 'squishes' (like Tim says in the video) The question should be: Which activation function squishes values between -1 and 1? No response Device: Laptop OS: Windows 10 Browser: Chrome Version: 103.0.5060.114 No response
Just changing the word in this file should suffice, let me know if you want me to open a PR.
please go ahead.


There is a typo in learn-intermediate-css-by-building-a-picasso-painting, is used instead of . Here is the typo Here is the markdown file 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.
can i take this up
feel free to open pull request after you have read the guidelines


In both the sentences on this line the word should be surrounded with backticks: In this line there is a period (full-stop) where there should be a comma. 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: 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.
Hello, I want to work on this.
Heyy, is this assigned to anyone?
I will work on this.
Hey! I would like to work on this issue.
Hi I have fixed the issue and you can find the PR here
Is this issue available or has it been resolved?


The word json should be capitalized in the hint text: The endpoint /json should serve the json object {"message": "Hello json"} Referenced challenge file: curriculum/challenges/english/05-back-end-development-and-apis/basic-node-and-express/serve-json-on-a-specific- Use capitals instead of lowercase letters when referring to JSON No response Not applicable No response


In RTL, these texts will be broken, because they are a mix of special character, numbers, which follow different direction in RTL than LTR. ~1.2.13 1.2.x 1.x.x To fix these, add code elements around the version number. File: change to there are multiple instances in these files All of them should be wrapped in code elements Speaking of weird interactions. Using special characters that relay on direction will break in RTL, because it will be reverted, in example. will be we can wrap in
I want to contribute towards this issue. Can you assign this issue to me.
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


'just' is a bad word. If it doesn't already, our style guide should dissuade it. It implies the task is 'simple' which can frustrate learners who don't get it right. N/A Text should use friendly language. Possible fix: Note: the selector technically doesn't have the properties; the selector is part of a CSS rule that sets the desired properties. No response All No response_


We have instances which is used to provide spacing for both side of the component to wrap it up in specific space Example: We can make the Spacer component smarter, and allow it to accept children, so the code be like. But in doing so, we need to: change the current values and how the padding is calculated, the old layout shouldn't change with the new spacers. change which padding has the new value, I think paddingBlock, which is paddingTop and paddingBottom, is the new padding property . I am requesting this because this will make live simpler, and we have to worry about less elements while controling the layout, and if we have specific cases, we can still use
There is an abstraction with , we need address, it's better if we can specify the direction of the padding. but this can be sorted in another issue, for now to solve this issue please allow to have children prop inside of it, but the current styles of the curriculum shouldn't change. 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 prolific contributors will guide you through this.
Hey I have few questions while looking into this issue are few files where we have used Spacer, so may I just change wrap the content where ever needed or it would be handled in new issue? (I guess it must be done in other issue, as it would be easy to maintain) have implemented the children props, so may I raise a PR? Will break this issue in 2 other issues as well. 1 for wrapping the places it has been used, and 2nd for specifying the direction. I was wrapping and trying it out, I found out that Spacer has been used at every 2nd-3rd line of code, so do we need such a wrapper which would give spacing to all components present inside it or only to the parent component? EG: If we have 2 divs wrapped inside Spacer, then does both div have the spacing around them or only the outmost div.
sorry for the delay, it was difficult to understand what you mean without looking at an example of code, so to answer you: yes the direction will be in a second issue, but the spacing and values should still be in this issue, so the pages won't look different Of course, please raise a PR after reading the guideline, nicely done . The Spacer shouldn't affect the children inside of it, think of it as creating a container with margin-block to space the area of the children inside of it from other areas
I guess we could close this issue
will be closed when the height is removed in the other PR


In module, currently, the instruction says: Write an directive that goes through a list: and assigns each variable to a class, where the part changes for each item. Each class should set the the respective color. In order to avoid any sort of ambiguity and to make instructions more clear, I think the instructions should be as follows: Write an directive that goes through a list: and assigns each variable to a class, where the part changes for each item to the respective color. Each class should set the to the respective color as well. In module, the instructions should be as follows: Write an directive that goes through a list: and assigns each variable to a class, where the part changes for each item to the respective color. Each class should set the to the respective color as well. <img width="521" alt="image" src=""- Device: Laptop OS: macOS 13.0.1 Browser: Chrome Version: 107.0.5304.110


Looking forward for reporting a security issue: Please report security issues by following our security policy: Describe the bug A clear and concise description of what the bug is. To Reproduce Steps to reproduce the behavior: The real url for spanish is so i expect that when running locally I could click on globe icon and then choose spanish to go to: http://localhost:8000/espanol/learn/javascript-algorithms-and-data-structures/basic-javascript/storing-values-with-the-assignment-operator But instead i get gatsby dev 404 page. Expected behavior I expect to be able to get to spanish version page. Screenshots If applicable, add screenshots to help explain your problem. <img width="1429" alt="image" src=""Additional context Add any other context about the problem here.
This is actually because the app doesn't build every language at once. It builds a single language - our internationalised sites are separate instances of the application. The language selector does not work locally because there is, in fact, nothing at the endpoint.
We could add a note to: like To see how the client renders in another language go to I think that should be enough.


There are some modules ( e.g: module ) within the section of the course where, when the user submits their solution ( whether correct or incorrect ), then they end up with the following warning within the console: Since this warning is not relevant to the solution submitted by the user, I think it could make the user feel confused. In order to avoid any sort of ambiguity and to keep things as clear as possible, I believe when the user submits their solution and if their submitted solution is not correct ( partially or completely ), then they should only be getting relevant error messages printed onto the console which can help them in debugging their solution, or if their submitted solution is entirely correct, then they should be getting the following messages only ( within the console ): ! Device: Laptop OS: macOS 13.0.1 Browser: Chrome Version: 107.0.5304.110
I can see that the error message is pretty cryptic even though it happened somewhere along the stack trace. If you don't mind, can you comment your JSX element so that me and other people can paste the code immediately in the editor and see it for ourselves ?
Can you provide the path of this file.
If you want i can work on this issue. Can you assign this to me. I am expert in react but first timer in open source community. So apologize in advance if i am going out of way in asking this on the thread
Pretty sure that has been there forever. It looks to be caused by running tests with the production builds of React. Maybe we can just use the dev builds. Tested it briefly and it didn't seem to break anything, but I'm not sure what the reason for using the production builds is. Or maybe the error message can be silenced somehow.
Hey thanks for looking into this issue. It would be great if this error message could be silenced somehow or as you mentioned, if using dev builds solves this issue, then I think it would be a good idea to do so given that it doesn't break anything. Let me know what you think about it. Cheers!
I'm pretty sure I have seen this error on other React (maybe React/Redux) challenges as well. I'm not sure if it is just because the prod build is smaller/faster or what? I see no real harm in using the dev build and it would/should make the error go away. But someone from the dev team will have to look at this and approve it. is using the dev build of React an option here?
I'm not entirely sure, tbh. Maybe something can provide insight on.
So, the problem is that we currently bundle react-dom-test-utils into the script that runs the tests, so changing the meta doesn't help us. However, once is in, we will have full control over the scripts that we import and can choose which build we want for the react lessons. I propose we revisit this issue once that's in main and experiment with development builds then.
I am interested to work on this issue. can anyone explain me more clearly what to do next?
Can I work on this?


I believe that Accessibility Quiz step 13 is the first time we introduce the CSS child combinator. (I've looked through the finished CSS for all previous courses and have not found it.) The instructions and hint for step 13 merely say to use it but they don't explain what it does or why it is needed. And really, since the only child in the is the , and there are no embedded s in the list items, then it is not obvious why it is needed in the first place. We also require the child combinator in later steps and I am not finding any explanation for it in those steps either. The current instructions for step 13: The hint if you don't use the child combinator: Add additional information to the instructions: Give the name of the selector (child combinator) Describe what it is used for Explain why it is being used in this step No response N/A I don't have any suggested text at the moment, I just wanted to get this issue recorded so I wouldn't forget about it. And to be honest, I'm not sure what the rationale is for requiring the child combinator here. Perhaps we are just using it merely for the sake of introducing a new feature in CSS?
It would make more sense to introduce the selector in an instance where it is actually needed and with its usage explained. Having said that I don't mind its use here. It should still be explained, because otherwise, what is really being taught? The why might be a little harder to explain in this example if it is only done as a coding style choice. Personally, I do think the child combinator is often preferable to the descendant combinator when applicable, especially when using type selectors. It is more explicit about the location of the targeted element which makes it less side-effect prone (i.e. later targeting elements you didn't mean to as more code is introduced or changed). It is less work for the matching engine. I believe it was used as a bit of a performance enhancement back in the day, but I doubt it really matters much today in most cases (it might if the DOM tree is big enough).
Adding an explanation of the selector would be good. Maybe something like:
Alternative explanation for CSS child combinator step 13: To select all children elements within the parent elements, we use the selector. In this step, use the selector to target the unordered list elements within the elements, and use Flexbox to evenly space the children.
I feel like that's a bit too reductive.
I'm happy with Naomi's suggestion. This seems like a good first timer issue if we are done discussing.
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: A pull request which resolves this issue should 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.


We are currently grouping the articles in the Trending Guides section using a : But I think we should use and to group the items instead. This helps communicate the relationship of the links better, and allows us to remove the attribute from the code (ref: ). Pseudo code: Links in the Trending Guides section are rendered as a and are grouped within a . The change should be made to the following files: Learn platform: News platform: Forum platform: There is no visual regression All attributes in are removed (there are 2 of them in the file) Both Footer snapshot and e2e tests are updated accordingly Bonus point if you can merge tests in this suite into a single test case: This change is to improve cost efficiency as e2e tests are expensive to run.
This issue is not yet open for contribution. I'll wait for others to review the proposal first before marking the issue up for grab.
Can i contribute on this
Sounds like a plan. This is also a problem on and
Oh, thanks I forgot about /news and /forum. I found the repo for /news but I'm not sure where the code for /forum is. Could you help link it to the PR description? (I've already the /news link there.)
Here is the relevant code for the forum:
Alright, thank you Shaun! I'm going to open this one for contribution. we will be happy to review your PR if you are interested in contributing
Can I contribute to the above issue?
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
This isnt open for contribution
Sorry, I'll need your (and the staff team's) help with this. The PR targeting /learn has been merged, but the changes include some CSS modifications and require local testing. I'm happy to apply the changes to the and repos myself, but I'm not really sure how to setup and test them properly I think it's probably faster to let a staff member handle these changes.
I know how to test and deploy forum changes, but I do not know how to test the changes in . Maybe can test please?