Welcome to the Treehouse Community

Want to collaborate on code errors? Have bugs you need feedback on? Looking for an extra set of eyes on your latest project? Get support with fellow developers, designers, and programmers of all backgrounds and skill levels here with the Treehouse Community! While you're at it, check out some resources Treehouse students have shared here.

Looking to learn something new?

Treehouse offers a seven day free trial for new students. Get access to thousands of hours of content and join thousands of Treehouse students and alumni in the community today.

Start your free trial

General Discussion

Please critique my practice page/'app'

So as I've been learning, I've been working on little side projects to reinforce what I'm learning. I created this practice page just to work on some simple stuff but I'd like help making it better.

I'm sure the code is atrocious and I'm very much still a beginner so please give me any and all advice.

http://codepen.io/anon/pen/EaKLej

7 Answers

Hello! It works great! I think its cool to see all those little tools together. The design is a little dark, but it looks good! I have a question, what is this codepen site? I've seen it quite a bit and i have no idea what it is. Is it like, a web developer website?

Codepen is, as I understand it, a place where designers and developers (or really anyone who is so inclined) can work on projects and share them with others easily.

The code in my example was written in sublime text 3. I just copy/pasted it into codepen so that I could share it easily. There are probably better ways to share code, but codepen works out really well for things like this so it seems ideal.

Thanks for trying out my little project. I appreciate it!

I like the app in general. The only thing, I'd probably get rid of is the javascript alert box, and just use ordinary text fields. With that being said, being a bit of a D&D geek, I loved the multi-denominational dice :) Keep up the good work.

I agree. Its annoying having the alert boxes pop up. Ordinary text fields would be much nicer.

I agree that the alerts are pretty annoying. I'm not sure I know how to produce the same functionality using text fields just yet. I've not tried it though so maybe I can make it work. I'll give it a shot and let you know how it goes!.

I really appreciate you all taking the time to look over it!

Well thanks! I really appreciate you sharing it with us!

Question, Nick. Did you code the app using javascript?

Actually, I didn't code it, Mark Shaw did, I just replied to your comment.

My bad. Lol. Just finished finals for the semester, and I'm a wee bit crispy in the brain.

I have 6 to do in the next 3 days. Not too excited haha.

Yes, it's javascript.

Here is a reworked version with inputs instead of prompts. Let me know what you think.

Also, your feedback on the previous version helped me to learn some new stuff so thanks for that!

http://codepen.io/anon/pen/dPXmbK

I would advise you make buttons for each one. There is only a few choices for each one, and its a pain to have to type in what you'd like for each one. You could just click the button and it'd auto output it.

I actually considered that and I think that's what I'll do next. I wanted to go ahead and do it with inputs just to see if I could do it. I'll get to work on it!

Already on your way to becoming a fantastic web programmer! Hahahaha