Looking to hire developers?
@madegilangaditya
@FelipeGuerra5
Submitted
This component is made in React, using ViteJS, using the:
useState to manage changes and variables useEffect and axios to fetch the API LocalStorage so the user does not have to go through all the advices every time the browser is refreshed. Css as Module not quite necessary for this project, but good practice for scalable projects.
Any suggestions are Welcome!
@madegilangaditya
Posted
Hi, Felipe great work on finishing this challenge. The solutions just look great. Just a little correction for me regarding the responsive view, in this class ._card_qel2f_21 which is the container of your advice generator rather than use width: 320px; better use max-width:320px so when the screen is below 320px, the container will not pass through the screen.
Also, try learning mobile-first workflow standard for website design. You can check the reference here: https://www.w3schools.com/css/css_rwd_mediaqueries.asp
Good job, keep it up 👍
@devwinner-sek
Submitted
Hello. For this challenge, please :
@madegilangaditya
Posted
Hi, congratulation for finish your challenge, thats really good from my side. Here are some suggestion for me that I found on your solutions.
I think thats it from me hope can help.
@pilatech
Submitted
Hey there!
I'm realising that I don't have a uniform way of creating dropdown menus. However, toggling classes to show or hide a dropdown menu seem uniform in my workflow. The question is: What's the "best" set of styles can be swapped to create a more animated/smooth toggling? display: none and display: block seem to not work well
@madegilangaditya
Posted
Hi, congratulation for finishing the challenge, my suggestion for the dropdown menus, maybe try mix visibility and opacity on it instead using display, for the mobile can manipulate max-height and visibility. Hope can give you insight on it
@Deepthi-Ramesh
Submitted
Hello everyone ! Here is the social-proof-section-master web design which is totally responsive using HTML & CSS .I am open to take up any suggestions . I am not able to fix the background images as it was difficult to recognize in the design. Thank you
@madegilangaditya
Posted
Well done for finishing the challenge, regarding your issue about backround image, I think you need to wrap your container with div then you can add background image in the wrapper div. Here some source to style background image : https://developer.mozilla.org/en-US/docs/Web/CSS/background
@Alatmal
Submitted
Was really hard for me. If you can give me some advice to make it better in CSS I will appreciate it forever.
@madegilangaditya
Posted
Well done for completing this challenge man, just a couple things I can suggest you
@MSingh-web-dev
Submitted
This is my first front-end mentor challenge that I have completed. Please take time aand provide your feedback.
@madegilangaditya
Posted
Good job for finishing your first challenge. There are some issue I found regarding your border radius not show in bigger screen on container class. Instead using width 80vw. Better use max-width: ...px because vw size is relative depending on your browser screen size, so that make your container is more bigger than the content itself when use display:flex that make border-radius not showing in bigger screen.
@nirban256
Submitted
Made my first responsive design using HTML and CSS and using media queries
@madegilangaditya
Posted
Nice job for finishing the challenge. Maybe try use display:grid instead of using transform:translate. Also the border top of every column need to be diferent color like the design.
@MurisKamaa
Submitted
@madegilangaditya
Posted
Good job for your first challenge, Here are a couple issue that I see on your project.
@sergioreynoso
Submitted
@madegilangaditya
Posted
Nice work man..
@foolhardy21
Submitted
I have made the changes mentioned below in comments. Feel free to critique further. Thanks.
@madegilangaditya
Posted
Hi Vinay, Congratulations for finish this project, there an issue in your background image. you need background-repeat: no-repeat; so your image will not repeated.
@Moh2106
Submitted
HI every comment is welcome .
@madegilangaditya
Posted
Hi Moh, congratulations for finished the challenge. Here are some suggestions for you:
justify-content: center; in your flexbox to make your column align center.@RachelOpuba
Submitted
I have just completed my first challenge on this platform. Kindly review and let me know areas I can improve on, as I am a complete beginer. Thank you.
@madegilangaditya
Posted
Hi, well done for your first challenge, I have little suggestion for your container border radius. Instead of put it on every div, better put in main class div and add overflow:hidden so your image will not overlap your main div border radius
