@NicolasPirezGit
Latest solutions
responsive Age calculator app using tailwindcss and javascript
Submitted 23 days agoI know there are a lot of repeated blocks that I can simplify by using functions, I am still learning and I will be taking a look through this as I learn more and work on the improvements that I have to do.
Responsive newsletter signup form made with tailwindcss and some JS
#tailwind-css#accessibilitySubmitted about 1 year agoThe javascript code, I feel like there is a much more better way to make the email input get the email and the submit button not refreshing the page when clicked.
Responsive intro-section made with tailwindcss and javascript
#tailwind-cssSubmitted about 1 year agoHow to make the menus open one at a time, and have them close if anywhere on the page but the menu element itself is clicked on. I would also love to know how to get the shadow overlay when the side menu opens to work without it breaking the page, if you look at my solution I already attempted it with an empty div but that seems to break the code, any solution on how to go about that is welcome.
Time tracking app made with tailwindcss and javascript
#accessibility#tailwind-cssSubmitted over 2 years ago
Latest comments
- @KwakuAldo
Your site isn't responsive, it doesn't look good on mobile. It really looks good on desktop based on the screenshot comparison with the design. You should make the mobile design as the jpg in the design folder.
- @samlan24@KwakuAldo
Hello Sam, you need to put a hover state on the image. You can have a look at my solution here to help you with that https://www.frontendmentor.io/challenges/nft-preview-card-component-SbdUL_w0U/hub/nftpreviewcardcomponentmain-made-with-css-flexbox-QYjZ70xgz
Marked as helpful - @xeuxdev@KwakuAldo
-
Always try and use semantic html, instead of <div class="container"> use the html main element, then section the cards. Avoid using too many divs.
-
you should set a max-width rule for your body element to avoid it over stretching on wider screens, my suggestions is to use max width 1080-1280(anything between these numbers shd be okay for this challenge)
-
Also try and use the font-family that was specified.
Everything else looks good. Happy Coding! 💪🏿
Marked as helpful -
- @samlan24@KwakuAldo
You need to put some margin between the elements, particularly between your <p> tags to make the layout cleaner, also in mobile view the cards are to narrow, try adding some width to it fill up the screen a bit.
Marked as helpful - @Wantaiq@KwakuAldo
you need to set a max-width for the body to about 1440px, the element stretches too much that it sqews the design, plus you are leaving a lot of white space at the bottom of the section containers.
Other than that everything seems fine and dandy, happy coding 💪🏿
Marked as helpful - @dannygomes@KwakuAldo
Hello Danny, this looks great. The mobile design is almost pixel perfect. Well done.
