@oseji
Submitted
How can i implement the side navigation bar on mobile screens? Also how could i have included a transition animation to the dropdown menu for FEATURES and COMPANY?
@abhik-b
@oseji
Submitted
How can i implement the side navigation bar on mobile screens? Also how could i have included a transition animation to the dropdown menu for FEATURES and COMPANY?
@abhik-b
Posted
Hello Ose , Your solution looks great on large screen & nice use of tailwind , well done ๐ฏ
Just some opinions :
.invisible-container
this position:absolute
so that main does not shift when you click on a dropdown link.hidden
class , give some transform:translateY(20px);opacity:0;
to hidden class. What will happen is as soon as hidden class is removed from 1 of the ul , the transform & opacity will become default. Also add some transition to the ulHope this help ๐ค& Please keep Coding such nice solutions ๐ฅณ
Marked as helpful
@mohamedyasser27
Submitted
this one was hard because i couldn't do the progress the bar but using css custom variables along the input event allowed me to create a script that tracks the progress
but a problem i had was the resizing which made the range break so i had to using resize observer to keep it consistent but i don't think it's the right approach but i couldn't come up with a formula in css that doesn't make the progress bar exceed the button on resize
@abhik-b
Posted
Hello Mohamed , Your solution looks great & it is very responsive . Great Job done ๐ฏ
Just a opinion : Yes you are right , you do not need resize observer. Instead you can use responsive css media queries. What I did was :
.slider-container
& made it position:relative
.label
with width :90%;
so the label always has 90% width of the parent.slider-handle
span & .slider-fill
span. As the name says 1 is for showing the custom fill & other for custom handle.Here is my code
Hope this helps ๐ค & Please keep coding this nice solutions ๐ฅณ
Marked as helpful
@kostyafarber
Submitted
This one was a fun one! I decided to dip my feet into the world of React.
It was definitely overkill for this project but the point of it was to start getting comfortable with React and it's workflow and syntax.
I tried to have more structure to my SCSS. I added a CSS reset to revert some settings to make development easier.
I also spiced things up and made a product card... of myself as a graded collectible. Not a bad price either ๐.
Please feel free to give feedback on my React code, project structure etc.
Keep on coding!
@abhik-b
Posted
Hey Kostya ๐
Your solution looks fantastic . I liked the concept of you as a product card ๐ Your react component preview
is great & the structure you used to name the classes seems good.
Just a opinion : adding some padding to .root
div will give some spacing to the card in mobile screens.
Fantastic job , keep up contributing such amazing solutions ๐ฅณ
Marked as helpful
I'd like receive your feedback
@abhik-b
Posted
Hello Erik ๐
Your solution looks excellent ๐ฅณ The animations are great , shadows to the images are great & your site is responsive as well. Great job done
Just a observation : When I click on left button slider animations are great but when I click on right button slider animations are not as great as left one.
Other than that Great job , please keep contributing such amazing solutions. ๐
Marked as helpful
@kara-harshit
Submitted
@abhik-b
Posted
Hey Harshit ๐
Your solution looks nice & your tailwind usage seems great ! Though it would have been nice if it was responsive for mobile screens but anyway nice try.
Just some opinions :
"
from body"
flex flex-col gap-3
then for each chatleft
class you don't need to give bottom margin & overall the code would be organized.Keep doing these projects & Your skills will much better !! Happy Coding ๐
@Lekanjoy
Submitted
@abhik-b
Posted
Hello Alabi ๐
Your Solution looks very nice !!! It is very responsive & it's interactivity seems amazing to me. Great Job ๐
Just some opinions :
.box
has a fixed max-width
then on large screens it will look more nice. Example : max-width:550px
.billing
has justify-content : space-evenly
instead of justify-content : space-between
then the transition from large screen to mobile screen looks slightly betterYour solution is very close to original design , Nicely Done !! Please Keep Coding such amazing solutions
Marked as helpful
@arevalosebastian
Submitted
I would like feedback, tips, all is welcome, thanks.
@abhik-b
Posted
Hello Sebastian , Your solutions is fantastic & it is very responsive also. Well Done ๐ฅณ๐ฅณ
Just a opinion is if you give bottom: 0px; right: 0px;
to tooltip
class for small screens then it will stay responsive . Other than that great job and please keep contributing such amazing solutions ๐คฉ๐
@nicolasgonzalez98
Submitted
I need help with the image of the girl and the double bg image. Thank you so much and sorry for my english.
@abhik-b
Posted
๐ Hello Nicolas , Your solution looks nice ๐
img
tags in html instead of background-image
in CSS for the girl image then it will look even nicer & solve the problem.background-size:contain
inside main to make it look nicermedia-query
for responsiveness & stay away for viewport units vw
for width & height of simple elements like cardOther than that overall it looks nice so Please keep solving challenges & happy coding ๐ฅณ๐
Marked as helpful
@SuhodolecA
Submitted
Hi everyone! I spent a lot of time working on this project. I would appreciate any of your comments and advice!
Thanks!
@abhik-b
Posted
๐ Hello Anton , I see you you spent your time creating a perfect solution , Very well done ๐ฅณ๐ Your solution is very responsive , works & looks very well. Please keep coding such fantastic solutions ๐คฉ๐
@chiamaka-ugwu
Submitted
@abhik-b
Posted
๐ Hello Chiamaka, Your solution is great & responsive enough . Good job ๐ I am just sharing some of my opinions that might make it look nicer.
object-fit: cover;
to img to make the image look responsive & nicecard-container
instead of width:30%
use max-width:400px
(use value of your own choice) can make the card look nice & responsive for large screens.Other than that your solution is nice , Well Done & please keep coding amazing solutions like this ๐๐
Marked as helpful
@aleksFedotov
Submitted
Feedback will be highly appreciated
@abhik-b
Posted
๐ Hello Aleksandr , Your solution is fantastic & the animations are ๐ฅ๐ . I also liked the hover effect on dice button. Great job , Well Done & Please keep coding such amazing solutions ๐คฉ๐
Marked as helpful
@Yash-devasp
Submitted
I have complete my first challenge on frontendmentor.io and with this I am starting a new journey on this path. I had a lot of fun completing the first assignment.
As I am a beginner and have just started learning about web a couple of days back so I am sure my code can be improved. Please help me in my journey by pointing out ways in which my code can be improved or if there could be a better way to tackle a problem.
@abhik-b
Posted
๐ Hello Yash ,Congratulations on completing your 1st challenge. Your solution is fantastic & it is very responsive ๐คฉ๐
Some of my opinions I am sharing with you :
margin: 0;
to *{ }
to remove unwanted margin from all elementsheight
from card
& give gap:1em
to make the card look more like the designbox-shadow
is very niceVery Well Done ๐คฉ & Please keep coding such amazing solutions ๐ฅณ๐
@GitNutts
Submitted
I wouldn't have even known where to start on a simple project like this a year ago but I can complete it now like it's second nature to me. It's a great feeling!
@abhik-b
Posted
๐ Hello Jordan , So nice to know that you are feeling great completing this challenge. It looks absolutely perfect to me & it is highly responsive as well. Great job well done ๐ฅณ
Please keep contributing such amazing solutions ๐คฉ๐
@Jay-0331
Submitted
Hello, this is my third submission
hope you all can review it and give some feedbacks
Thank you
@abhik-b
Posted
๐ Hello Jay , Your solution seems perfect to me & it works perfectly . Only opinion I am sharing with you is for small screens ( (min-width: 426px) and (max-width: 1024px)
) if you set a max-width
to the calculator container then it can look nicer. Other than that great job , well done๐ฅณ๐ฅณ๐ฅณ Please keep coding such amazing solutions ๐คฉ๐
Marked as helpful
@Kamasah-Dickson
Submitted
Hi guys, It has been a while now making a new challenge. yep am here with a new challenge that I tried my hands on. What do you think I could have Implemented in this solution. Please do not forget to like and comment as always. Thank You : )
@abhik-b
Posted
๐ Hello Kamasah , Your solution is wonderful & responsive . Even I am working on this challenge & now I can look to your solution as a reference well ๐คฉ
Only opinion I am sharing with you is that for the .circle
input checkbox you can make it a circle by giving the below styles to make it look even nicer.
width: 16px;
appearance: none; //this is very important otherwise checkbox won't use any other css styles
height: 16px;
padding: 4px;
border: 2px solid hsl(289, 100%, 72%);
border-radius: 9999px;
Then finally give some gap to .price-1
& .price2
to give some spacing & it will look appropriate. Overall Great job well done๐ฅณ๐ & please keep coding such nice solutions ๐
Marked as helpful
@Miculino
Submitted
Hey guys!
Hope you're having a great time working on Frontend Mentor challenges :)
I'm finally back with another lovely challenge that I had a blast working on, though it took me quite a while to finish it for reasons that you'll discover once you see the website and its Github repo :D
Frankly, I still wanted to try out a few more ideas, but at the same time, I had pretty much implemented all the features/customizations I wanted for this project.
Feel free to check it out and let me know what you think works / doesn't work for this project and where I could make further improvements/tweaks! :)
P.S: Thank you, to each one of you who checked out my previous challenge (QR Code Generator) and supported me. I appreciate it a lot and I hope these kinds of projects will also inspire you to try out your own unique and creative ideas! :D
See you in the next challenge - happy coding, y'all! :)
@abhik-b
Posted
๐ Hello Buhaianu , Your solution is Excellent , the animations are just top-notch ๐ & it is very responsive. Well Done & please keep contributing fantastic solutions like this to inspire us ๐ฅณ๐
Marked as helpful
@L0rdix
Submitted
I think i did a pretty good job for my first project, but im not sure how you guys see it. Would love to hear your feedback and get to know what to improve on. So yeah, what can i do better next time?
@abhik-b
Posted
๐ Hello Amir , Congratulations on completing your 1st project. The solution is amazing & it works correctly Well Done ๐คฉ๐คฉ๐คฉ Only opinion I will share is remove margin for left & right from .sec_container
& please add another media query for min width 500px then give margin for left & right this can make it more responsive. Other than that you did a Great Job, Please keep contributing wonderful solutions like this ๐ฅณ๐
Marked as helpful
@JearlDev
Submitted
Comments and criticisms are welcome :)
@abhik-b
Posted
๐ Hey Joshua , Your solutions seems perfect to me, Fantastic Job Well Done & please keep contributing such amazing solutions ๐
Marked as helpful
@Dantedavidson
Submitted
Fun challenge. Next time I would spend more time with the animations and maybe do some sort of slot machine effect for the computer choice. Let me know your feedback!
@abhik-b
Posted
๐ Hello Dante , Your solution is amazing , current animations are nice ๐ , it is responsive as well. Very well done ๐ฅณ & I would love to see slot machine effect for the computer choice so please keep creating such amazing solutions ๐คฉ๐
@Rishabh-Codes27
Submitted
Do check out my solution and let me know if anything is wrong. Tried my best in doing it as close as given in the design. Also got to learn about the section tag.
@abhik-b
Posted
๐ Hello Rishabh , Your solution is amazing & highly responsive , Great job ๐คฉ
If you want to position the card center on every screen then :
position:relative
from card & attributiondisplay: flex; flex-direction: column; min-height: 100vh;
to bodyflex:1
to section & remove the height
& padding-top
padding-top
from attribution (maybe 20px)Other than that your solution seems great , Well Done ๐ฅณ๐ฅณ๐ฅณ & Please keep coding amazing solutions like this ๐
Marked as helpful
Built with CSS Grid & Vanilla JS.
If anybody can help me get the flip animation, please let me know. I know I could have used a jquery plugin for this, but didn't wanted to go that route.
Any feedback is appreciated!
@abhik-b
Posted
๐ Hey HackMort , Your solutions is very nice & responsive Well Done ๐ฅณ๐ฅณ About the flipping animation without jQuery , I took inspiration from this codepen. Also I think that instead of using pseudo elements for background images if you set them on body itself , the background images would be visible.
@media screen and (max-width: 526px)
body {
background-size: auto;
background-position: center bottom, -850px bottom;
}
body {
background: hsl(235, 16%, 14%);
background-image: url(./images/bg-stars.svg), url(./images/pattern-hills.svg);
background-position: center bottom, center bottom;
background-repeat: repeat, no-repeat;
background-size: contain;
}
Great job with the timer & please keep coding amazing solutions like this ๐
Marked as helpful
@stephmunez
Submitted
Hey everyone!
I just completed another challenge. ๐ค๐ฝ
Super proud on this one because it is my first solution using React, it's a bit of an over kill but for practice purposes I added a Testimonials component instead of just putting everything to the App component.
I also realized how frontend libraries like React make life easier from development to production, I can't wait to learn and grasp more about React and its magic!
One more thing, I already added an <h1> tag and hid it out of view but the report still says there's an error, if someone can tell me why it will be much appreciated.
Feedback is always welcome. Please let me know of any issues you may find and/or how I can improve my code :)
Happy coding!
@abhik-b
Posted
๐ Hello Stephen , Your solution looks perfect to me ๐๐ and Yes react definitely makes life easier !!
Just some of my opinions which can help you :
article
(testimonial) & rendered all of them inside testimonials component & this way your code would have been more easy to read.Very well done ๐ฅณ๐ฅณ๐ฅณ & please keep contributing such amazing solutions ๐คฉ
Marked as helpful
@Onyeka-Aribeana
Submitted
Hi everyone. This is my first HTML-CSS-JS project (after watching a lot of tutorials ๐ ). Would really appreciate any feedback on the improvement of this project. Thank you in advance ๐.
@abhik-b
Posted
๐ Hello Onyeka , Your solution is fantastic , it is highly responsive & looks perfect to me. Your form validation works great & I loved the way you wrote the js code.
Well Done ๐ฅณ๐ฅณ & please keep contributing amazing solutions like this ๐คฉ
Marked as helpful
@Zeiyar-Lynn
Submitted
It has been quite a long time since I submit the last one. Here I'm, with new technologies. For React, what animation library do you prefer?
@abhik-b
Posted
๐ Hello Zeiyar , Your solution looks perfect to me , it is very responsive & looks fantastic to me, Great Job Well Done ๐ฅณ๐ฅณ For animation or transition with react you can use framer motion and I agree with what @denielden says .
Please keep contributing amazing solutions like this ๐