its getting really easy to do projects like this...tho i did it half heartedly...i made it responsive to mobile, tablet and laptop screen sizes.. i hope i can start earning money with it so i can focus more
MolinaEf
@MolinaEfAll comments
- @dullarzeeWhat specific areas of your project would you like help with?@MolinaEf
its good
- @dullarzee@MolinaEf
I would include the eye icon on the img container and use position absolute to put it over the nft when hover by changing opacity. Also u should add an cyan overlay when the nft is hovered
- @wess-MHHW@MolinaEf
It looks great, I'd maybe just reduce the animation time of the nav options
- @angelveliz1837@MolinaEf
I would put the media query as a max width: 768px cause the values between 375 and 800 px are looking weird. the rest is fine
- @gowthamjk08@MolinaEf
looks fine except for one of the radio buttons. try setting width in main to 700px and it looks great
- P@jdillard2532@MolinaEf
looks great bro
- @hmonster013@MolinaEf
honestly, its better than the original with the gradient
- P@brunagoncalvesWhat are you most proud of, and what would you do differently next time?
I'm proud to complete another project, it's not 100% yet, but I intend to make adjustments.
What challenges did you encounter, and how did you overcome them?I feel like I'm getting better at JavaScript, I think I have a lot to learn, I intend to practice more.
What specific areas of your project would you like help with?Any comments that help me improve will be welcome.
@MolinaEfhonestly, its amazing. I forgot about adding reset btn responsiveness and u made me remember thanks
- @Tsuna21182@MolinaEf
looks really good, especially animations. I'd put height: 100% on the body so the grid looks more centered but that's just me
- @dineshmoorty@MolinaEf
Hey, it looks good. Probably would give the container flex-direction:row-reverse so the image goes to the right but the rest is great
Marked as helpful - @Enisco29What are you most proud of, and what would you do differently next time?
I'm proud of being able to write javaScript
@MolinaEfwould change the background color to be the lightgrayish color for more contrast. The rest looks good
- @alaa-mekibesWhat are you most proud of, and what would you do differently next time?
px to rem & rpx & vw (cssrem) - This helped me for converting px to rem by Moving the cursor to
px
value, pressAlt + z
to convert rem@MolinaEfI'dd add a lil more padding to the boxes but that's just me. Rest looks amazing
- @F9F6EEWhat are you most proud of, and what would you do differently next time?
Achieving this challenge, i've learned more about flex-box and css grid.
What challenges did you encounter, and how did you overcome them?I've learn flex-box and grid more deeper, getting the good alignements was a bit tricky at some point but it was a good challenge.
What specific areas of your project would you like help with?Instead of using absolute position for the icon, aligning all the card content to the right and the text to the left helped to put the icons in the good position with some padding.
@MolinaEfGood job, it looks great
- @jsemenborodasWhat are you most proud of, and what would you do differently next time?
I would try to do all this with GRID, because on flexbox it turned out to be a complete mess.
What challenges did you encounter, and how did you overcome them?I encountered a problem with the arrangement of elements inside the card after the block sections were in a row.
What specific areas of your project would you like help with?In code, but I don’t really understand yet how to describe my thoughts.
@MolinaEfLooks really good, I also have to try grid cause this was a mess too
- @PrithivJith@MolinaEf
Looks really good regardless of the larger margins. Only the top margin of the title looks off.
- @AngelPenalver@MolinaEf
Looks good, pretty much like mine. I also got the size a lil smaller but it works fine
- @Otep02What are you most proud of, and what would you do differently next time?
This blog post challenge is so nice. Very simple and easy, for beginner like me. Medyo mahirap pa din hehe
@MolinaEfadd the border black 1px to your container, the rest looks aight
Marked as helpful - @Omar25876@MolinaEf
Seems like u wanted to do something different so I cant really comment I'd still add some margin-top to ur subtext so it looks better
Marked as helpful