Keelan
@KeelanJonAll comments
- @ChamuMutezvaSubmitted almost 5 years ago@KeelanJonPosted almost 5 years ago
Hi ChamueMutezva! This is coming along well. In regards to your question about font-size, it's a 700 font-weight and I've got mine set at 4rem, I tried to adjust it when previewing yours but it wouldn't change which tells me you may not have the 700 bold montserrat font embeded.
One quick tip I learnt while coding this which I feel may also help you is in regards to the background images, you'd be surprised how no matter how much you learn, you always learn a little something new!
You can set multiple backgroud images like this:
background-image: url("images/bg-top.svg"), url("images/bg-bottom.svg");
And then set their positions seperately as seen below:
background-position: top right, bottom left;
Hope this helps!
Kind regards, Keelan
2 - @adarshcodesSubmitted almost 5 years ago
The toggle didn't work because I'm not started to learn Javascript.
@KeelanJonPosted almost 5 years agoLooking good Adarsh, did you prefer your style of button to the set challenge? Also, my design eye would recommend softer shadows. Other than that great job on this.
1 - @mtalhasubhaniSubmitted about 5 years ago
I tried to make it responsive but couldn't make it. I used all that I learned in two days of HTML and CSS.
@KeelanJonPosted about 5 years agoVery good try for a two-day learner! I noticed your solution uses absolute positioning and isn't mobile responsive.
For your next steps, I'd recommend looking into CSS Flexbox and then onto CSS Grid to easily produce a mobile responsive layout without the need for absolute positioning.
Absolute positioning creates the issue of removing your elements from the HTML flow, meaning they ignore all other content and end up overlapping on smaller devices.
Hope to see an updated solution from you soon!
Keelan
1