@i-blu
Submitted
@Olumide2596
@i-blu
Submitted
@Olumide2596
Posted
Hi, there, I went through your code and it seems you've not got to learning flexbox or grid yet. if you have, it will be nice to use it for this project, if not, nice job pulling this off, and be sure to try it again when you do learn flexbox.
Also, it's better to wrap all the divs in a container like this:
<body>
<main>
<div class='container'>
</div>
</main>
</body>
@CodeLamp168
Submitted
My first time ever using java-script independently and also using methods to implement specific styles and transitions after a click event.
My javascript format may be a bit sloppy and would love some tips on how I can make them cleaner and perhaps find different syntax to make it shorter.
@Olumide2596
Posted
Hello, completed this challenge yesterday and got some reviews that help me improve my code. This is the only part of the review I got that o think might be helpful to you.
You should use the <main>
element only to wrap the main content of the page. You should keep the attribution content in a <footer>
element and out of the <main>
element. These are semantic elements and are used to structure the page. You can read more about them here
@Olumide2596
Submitted
Feedback and any tips on how to improve my code are highly appreciated. Also, I'm having the challenge of unchecking a previous rating if I choose another. Thank you😁.
@Olumide2596
Posted
Thanks a lot, I will work on it using your feedback
@Ax-cd
Submitted
Hi! This is my solution, there are still a few things I would like help with:
@Olumide2596
Posted
@pbitonga17
Submitted
I dont know how to put the monitor on the very left side of the screen. If someone knows please tell me :)
Also in this project i tried to use the 'clamp()' and added some basic animation on my header and button.
Any comment or advise will be appreciated. Thank you!
@Olumide2596
Posted
i did a margin-left of -(minus) and overflow: hidddeon; on the parent