Hello freeCodeCamp community! Junior Dev here and I am working on building my skills to secure my first web dev position and I’ve created a product landing page for my portfolio. I would really appreciate some feedback before I submit it as part of my portfolio for jobs.
Some of you have already reviewed my portfolio and offered some fantastic suggestions; I did my best to implement those recommendations here. If there is something that you had pointed out before and I have failed to address then I apologize; feel free to point it out again!
I have a couple of questions that I am particularly interested in hearing about.
-
Images in the hero slider move slightly when a color is chosen from the color picker. Is this something I should address or is it too insignificant to worry about? (Images are Nano Banana, generated from Gemini.)
-
The second image displayed in the slider is slightly smaller than the other two; this was a compromise I made to maintain readability after I switched to a mobile-first design because when I used the
object-fit: cover;property it clipped the overlay text on the shoe. Is this approach fine? Or is there something better? -
Does the spacing of the hero text, ‘Signature Series’ and the h1 seem well-balanced?
-
I feel the shoe size chart and shoe size chip components look disconnected to one another; I’ve seen this approach used in some other designs but I’m not entirely sure about it and would be grateful for another opinion on this.
-
Any suggestions/critique for the tablet and mobile versions of the site would also be hugely appreciated; I’m using min-width media queries for 390px, 768px and 1024px and I am confused by how the font sizes look too small on the device toolbar while rendering properly on my physical device; perhaps it is just me and I am no longer able to objectively analyze.
Please note: there is a form on the page which opens via the ‘Own The Air’ CTA button; the information is stored in local storage.
I’m very open to criticism, constructive or otherwise; Thanks in advance!