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

Order Summary Component with HTML, CSS, & Flexbox

Aliaa 25

@aig0041

Desktop design screenshot for the Order summary component coding challenge

This is a solution for...

  • HTML
  • CSS
1newbie
View challenge

Design comparison


SolutionDesign

Solution retrospective


Hey everyone! I just completed this project and I know my code is really sloppy and I'm sure it could have been written in a more organized way, I just believe that the best way for me to learn is to dive in and apply anything that I know, so that's what I did.

This is the first time that I use flexbox in a project and I'm still not really sure I understand it fully, so I'd really appreciate if you point out any mistakes I have with it specifically.

Again, I'm only a beginner and I know there is so much I don't know yet, so any suggestions or tips to improve the code overall would be really appreciated. Thanks!

Community feedback

@elidrissidev

Posted

Amazing work Aliaa! Not bad for a first challenge. Keep it up!

For the images problem, it's because of the URL you're using:

  • http://127.0.0.1:5500 is a URL only accessible on your machine when you were running the http server (Using Live Server Vscode extension?).
  • file:///Users/... is a URL used to access files on your local filesystem (Mac in this case) and is again specific to your system and won't work elsewhere.

So the fix for this is to use a relative URL, in your case it's gonna be ./images/<image name>.

As for the rest I only have 3 suggestions:

  • Every html page should have a level 1 heading h1 that describes the main content of the page.
  • Consider using semantic html elements instead of generic divs where appropriate, for example the "Cancel" text can be inside a a or a button element because it conveys that it can interacted with.
  • Check the accessibility report page for more.

Good luck to you!

Marked as helpful

1

Aliaa 25

@aig0041

Posted

@elidrissidev Oh yes I was using the live server extension. I didn't realize what the correct URL was though, but i changed it now and it works so thank you so much! I also really appreciate all the suggestions; I'll definitely keep them in mind.

0

@elidrissidev

Posted

@aig0041 you're welcome!

0

@mdagudo

Posted

Hi! Good job. I noticed you used:

background-image: url(file:///Users/aliaagabllah/Desktop/Projects/Order_Summary_Component%20_Project/images/illustration-hero.svg);

in your CSS. That's why you can see it in your computer, but we can see them in the github page. You need to use:

background-image: url("./images/illustration-hero.svg");

Because you uploaded the images folder and the .svg to your repo.

Keep coding!

Marked as helpful

1

Aliaa 25

@aig0041

Posted

@mdagudo I changed it and it worked. Thank you so much!

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