DEV Community

Yuri Alves
Yuri Alves

Posted on

Roast my site

SPFound is Where you can find help for your latest side projects or find side projects to help.

Website: https://spfound.netlify.com

Oldest comments (22)

Collapse
 
niorad profile image
Antonio Radovcic

the header-banner-layout is broken and nothing happens when I click the projects on the bottom.

Collapse
 
yuriguernsey profile image
Yuri Alves

Thank you for pointing that out to me. I will be sure to fix these problems soon.

Collapse
 
kevinhch profile image
Kevin • Edited

check your console, it has an infinite error messages of "VM10860:1 Uncaught SyntaxError: Unexpected identifier"

Collapse
 
yuriguernsey profile image
Yuri Alves

Oh, I see that error now. It only happens when you press the 'All' Filter button.

Thank you for pointing that out to me.

Collapse
 
jsrn profile image
James
  • If there are no results for a category, it'd be nice to convey that to the user somehow, rather than leaving it totally blank.
  • Having a "Home" button in the nav bar feels a bit redundant on a single page app.

Really love the header.

Collapse
 
yuriguernsey profile image
Yuri Alves

Thank you for the feedback. I will be sure to add some text to inform users when there are no results.

The website does have another page, but I still have yet to finish the page before implementing it to the website.

Collapse
 
pol_wekesir profile image
wekesir

On a desktop, I think the links at the top right of the navbar need a little more spacing between them. Just for elegance

Collapse
 
yuriguernsey profile image
Yuri Alves

Ok, I will be sure to add it now

Collapse
 
kevinhch profile image
Kevin • Edited

the product cards should have a margin top and bottom in mobile view, and the nav bar should be stay too.

Collapse
 
camto profile image
Benjamin Philippe Applegate • Edited

It's a tiny nitpick but dang the shade of blue on the project cards is just burning my retinas. Also how the view moves around jankily when changing filters.

Collapse
 
yuriguernsey profile image
Yuri Alves

All feedback is appreciated even if it’s tiny.
Anyways, I will sort that out as soon as possible.

Collapse
 
dtrancloudguru profile image
dtrancloudguru

The banner looks like you're trying to sell me something

Collapse
 
yuriguernsey profile image
Yuri Alves

Is that not the right approach?

Collapse
 
wattafot profile image
wattafot • Edited

First of all, great job dude!
i like your background, where i can get those images from?
i dont like the font u are using beneath the headline. its kinda hard to read for me. maybe change that :)

go on!

Collapse
 
yuriguernsey profile image
Yuri Alves

Thank you, I got it from undraw.com. I'll definitely change the font.

Collapse
 
rvarela94 profile image
Mr. Varela πŸ‘ŒπŸ»

A couple of notes:

  • With 1920 x 1080 resolution, the small text on the title is overlaid. I advise you to correct the "line1" and "line2" height. @media screen and (min-width: 1400px){ line1{ height: 45% -> 50%;} line2{ height: 49% -> 55%;}}

  • Maybe it's on purpose, but the image disappears at 971px width while testing for responsiveness to minimum possible width. You could place the image between the blue background and the title between 730px and 971px.

  • Maybe add some text when there's no results for a category.

Keep up the good work!

Collapse
 
yuriguernsey profile image
Yuri Alves

Thank you for the feedback. I really appreciate it. I will start working on those points right away.

Collapse
 
larizzatg profile image
Larizza Tueros
  • I like the text effect of the introduction and the font family for the title but it's too much for the lines. Instead use Righteous,cursive for the lines;

  • When no results are found the page jumps to the top. You can prevent this if you put a div with some padding and the default text that everyone already told you about.

  • Padding is your friend to create breath and space! Decrease the margin of product cards and add more padding (mainly on the top/bottom) to it. If you do it it's going to look off, that's because you had to put top:0 to the ProductCards Content

  • Add more padding (x) to the introduction on mobile

  • Add more margin (y) to the ProductCards on mobile.

I'm enamored with the logo and the header.