Design comparison
Solution retrospective
Ok so I'm still in training please be indulgent. To indent the text I used 'br' with a class that I change to 'display: none' or 'display: inline'. What other solutions do you know?
Community feedback
- @palakg01Posted over 3 years ago
Great work done! You can try keeping the tablet (medium devices) view horizontal rather than vertical just like the one done for desktop
Marked as helpful1@AlexGDevProPosted over 3 years ago@palakg01 Thanks for your feedback.
I used both sizes requested (375px, and 1440px), but if I push it any further I would probably do a tablet view. Maybe later.
Thanks
0 - @Yemisrach15Posted over 3 years ago
Hey Alex! Good job!
You shouldn't use
<br>
to style the site. Instead add margin/padding to the right and left of the heading. Your solution doesn't have to exactly match the design though. If you get it as close to the design as possible, it's enough.Happy learning and coding😊
Marked as helpful0@AlexGDevProPosted over 3 years ago@Yemisrach15 Thanks for your feedback !
I used the 'br' because I couldn't get the exact same indentation.
I tried the padding / margin, with a 'text-align: center' but it didn't look the same.
I will continue to search a bit.
Thanks
1@Yemisrach15Posted over 3 years ago@AlexGDevPro No need to get the exact same indentation in my opinion. Don't stress much over that.
You're welcome
1@AlexGDevProPosted over 3 years ago@Yemisrach15
I took off 'br' and use padding like you told me.
Perfect !
I made my code more responsive too.
Hope you'll find it good ;)
1
Please log in to post a comment
Log in with GitHubJoin our Discord community
Join thousands of Frontend Mentor community members taking the challenges, sharing resources, helping each other, and chatting about all things front-end!
Join our Discord