Hi, @itzvikashgupta. It's a good solution, keep coding... Here are some tips:
-
The
display: grid;
rule doesn't work like adisplay: flex;
. When you typedisplay: flex;
it works immediately with some initial values for the flex-container and the flex-items. However, if you typedisplay: grid;
it doesn't affect any element to set some initial values. It just creates columns and puts each grid item in it. In fact, that's the default behavior of the browsers. So you can get the same effect withoutdisplay: grid;
in this project. -
You can use the
strong
HTML element instead of thespan
element, as shown here:<li><span>Total: </span>Approximately 10 minutes</li>
<li><strong>Total: </strong>Approximately 10 minutes</li>
-
You can use the
article
semantic element for your container and separate them with thesection
semantic elements. This way is more natural for this project and you'll be more comfortable when you're styling the HTML. -
You used the nested CSS, it's a good way to adopt some new CSS features. Maybe you want to check
logical properties
...
I hope these will help you. Keep coding and have a nice day
Marked as helpful
@itzvikashgupta
Posted
Really learned some new things from your comment definitely gonna apply those. Thanks!π @adonmez04
@itzvikashgupta You're welcome. If you want to learn more about in CSS, I recommend you this source: https://web.dev/learn/css
Marked as helpful
@itzvikashgupta
Posted
sure! thanks @adonmez04