Skip to content
OverviewSolutions
6
Comments
7
rita
@ritadoumet

All comments

  • rita•280
    @ritadoumetPosted over 2 years ago

    Amazing work! You imitated the design perfectly, as well as made it look nice on small & big screens. Your code looks well structured, especially the CSS classes. Kudos to you!

    Marked as helpful
    0
  • ronaldlamdev•430
    @ronaldlamdevSubmitted over 2 years ago

    I'm unsure on why there's large space in between the gaps of my columns in my footer section. Is there a better way to reduce the gap in those columns rather than set a fixed width for the columns? Is it alright to create a different stylesheet for the media queries, so I don't have to include them all in one styles.css? Any constructive feedback is very valuable. Took me a while to complete, but I had fun coding and learning in the process.

    fylo landing page using html, css, and vanilla javascript

    1
    rita•280
    @ritadoumetPosted over 2 years ago

    Your website looks great! You definitely matched the style to a high degree. Kudos to you! However, I think you should optimize the website for bigger screens, as it looks stretched out on a desktop screen. Also, some buttons, when hovered over, do not look clickable. You should just set the cursor to a pointer. Good job for using semantic HTML and reusable JavaScript functions. Keep up the good work! :)

    Marked as helpful
    1
  • rita•280
    @ritadoumetPosted over 2 years ago

    That's a good start on this website! The design is very close to the requirements, you nailed this! However, there are a few points that you might want to improve: 1- Responsiveness. Your website looks great on a phone screen, but it looks stretched out on a desktop screen. Try optimizing your CSS and making your website responsive. 2- Make it that only one number is selectable in the feedback section. Other than that, good job!

    Marked as helpful
    1
  • rita•280
    @ritadoumetPosted over 2 years ago

    Your solution looks great! All of the components look in place, and it seems like you have imitated the design closely. It's also great that the design works for both small and large screen sizes. There is one simple thing that needs to be fixed: the register button and the social media icons. When you hover over them, the cursor should change to a pointer. And the font of the button looks off. But other than those small details, I say chapeau bas!

    Marked as helpful
    1
  • rita•280
    @ritadoumetPosted over 2 years ago

    Your work looks fantastic! I tried it on a range of different screen sizes (laptop, phone, and tablet), and it looks well on both screen sizes. Besides, your code is easily readable and understandable. It helped me (someone who knows next to nothing about APIs) to understand and get started with APIs. So thank you, kind stranger!

    0
  • rita•280
    @ritadoumetPosted about 3 years ago

    Nice solution! There is just one small issue : the result overflows from the screen if the number is too big. Other than that, I believe your solution is great. Happy coding :)

    Marked as helpful
    1
  • rita•280
    @ritadoumetPosted about 3 years ago

    Kudos to you! Your solution is very simple and elegant. I found it smart that you minimized the use of JavaScript and solved the problems using mainly CSS. Good job :)

    Marked as helpful
    1