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

Profile-Card-Component-Solution

P

@GGSWEngineer

Desktop design screenshot for the Profile card component coding challenge

This is a solution for...

  • HTML
  • CSS
1newbie
View challenge

Design comparison


SolutionDesign

Solution retrospective


Hello everyone!

In this project, I started to use specific units for properties and I feel like I'm improving. Still experimenting with width percentages and max-widths using pixels. If anyone can give me some pointers on width using percentages, I would appreciate it!

Also, adding the two big circles is a bit of a challenge for me. When I tried using position relative or absolute, it still affected all of the other elements in my HTML. Does anyone have tips on how to easily add these two circles to my background?

Thank you for your time,

Gerardo Garcia

Community feedback

Bryan Li 3,550

@Zy8712

Posted

Your site looks pretty good. To add in the two circles to the background you need to use background-image and background-position. Here is what I did:

  background-color: var(--theme-dark-cyan);
  background-image: url("./images/bg-pattern-top.svg"), url("./images/bg-pattern-bottom.svg");
  background-repeat: no-repeat, no-repeat;
  background-position: right 52vw bottom 35vh, left 48vw top 52vh;
  /* top img is offset 52 percent of the vw from the right and offset 35 of the vh from the bottom*/
  /* bottom img is offset 48 percent of the vw from the left and offset 52 of the vh from the top*/

Hope you find this useful 👍

Marked as helpful

1

P

@GGSWEngineer

Posted

@Zy8712 this was just the comment I needed, thank you!

0

@Mennatallah-Hisham

Posted

Hi Gerardo,

I can see your effort, You did great 👍

Here are some suggestions to improve your code:

Headings

  • each page should have one h1 tag,h1 helps the web understand the content. also skipping h1 may confuse assistive technology users.
 <h1>Victor Crest</h1>

ALT

  • images should have descriptive alt text, which is important for SEO and screen reader users
<img src="" alt=" Headshot of Victor Crest"

Accessibility

  • you can use aria-labelledby to define a name/relation between the number and their titles for screen readers
<p aria-labelledby="title">80k <p>
<p id="title"> follower </p>

SEO

  • you can add meta description for better SEO, it provides a brief summary of a web page
<meta name="description" content="..........."/>  

here are some helpful articles:

hope you find this helpful. Happy Coding

Marked as helpful

0

P

@GGSWEngineer

Posted

@Mennatallah-Hisham hi! thank you for your comment, made the changes and its looking much better!

0

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