Skip to content
OverviewSolutions
11
Comments
12
P
Monika
@monika-webdeveloper

All comments

  • P
    Petar Rakocevic•320
    @PetarR21
    What are you most proud of, and what would you do differently next time?

    I learned how to implement form validation in React and provide feedback to users. I also practiced managing component state and conditional rendering for success messages.

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

    Hardest thing was writing a CSS media queries, for smaller screens.

    What specific areas of your project would you like help with?

    I would like to improve the code, so it is more robust and clean.

    Responsive Newsletter sign-up form with success message using React.js

    #react
    1
    P
    Monika•180
    @monika-webdeveloper

    Really good job! Congratulations! Your code looks clean and organized. I think you should put a little more effort to the CSS code.Your card is a little bigger than the design.Also I'm not sure about the hover effect. The design says it's active not hover. Also I think it's a good idea to add some box-shadow as it was shown in the design.

  • P
    Monika•180
    @monika-webdeveloper

    Good job! Your HTML code looks OK. But I think there are some problems with CSS. First of all the hight of the card is too big. The colour of the main heading is too dark. Also the active image is too small. There should be a padding added on both sides.It also doesn't look good on smaller screens so I don't think it's responsive.

  • P
    Paul Wonka•260
    @pdoubleu30
    What are you most proud of, and what would you do differently next time?

    I enjoyed this challenge even though I know there are flaws. I am hoping to get feedback so that next time I try this challenge I will improve.

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

    The biggest challenge I had was utilizing grid for the desktop version of the page. I was attempting to utilize online resources to get it just right but was unable to as of yet.

    What specific areas of your project would you like help with?

    I know my solution is not perfect so I am open to any and all feedback to improve. I am aware my code is likely too long so if I could get suggestions to simplify that would be appreciated. Also, if I could get tips on how I can improve the spacing more accurately on the header on desktop version using grid or flex box I would be grateful as well. Thank you!

    responsive page using css & html

    1
    P
    Monika•180
    @monika-webdeveloper

    Good job generally! Although it needs some improvements. Your solutions includes semantic HTML. I can see you have use different imagaes for different screen resolutions. But there are definitely some problems with padding and spacing. Also font weight and font size look a little bit different. Maybe you should read Figma more carefully.

    Marked as helpful
  • P
    Monika•180
    @monika-webdeveloper

    Good job! Your solution includes semantic HTML. However, there ara some CSS differences between your solution and the original design. In my opinion the size of the cards differes from the design. There might be a problem with the padidng or gaps between flex or grid items. And the mobile solution needs some improvements.

    Marked as helpful
  • P
    Monika•180
    @monika-webdeveloper

    Good job! Your landing page looks really good. Your solution does not differ considerably from the design and the layout looks good on a range of screen sizes. I think your HTML code is well-structured, readable and reusable. I think that you should pay more attention to padidng as there are some differences.

  • Ashraful Islam•100
    @ashrafitachi
    What challenges did you encounter, and how did you overcome them?
    • Had difficulties with the height of img in media queries.
    What specific areas of your project would you like help with?
    • Responsive CSS

    Responsive product card

    1
    P
    Monika•180
    @monika-webdeveloper

    It looks pretty good. Your solution doen not differ considerably from the design. But I think it still needs some improvements. The main title should be bold. The font of the paragraph is too dark. And it takes too many lines. The font may be too big. The price should be bold, too.There is also no hover affect on the button.

  • P
    Monika•180
    @monika-webdeveloper

    Good job! But there are some things that need improvements. Firsty, the background color is different (yours is too pink). The height of your card is too big and the paddings are different, too. That's why your solution differ from the design.Also on smaller devices the paddings are different. And probably the font is too small ( it's hard to read)

    Marked as helpful
  • P
    Monika•180
    @monika-webdeveloper

    Good job! Your site looks really good. Your HTML code is clear and simple. I think there must be some small mistakes in your css code. There maight be something wrong with the padding. The card padding is 40px and yours a little bit different. But generally it looks pretty good.And the image size is 88px not 90px. Some minor changes and it will look perfect.

  • P
    Monika•180
    @monika-webdeveloper

    Good job! But your solution slightly differ from the design. I think it's a bit too high. I also think that the HTML shoul be a little simple as there is a paragraph and then a button inside.Is it necessary? I'd also try to use classes more often and stick to lowercases.As sometimes the class name is uppercase, sometimes lowercase or there is no class name.

  • Wawowah•450
    @wawawoh
    What are you most proud of, and what would you do differently next time?

    i managed to create an accurate layout similar to the one displayed for all devices

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

    understanding githhub, git and mainly learning how to upload to frontend

    What specific areas of your project would you like help with?

    when on the page my solutions are quite large and span more than the viewport

    first qr code

    4
    P
    Monika•180
    @monika-webdeveloper

    Good job! But I think some improvements are necessary. First of all, you should change the background color, card color and place the card in the center. You should also pay attention to spacing. I think font color and font size should be different, as well. Don't forget about correct border radius and box-shadow.

  • Wawowah•450
    @wawawoh
    What are you most proud of, and what would you do differently next time?

    i managed to create an accurate layout similar to the one displayed for all devices

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

    understanding githhub, git and mainly learning how to upload to frontend

    What specific areas of your project would you like help with?

    when on the page my solutions are quite large and span more than the viewport

    first qr code

    4
    P
    Monika•180
    @monika-webdeveloper

    Good job! I think you should pay more attention to paddings and border radius. And change the background color and put the card in the centre.

  • Asilcan Toper•2,960
    @KapteynUniverse
    What specific areas of your project would you like help with?

    Any feedback is appreciated. Is there a way to change only the text color of the logo SVG, or is the dark mode logo missing? Also, is there a better way to fetch data, toggle states, and handle dark mode?

    Browser Extensions Manager UI

    #accessibility#react#tailwind-css#typescript#vite
    5
    P
    Monika•180
    @monika-webdeveloper

    I like your code. Good job!