Feedbacks will be appreciated :)
Julio Velezmoro
@LakorthusAll comments
- @adram3l3chSubmitted over 2 years ago
- @bubakowskiSubmitted over 2 years ago
Sometimes i have problems to contatin whole vw and vh, any ideas? check also the mobile view
@LakorthusPosted over 2 years agoHi @bubakowski, i had the same question not to long ago! this video from youtube make everything more clear https://www.youtube.com/results?search_query=rem+em+px+vh+wh
Awesome work by the way!! I hope this help you as it did to me.
1 - @HenryAguSubmitted over 2 years ago
Updated version
@LakorthusPosted over 2 years agoGreat work! Just one thing to consider:
<main="container"> wrapping all the content after the body that way you can get rid of the html issues, also take in consideration give an attribute on the image that way is more easy to understand what is that image about. =) i hope this works for you as it did for me.
Keep up the good work!
1 - @f5rcbhSubmitted over 2 years ago
it was pretty fun and not that difficult , the only problem i had is how to make a an to image above each other and make one of them appear on top the other one. iam very welcome for any advise thanks <3
@LakorthusPosted over 2 years agoHi @f5rcbh Great work with this challenge!!
For the image of the eye appearing when you hoover i add a div with the to image and after in css i played with the positioning and opacity. Feel free to check my repo where i use:
class = Overlay --> Div class = Middle --> Eye image id = cubik --> Cube Image
I hope you can find this useful and keep up with the good work!
Marked as helpful0 - @Bamidele124504Submitted over 2 years ago
i had issues trying to style the svg image to fit to its container, but with a little research i figured it out.
@LakorthusPosted over 2 years agoGreat work! Just one thing to consider:
<main="container"> wrapping all the content after the body that way you can give flex to the body align-items (center), justify-content (center) and height (100vh) to move the box summary to the middle of the screen. =) i hope this works for you as it did for me.
Keep up the good work!
0 - @sevrinnSubmitted over 2 years ago
I know the code for rating buttons is awkward. I know that there is a way to loop through those numbers. I'm a little unsure tho. Would sticking those variables in an array and then mapping through them work?
I was having a hard time getting the attribution to go where I wanted it to. I need to fix some things so the mobile version is a little more responsive. It can always get better.
@LakorthusPosted over 2 years agoHello @sevrinn. It looks great! Keep up your good work! 🙂
1