Skip to content
OverviewSolutions
11
Comments
8
Rita
@MargaritaBusyginaCa

All comments

  • Rita•355
    @MargaritaBusyginaCaPosted 14 days ago

    looks good! I would just add a cursor: pointer to the navbar elements

    Marked as helpful
    0
  • clickglue•460
    @clickglueSubmitted 20 days ago
    What are you most proud of, and what would you do differently next time?

    I was able to build this app reasonably fast, and I think a bit more precise than previous challenges.

    What challenges did you encounter, and how did you overcome them?

    Styling sliders and checkboxes proved to be far more complicated then I thought. With the help of ChatGPT I could solve it.

    Password generator

    1
    Rita•355
    @MargaritaBusyginaCaPosted 14 days ago

    For the styling, you might want to increase the padding for your form. The spacing between password strength (for example, "Strong") and the strength boxes need to be smaller - you can try using flex on that container and then adding gap to it.

    0
  • Rita•355
    @MargaritaBusyginaCaPosted 14 days ago

    Hi, it seems like the checkbox values don't work properly. When I try to select all the checkboxes, the generated password doesn't include numbers or symbols.

    1
  • mostafa-ahmed-soror•20
    @mostafa-ahmed-sororSubmitted 2 months ago
    What are you most proud of, and what would you do differently next time?

    iam proud that i started doing what i like .. i will try to do it faster now

    What challenges did you encounter, and how did you overcome them?

    to create a table showing nutrition

    recipe page using HTML & CSS

    2
    Rita•355
    @MargaritaBusyginaCaPosted about 2 months ago

    It looks very similar, but you are using the wrong font. Also, you need to add a responsive design (for example, using media queries) so it looks good on smaller screens as well.

    0
  • Rita•355
    @MargaritaBusyginaCaPosted over 2 years ago

    Good job! I noticed, however, that the background image doesn't take the whole screen and ends in the middle of the screen(especially on big desktop screens). I think you should try setting background-image properties like that :

    background-size: contain; background-position: 120% 120%; background-repeat: no-repeat;

    For the mobile version, your containers are not centred, they stick to the left of the screen. To fix that you can use either use:

    1)margin: 0 auto;

    2)or flex justify-content:center

    Marked as helpful
    0
  • Rita•355
    @MargaritaBusyginaCaPosted over 2 years ago

    Looks great! I would only suggest making your main containers bigger, so it occupies more space on the screen. You can implement it by adding padding to the inside components (for example, you can increase the padding of "Total this month $478.33") I think if you have one component on the page, you can afford to make it bigger, so it doesn't look too empty. Overall, good job!

    Marked as helpful
    1
  • Rita•355
    @MargaritaBusyginaCaPosted over 2 years ago

    Looks good! I think you might want to check the width/max-width of the containers as they are a little wider than the original. Also, the progress bar looks very nice, but you might want to add a border to it. I checked your code and noticed that you have a file called "Normaliz.css", could you please tell me what you used it for, I am just genuinely interested cause I've never seen anything like this before.

    0
  • Rita•355
    @MargaritaBusyginaCaPosted about 3 years ago

    Good job, Ahmed! I like your version even better

    Marked as helpful
    0