Design comparison
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
- @Zy8712Posted 12 months ago
Your site looks pretty good. To add in the two circles to the background you need to use
background-image
andbackground-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 helpful1@GGSWEngineerPosted 12 months ago@Zy8712 this was just the comment I needed, thank you!
0 - @Mennatallah-HishamPosted 12 months ago
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 helpful0@GGSWEngineerPosted 12 months ago@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 GitHubJoin 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