All solutions

Hi everyone! I'd like some feedback specifically on these things: -I was having trouble with the responsiveness of the hero image switching between mobile and larger screen layouts. You can see that right at the breakpoint, the hero image for the larger layout is quite small and not ideal. Is there a way to make the layout look better for all screen sizes? Do I need to adjust the breakpoint, or are there other ways to style the hero image to make it more responsive? -This was my first time doing form validation with JS, so please let me know if it looks okay!

Any other feedback is welcome as well. Thanks!

This is my 5th Frontend Mentor challenge! It's my first time using CSS Grid, so this was a good learning experience for me.

I'd like specific feedback on a couple things:

1. I've used Grid on the reviews, ratings, and overall layout on the desktop view. At the 1440px view, the spacing between the ratings in the upper right is larger compared to the design. Does anyone know how I can get the spacing closer to the design?

2. The background decorations were giving me some trouble when trying to make them responsive. The mobile decorations don't expand when the viewport width increases, which looks fine for the top decoration but the bottom decoration has a clear left edge that doesn't look good if the width isn't 100%. In another version I tried using background-size: 100% so the background decorations would scale with viewport size up until the desktop breakpoint, but they didn't look good that way either. What would be a better solution?

Of course any other feedback is welcome. Thanks!

Hi all! This is my 4th challenge. I definitely had some trouble with the circle patterns in the background, so if there's anything about my approach there that could be improved upon that would be great to know! I was also having trouble with the responsiveness of the background design and having a media query to adjust the positioning of the circles feels inelegant.

I'm also unsure about semantics of component cards like this. Should the name of this profile card be an <h1>, or would it make more sense for there to be a screenreader-only <h1> and have the name be an <h2>? Should the sub-labels for the stats and location be something other than <span>?

Thanks!

Hi all! I'd like some feedback on CSS layout and responsiveness. I'm also not sure about the accessibility of hiding the <h1> element and how best to do this. I took a guess and thought that I should hide it in this case, since it didn't seem like it would make sense for each of the headings of the columns to be <h1>. Thanks!

This is my second finished challenge. I'd specifically like feedback on accessibility & HTML semantics and responsiveness, but am open to any feedback. I'm still trying to get comfortable with layout and spacing, so if you see any problems with my CSS regarding those aspects please let me know too. Thanks!

Hi all! I'd like specific feedback on code structure and semantics, as well as any feedback on the CSS I've used for layout, spacing, and responsiveness. Is there a way to increase responsiveness? Thanks!