Skip to content
OverviewSolutions
11
Comments
20
Divine Ugorji
@DivineUgorji

All comments

  • Divine Ugorji•350
    @DivineUgorjiPosted 10 months ago

    Hello there 👋, congratulations on completing this project ✅. However, I found that your live URL for viewing the project is broken. Please check and fix it 🙏.

    0
  • Divine Ugorji•350
    @DivineUgorjiPosted about 1 year ago

    Hello, @AvoscanEmile 👋 congratulations on completing this challenge 🙌

    Your solution is nice and thoughtfully implemented with simple animation effects ✅

    Your HTML code is semantic and your project is responsive across multiple screen sizes Your code base is well-structured and readable.

    Congratulations, and keep up the good work.

    Happy coding 🙏

    1
  • Julie•150
    @jclegg31Submitted about 1 year ago

    Had a tough time with the background lining up and the size of the desktop version overall. Not sure how to build without the figma files, it's just really a guess? Do you all have the same struggles? I think it looks ok but felt a little wobbly building it vs the component challenges.

    Would love any feedback or suggestions. Thanks!

    Huddle Landing Page - HTML/CSS

    1
    Divine Ugorji•350
    @DivineUgorjiPosted about 1 year ago

    Hello, @jclegg31 this challenge is a little tricky even though it looks simple, and congratulations on completing it 🙌

    I can relate to all the struggles you mentioned on a personal level 😊 and it can be overwhelming at times, which is part of what informed my decision to get the premium plan and get the Figma files for the challenges. Even at that, it takes a lot to make pixel-perfect designs.

    You can check out this Chrome extension and learn how to use it if you are very much interested in pixel perfection.

    From what I can see, you are doing absolutely well 👍

    • Your solution to this project is nice and thoughtfully implemented ✅
    • Your solution is responsive across multiple screen sizes
    • Your code is semantic, well-structured, and readable.

    Keep up the good work, and may the force be with you.

    All the best, and Happy coding 🙏

    0
  • Divine Ugorji•350
    @DivineUgorjiPosted about 1 year ago

    Hello @tsh22 👋 congratulations on completing this challenge 🙌

    Your solution is nice and thoughtfully implemented ✅

    • Your HTML code is semantic, however, you overused the div container. Most of the contents of the statistics could be wrapped in paragraphs and list elements accordingly, so you might want to pay attention to that in your next projects.
    • Your solution is responsive across multiple screen sizes
    • Your code is well-structured and readable.

    Once again, nice implementation, and keep up the good work.

    Happy coding 🙏

    Marked as helpful
    1
  • Divine Ugorji•350
    @DivineUgorjiPosted about 1 year ago

    Hello@ZainabProgrammer 👋 congratulations on completing this challenge 🙌

    Your solution is nice and thoughtfully implemented ✅

    • It's responsive across multiple screen sizes 💻

    • Your code is semantic, well-structured, and readable. 💯

    I really love your design implementation, and the API you used gives good details of the words searched. In fact, I'm bookmarking this project so I can reference the API later. 🚀

    However, I have a few suggestions. The home icon is just decorative, I think it would be a much better use case if it is used to refresh the page for a fresh word search. Also, the search functionality is triggered only when the search icon is clicked, I think the search functionality should also work when the Enter key is pressed after inputting the search words.

    The application would have a much better user experience if these issues were corrected.

    Overall, your solution rocks, and has excellent implementation.

    Happy coding! 🙏

    0
  • Divine Ugorji•350
    @DivineUgorjiPosted about 1 year ago

    Hello, @Upond 👋 congratulations on completing this challenge 🙌

    Your solution is nice and well implemented ✅

    • Your HTML code is semantic

    • Your code is well-structured and readable.

    • Your solution is responsive across multiple screen sizes, however, I found that the intro contents begin to overlap for a screen size lower than 300px. I know this is an edge case, and there are not many devices on such screen sizes, however, it would be nice to create some spaces for the intro contents for devices in that screen size range.

    overall, your solution rocks and I like it. 👍

    Happy coding 🙏

    Marked as helpful
    0
  • Divine Ugorji•350
    @DivineUgorjiPosted about 1 year ago

    Hello there 👋 and congratulations on completing this challenge 🙌

    Your solution is nice and thoughtfully implemented ✅

    • Your HTML code is semantic
    • The solution is responsive across multiple screen sizes
    • You made your code well-structured and readable.

    Congratulations, and keep up the good work.

    Happy coding 🙏

    Marked as helpful
    1
  • Divine Ugorji•350
    @DivineUgorjiPosted about 1 year ago

    Hello @@girldocode, Congratulations on completing and submitting this challenge. I really love your design implementation,

    -The project is responsive and nice, especially the way you used Grid properties to achieve seamless responsiveness without the excessive use of media query. -Your code is well-structured and easy to read and understand, however, I'd suggest that you check out BEM for a more semantic CSS naming convention.
    -You can use a media query to remove the excessive border-radius in the mobile view of your project.

    Great work all the same, and I hope that my suggestions will help to make your work even better.

    All the best 🙏

    Marked as helpful
    1
  • Om Prabhat•30
    @om-prabhatSubmitted about 1 year ago

    Frontend Mentor - Order summary card solution

    This is a solution to the Order summary card challenge on Frontend Mentor. Frontend Mentor challenges help you improve your coding skills by building realistic projects.

    Table of contents

    Overview

    The challenge

    Users should be able to:

    • See hover states for interactive elements

    Screenshot

    Links

    My process

    Built with

    • Semantic HTML5 markup

    • CSS custom properties

    • Flexbox

    • CSS Grid

    Author

    Responsive page using flexbox and grid

    1
    Divine Ugorji•350
    @DivineUgorjiPosted about 1 year ago

    Hello @om__prabhat, congratulations on completing and submitting this challenge. Your implementation looks really nice but I could not help but notice that your background image is not well positioned. Apply the following properties to your code:

      background-color: hsl(224, 100%, 94%);
  background-image: url(image-url);
  background-position: top;
  background-repeat: no-repeat;
    • I would also suggest the use of BEM methodology for your CSS naming conventions.
    • I observed the overuse of the pixel unit in your code base. It's always good practice to learn how to use relative units in your projects as it helps with responsiveness. You can check out this blog to learn more about this.

    overall, your implementation is very nice, and I hope these few suggestions of mine will help you to improve it better.

    All the best 🙏

    Marked as helpful
    1
  • Divine Ugorji•350
    @DivineUgorjiPosted about 1 year ago

    Hello there, congratulations on completing this challenge. Your solution is nice and responsive and your code is well structured. However, I'd like to suggest that you use more semantic naming conventions for your classes, elements, and containers. This helps in making meaning of your codebase and helps in the future maintenance of your code. E.g; for this particular project, you can use a Sedan container, Suv container, and Luxury container instead of col 1, col 2, and col 3. I think you get the point now. You can check out the BEM methodology for naming conventions in your projects.

    Overall, lovely implementation. Happy coding!

    Marked as helpful
    0
  • Karol•1,620
    @karolbanatSubmitted about 1 year ago

    Hi. After a long time I post this solution. It took me a little bit longer due to the things happening in my life, but I will finally post it. Started it with testing, but later abandoned it. Worked on it a little each week. It will certainly have some bugs and issues, so if you find one let me know. Feedbacks are welcome. Have a nice day.

    Dictionary Web App

    2
    Divine Ugorji•350
    @DivineUgorjiPosted about 1 year ago

    Hello there, congratulations on completing this challenge.

    I really love your design implementation, it looks really lovely. However, the home icon gives a 404-page error, and the theme toggle switch is not working. The solution would be much better if you correct those issues.

    once again, excellent implementation. Happy coding!

    Marked as helpful
    1
  • Reginaldo Junior•20
    @RsajrSubmitted about 1 year ago

    I found difficulty in manipulated the boxes. Probable in "

    QR-Code

    1
    Divine Ugorji•350
    @DivineUgorjiPosted about 1 year ago

    Hello there, nice effort in making this project. However, I'd like to point out that you did not add the GitHub link to this project while submitting your solution. Always deploy your projects to GitHub, and share the link here, it helps the community to give you better feedback on your projects and code.

    Hope my suggestion helps. Happy coding!🙏

    Marked as helpful
    0