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

All comments

  • Aliaa 25

    @aig0041

    Submitted

    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!

    @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
  • @tesla-ambassador

    Submitted

    It's been a while (4 months) since I did some web programming. I took on this challenge to boot up. It's kind of rusty but help me out wherever you can! Thank you!

    @mdagudo

    Posted

    Hey! Thanks, I was struggling with the input and the button and your code helped me a lot :)

    1
  • @mdagudo

    Posted

    Good job!!! It looks really similar to the original, that's great :)

    You should check those accessibility and HTML issues!

    Keep coding!

    Marked as helpful

    0
  • @Johan-Jubin

    Submitted

    Made my first project in Frontend Mentor, I had 2 problems:

    1. I can't position the background circles in their respective positions.
    2. I can't add a white circle behind the profile picture.

    All your valuable feedbacks are always welcome. Please review my code & suggest ways to tackle the problems

    @mdagudo

    Posted

    Hello! You can also add the background images in the css, you just have to add the two images as the background-image of the body, then make sure it won't repeat and finally position them simultaneously with background-position. For the "circle behind the profile picture", you can add border to that picture :) Keep coding!

    1
  • Ananth 50

    @Ananthkumaran

    Submitted

    Reviews will be appreciated, I was not able to add the background images on both mobile version as well as desktop version, as whenever I was adding the images I was not able to align the text over it and I was not able to position the background images to the top and bottom corner of the webpage, If someone can help me with this it would be great help.

    @mdagudo

    Posted

    That background images gave me headaches as well. I just submit my solution to this challenge and being able to see those images were really difficult. I even had to change the colors resolution of my screen! You can align those images with the property background-position but using units such as vw (viewport-width for left and right) and vh (viewport height for top and bottom) because they're relative to whatever viewport you have, even if you resize the browser. I recommend you to use mozilla dev tools so you can try values until you see something :)

    0
  • @mdagudo

    Posted

    :o the width and height of the container are on pointttttttttt

    1
  • @mdagudo

    Posted

    This looks perfect!! Keep coding :)

    0
  • @mdagudo

    Posted

    This looks great! Keep coding :) Try to read about Responsive Design because your container shrinks when I resize the browser window...

    1
  • @mdagudo

    Posted

    Good job! Keep coding :)

    0
  • @mdagudo

    Posted

    Great job :) If you'd like to make it look more similar to the original, you could adjust the height and width of your container, and making smaller the padding of div.cont2. Also the rounded border of the container. Keep coding!

    0