I'm Currently learning CSS and i'm open to your valuable suggestions to improve my solution.
Shaswat Singh
@shaswatsingh19All comments
- @SindhujaBandaruSubmitted about 2 years ago@shaswatsingh19Posted about 2 years ago
Hi, I think you could provide some interactivity to button while they been active and as Cancel button is also works like a button it should have like cursor : pointer as it looks good like that.
0 - @correlucasSubmitted over 2 years ago
š¾ Hello, Frontend Mentor coding community. This is my solution to the Four Card Feature Section.
š I had a lots of fun doing the challenge and doing some custom design improvements.
This is a solution that I've finished before but I decided to update it, when I saw a solution of guy Yefry Sanchez where he applied an hover effect on the card where the icon flips while hovered, so I tried to apply something similar to mine inspired on what he did. Here's is his profile @y25sanchez . Nice solution bro!
š» I added some features š
- šØ Text Gradient on main heading.
- š„ Glassmorphism on card hover.
- šØ Custom background.
Feel free to leave any feedback.
@shaswatsingh19Posted over 2 years agoAnother one of your great art I would say. Can I use your design to build the project if you don't mind?
1 - @correlucasSubmitted over 2 years ago
š¾ Hello, Frontend Mentor coding community. This is my solution for the Order Summary Component.
Feel free to leave any feedback and help me improve my solution or make the code clean!
I added some features:
- šØ Custom Wave Background
- šØāš¬ Custom Hover Effect on card elements
- š½ Card Banner Hover Zoom
I'll be happy to hear any feedback and advice!
@shaswatsingh19Posted over 2 years agoHey Lucas, how did you do the initial animation of main element when we reload the page, any resources for that ?
1 - @priynshuchouhnSubmitted over 2 years ago@shaswatsingh19Posted over 2 years ago
Hi, great work on completing the project. There is some issue with the design when we go below 350px width your layout breaks I would remove the max-width and use only width. As in container max-width saying that till 25rem it's fine with me but below it, it doesn't know what to do and breaks. Also, reduce the margin top and bottom from 200px to somewhat in the 100px range
0 - @ManviS1Submitted over 2 years ago@shaswatsingh19Posted over 2 years ago
You should see the problem , solve it then upload the solution
Marked as helpful1 - @jvmcpheronSubmitted over 2 years ago
This was more of a casual lil coding project I did on the side since I had a little extra time. Let me know what you think, and as always, if you have any advice, please share! I'm happy to hear.
@shaswatsingh19Posted over 2 years agoHey Jane, congratulations on completing the project
0 - @catherineisonlineSubmitted over 2 years ago
Hello, Frontend Mentor community! This is my solution to the Fylo landing page with two column layout.
I appreciate all the feedback you left that helped me to improve this project. Due to the fact that I published this project very long ago, I am no longer updating it and changing its status to Public Archive on my Github.
You are free to download or use the code for reference in your projects, but I no longer update it or accept any feedback.
Thank you
@shaswatsingh19Posted over 2 years agoCongratulation on completing the project, the SVG inside the linked info class would be 1rem
0 - @Rhenriquez94Submitted over 2 years ago@shaswatsingh19Posted over 2 years ago
Hi, Congratulations on completing this project
Everything is looking fine somethings I have noticed
- For smaller screens you should remove the margin of 130px from the image
- Background size should be cover and position should be center and it's look amazing
Marked as helpful1 - @mahadaamir35Submitted over 2 years ago
This design is responsive. Please do give feedback on how the code/markup can be improved. I will appreciate any kind of feedback.
@shaswatsingh19Posted over 2 years agoNice work, just that you should reduce your whole container width so that it will be perfect as the design. You also did mobile first design which is mostly preferred and that's great. You can watch Kevin Powell's video on width and max-width it is really great resource. I saw your code and you used both images inside the img div and put one to none and other one to display block. I would suggest to use both images as background-image and get them as url() it reduces the code and you didn't have to display block or none the the csss
Marked as helpful0