All Comments

    • HTML
    • CSS
    • JS

    Bookmark landing page

    1
    Israel Adefidipe360 | Posted 25 days agocommented on Cristian Camilo Rodriguez's "Bookmark landing page" solution

    Great work @RodCri! but you have a lot of improvements, I think you could make:

    • You work isnt responsive on smaller screen
    • the faq accordion can be improved on by ensuring once any section of the accordion is opened others are closed, no two sections can be opened at the same time. I think it will make for better user experience
    • You should also work on the last FAQ section CheersπŸ±β€πŸ‘€
    0
    • HTML
    • CSS
    • JS

    responsive bookmark web using react JS and material ui

    2
    Israel Adefidipe360 | Posted 25 days agocommented on Pham Tuan Anh's "Bookmark landing page" solution

    Great work @akalizk113! Just two improvements, I think you could make:

    • the placements of the page illustrations is not according to the design specification, or it might be my browser. Your hero section appears not to be not flexed in the desktop view, but the solution screenshot it looks flexed. Maybe you should check your work across different browsers
    • the faq accordion can be improved on by ensuring once any section of the accordion is opened others are closed, no two sections can be opened at the same time. I think it will make for better user experience. Other than that, your implementation appears to be perfect. CheersπŸ±β€πŸ‘€
    1
    • HTML
    • CSS
    • JS

    Responsive landing page using tailwind

    4
    Israel Adefidipe360 | Posted 27 days agocommented on Andres Jaramillo's "Bookmark landing page" solution

    hi @GhostmaneUrs, your implementation is great. The page overflows on smaller views, you might want to check that, by probably giving the page a max-width. Not really important, but the buttons styles should also have transition time

    0
    • HTML
    • CSS
    • JS
    • API

    Vite.js, React, Styled-Components

    3
    Israel Adefidipe360 | Posted 27 days agocommented on Lynden's "REST Countries API with color theme switcher" solution

    hi @MartinLundqvist great implementation. Your project only seems to have spacing issues, i feel you can do better with the spacing in the desktop view. The cards are not centered in the mobile view and the header also has some padding issues

    0
    • HTML
    • CSS

    Stats preview card component - SASS

    2
    Israel Adefidipe360 | Posted 5 months agocommented on Tom Peirs's "Stats preview card component" solution

    Great job Tom, you might want to work on your font sizes on smaller screens, other than that, you did a very nice job. Also check out the issues flagged by the platform. Keep Learning! cheers

    0
    • HTML
    • CSS

    Stats preview component solution

    10
    Israel Adefidipe360 | Posted 5 months agocommented on dewslyse's "Stats preview card component" solution

    Great job, your implementation is almost pixel perfect. The platform flagged some accessibility issues, you should check that out. cheers

    0
    • HTML
    • CSS

    Front-End-Mentor-Stats-preview

    2
    Israel Adefidipe360 | Posted 6 months agocommented on isaac's "Stats preview card component" solution

    Great job.

    • Instead of using ID which should be unique for each element. Use classes, it makes your styling more easy to, and you can use a class for more than one element too

    • the whole card is not well centered on all screens.

    • you should have main container, so you can center the body element. You can also put the text and image section in the container and flex them. See if that helps.

    0
    • HTML
    • CSS

    simple HTML and CSS

    2
    Israel Adefidipe360 | Posted 6 months agocommented on Krishna's "3-column preview card component" solution

    Looks, great on all screens. But since you have got a style sheet, you dont need the inline styles

    1
    • HTML
    • CSS

    Profile card component with HTML & CSS

    3
    Israel Adefidipe360 | Posted 8 months agocommented on YannikS14's "Profile card component" solution

    You can create media queries for more screen widths and style/position the background for them. Start the mobile background image from the 768px screen width and position it for that width, keep the mobile screen size position. you can position the desktop background Size for 1024px screen width too.

    Your implementation is great, keep coding!

    0