@uzoway
Bachelor degree in Communication management, PR . Head full of birds and crafts. Love designs, although both hands left. Web development has been stalking me from primary school . At University made my first home page. And then it started, the passion for front end programming.
Latest comments
- @TrakaMeitene
At the second section starting with egg, make the image as - background image, background-size:cover and no repeat( to all images in the containers), so it fits the container. ON my screen (bigger than 1440px) it is smaller than the view width.
You've done a great work, keep coding :)
- @shivam5750@TrakaMeitene
As I noticed, at the footer you've made list elements on hover, so it makes a white margin. Before the image place an anchor <a> , href="#", and then at css .social li a:hover . It should work better.
Very fast working. Keep doing :)
- @JulianIfesiokwu@TrakaMeitene
You should see the hrefs to the images. Background does not work. The buttons on mobile width: 414px spaces around. I suggest you to use just one button, arrow images inside of the button, and then you should position images inside of it. Here you can see how I've done it in my solution : https://trakameitene.github.io/Slider/
After all, nice job :) You have made a lot of effort on JS . Keep doing !
- @c0dz-wq@TrakaMeitene
It will help with your background image positioning.
background-repeat: no-repeat; background-size: cover;
I'm new to Js. Maybe you had a good resource to learn and could share ?
- @monicamclaughlan@TrakaMeitene
At the *(..) box-sizing:border-box; will be correct. I would rather use border box only *{ box-sizing:border-box;} cause margins affect the design. div. container height is too much. I would write height :100vh; But I'm not sure if it is the best practice. As the button tag has a border default, I would suggest to take it off : border:none; I would use @media so you could define margins for mobile view.
Clean code, it was easy to understand your work :)
