Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found

Submitted

Social profile links

P
Roberts P 120

@indaqoo

Desktop design screenshot for the Social links profile coding challenge

This is a solution for...

  • HTML
  • CSS
1newbie
View challenge

Design comparison


SolutionDesign

Community feedback

@damigand

Posted

It's nearly pixel perfect, this is great! Personally though, I think the main element should be the .card itself. You can just use the CSS property justify-content: center in the body given that it's already a flexbox element. Also, I don't think the CSS query for smaller screens is necessary. You can combine width: 100% and max-width: 327px on the .card element to make it responsive without using the query. But these are just persnickety things though.

Love to see the work!

1

P
Roberts P 120

@indaqoo

Posted

@damigand,

To address your points:

  1. I understand that the main tag is typically used for the main content, which, in this case, is a card. However, I wrote it this way to allow for the possibility of having multiple cards in the main section. This approach will make reusability of styles if we need to add more than one card.

  2. I generally follow a component-based perspective in my projects, which helps in maintaining a modular and scalable codebase.

  3. There is a note in the example readme.md file regarding CSS for smaller screens: "Ensure content is responsive and meets WCAG requirements by testing the full range of screen sizes from 320px to large screens." This is because some users still have smaller phone screens. I consistently use a mobile-first approach in all of my projects to ensure responsiveness and accessibility.

Thank you for taking the time to review my work.

1

@damigand

Posted

@RobertsPeirags

I can understand the first two points, they're pretty understandable. About the third point though, the solution I suggested (with very little testing on your live site) makes your card accessible even down to 220px of width (with the slight adjustment of text-wrap: wrap on the bio), and saves you time in the rest of the queries that you have established for other screen widths. Obviously this is my subjective point of view, neither of us are wrong (or so I'd like to think, sometimes I am wrong :P)

1
P
Roberts P 120

@indaqoo

Posted

@damigand While you can make screen size lover than 320 pixels... 320px is generally considered the minimum width for responsive web design, as it covers most modern smartphones.

But again, I might be wrong on this.

1

@damigand

Posted

@RobertsPeirags Yeah, there's no objective way to determine the right way of doing things. Hey, as long as it works! Love to see your work!

Cheers!

1
bobyanov 100

@bobyanov

Posted

perfect

1

Please log in to post a comment

Log in with GitHub
Discord logo

Join our Discord community

Join thousands of Frontend Mentor community members taking the challenges, sharing resources, helping each other, and chatting about all things front-end!

Join our Discord