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
Niall Maher
16,985 PointsOpinion on my new website
Hi everyone! Basically I have learned most my web skills on here in the last 6 months couldn't find a job (here in Ireland) so decided to go the free lancer route. Here is my new website for my company that has just been registered: -> www.WebApes.ie I'd love to know what you guys think :) I'm still working on a interactive portfolio section but needed a website online for my business cards ;)
2 Answers
Francis Manning
3,125 PointsLooks really good Niall! I'm new to programming...How do you get the tabs to switch while the user is scrolling?
Awesome website.Wish you the best!
Kevin Korte
28,149 PointsPretty good, actually really good for your how long you've been doing it.
My one critique right now.
- Your background image is big, and slow. It's too big. It's a whopping 1.8mb. On my 50gbps landline it still takes about 5 seconds to load. On standard 4G data like many cell phones in the US have, it loads in about 5.5 seconds.
Check out https://tinypng.com/ it is free. When I ran your image through here it dropped it from 1.8mb to about 600kb, which is a huge difference.
Good job, keep it up!
Niall Maher
16,985 PointsAwesome! I completely forgot I had a compressed version but of course in haste to get my project online! Now it should be running smoother on a mobile :P Thank you! I'm going to compress my css and javascript files tomorrow just wanted to see if people liked the overall feel of it.
Kevin Korte
28,149 PointsMuch better, that shaved almost 4 seconds of the ready event time.
Think you could compress your icon.png and your map-image.png? Your map-image.png is now the biggest and slowest asset you have, weighing in at 356kb where your header image is only 65.9 kb. Because it's at the bottom we don't notice it load slow, but it still takes up bandwidth resources for mobile devices.
Style and design wise, you ave a good theme, good cohesiveness to it. A great platform to evolve on.
Another thing I noticed that gave me a sad face, you have these nice ease-in/ease-out animations when I click on a nav link, and it scrolls me to the the correct, section, but your back to the top link in the footer just jumps you abruptly to the top. I expected the same cohesive scroll animation. I'd see if you can add that. It's minor, but it's a UX problem to address at some point.
Niall Maher
16,985 PointsThe jumping up is annoying me too! It's minor but I agree takes away the overall feel :P Yeah I could definitely compress the 2 other files. I really appreciate the feedback :)
Kevin Korte
28,149 PointsCool man. Your css is js isn't out of control, but a little compression there won't hurt. I think you're in good shape.
It's good I'm nitpicking little things like that link not having an animation. There is much else there apparent that could be better. Really solid job.
Niall Maher
16,985 PointsIt's the exact sort of feedback I need so I can improve and produce a higher standard :) Keep the clients happy
Niall Maher
16,985 PointsNiall Maher
16,985 PointsI actually got the inspiration off www.codepen.io prob the best place to pick up random bits of code and see how it works. It's basically using jQuery addClass and removeClass at certain points to change the background color.