@Arouca-b
Latest solutions
Latest comments
- @joshmichael23
Removing the
height: 80%and adding maybe a little bit of padding for the top and bottom will make it more neat across all devices. Right now there's some extra space when the height of the device is a bit too high. Overall good job! - @ioannisskr@joshmichael23
Good job on your project!
I don't recommend using the
<br>tag as it does not have any semantic meaning. You can use<span>inside your<p>tag and usedisplay: blockin it.Overall great design. Keep up the good work :)
- @rajendra2708578@joshmichael23
Great job on your project. You can make it better by using the font given in the style-guide.md.
What I usually do is go to fonts.google.com then search for the font. Add the necessary font-weights by clicking on the + and then click the cart on the top right. You can then use the
<link>or the@importunder the Use on the web section depending on your preference. Hope it helps :)Marked as helpful - @xirb22@joshmichael23
You can use the widths as reference when inspecting the element and changing the width according to the design. That way you can see if your project matches the it. :)
- @nadun-dhananjaya@joshmichael23
I love the spinning dice effect while fetching. Never thought of that 😁
Marked as helpful - @Abdulliadi52@joshmichael23
I noticed that there is a
max-width: 1440pxon your<div class="bod">. A white space is seen when the browser is more than 1440px. You can solve this by removing the max-width as it overrides thewidth: 100%.Also try to include a validation function on your upcoming projects as it was missing from this one.
Overall, good job on the project :)
Marked as helpful
