-
-
Notifications
You must be signed in to change notification settings - Fork 35.9k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Add a new step 1 to better explain what the pyramid generator is about #54801
Comments
Is there a way to show a pyramid similar to how the html/css projects preview their final results at the start of the project? |
Short answer, yes. However, this does create a new issue where the console is closed by default for projects that have HTML, CSS and JavaScript. So I am going to tag @ojeytonwilliams because he knows more about the editor then I do and if something like having the console open for HTML, CSS and JS projects is possible. If it turns out, this is possible then I am for this change. If it turns out this is not possible, then I am 100% against this change. We have seen a huge increase on the forum in campers checking the console more because it is always open and they can see their results immediately on the right hand side. |
Another option is to use Jessica's suggestion but to have that intro (to what a pyramid gen is) as the first step ahead of the current "step 1". Unless a step absolutely must have the learner change something in order to allow them to move on? |
I like the idea of adding a 'preview' step, mostly because it means we don't have to create a special case to handle the one project that wants console output as a preview.
It should be fine in production. The only issue I can think of is that it might not pass in CI, since the tests are expected to fail on the seed content. I'll have a quick play with this and see if that's the case and, if so, what needs to be done. |
Technically we have two projects that fall in this category The pyramid project and this gradebook app project |
Ah, good point, I'd missed that one. I'm not sure it makes sense to create a preview, though, since it would just be some console output without any context. The 'preview step' is nicer, imo, since it can include that context. Anyways, I've checked the tests and CI should be fine. The kind of challenge you wrote, @jdwilkin4, passes all the tests. Also, if you want, you can skip the hints section entirely. |
ok cool. I will go ahead and open this up for help wanted contribution |
Describe the Issue
For absolute coding newbies, a 'pyramid generator' will not conjure up anything meaningful and may leave users in the dark about where this project is headed (unless they have the foresight to click on the last step to see the final results), therefore it would be great to demo this in the very first step so everyone knows what the problem/goal is before starting this project.
First discussed on the forum here:
https://forum.freecodecamp.org/t/way-of-learning/689345/4
Affected Page
https://www.freecodecamp.org/learn/javascript-algorithms-and-data-structures-v8/learn-introductory-javascript-by-building-a-pyramid-generator/step-1
Your code
N/A
Expected behavior
t.b.d.
Screenshots
No response
System
N/A
Additional context
No response
The text was updated successfully, but these errors were encountered: