@NaQu2003
Latest solutions
Latest comments
- @Fayozxon
Hello my friend
I see that the description text on the hero section is two lines on the design, but your is one line and really makes it difficult and uncomfortable to read.
To solve this problem I suggest you to use the css
max-widthproperty. It does not let the element exceed the given width limit.For example if you set
max-width: 500pxthe maximum width the text can reach is 500px, in this way you can easily solve the problem.Good luck with the future projects!
Marked as helpful - @Abubakar-Tanko@Fayozxon
Hello Abubakar Tanko.
When I looked at your code I noticed that you used id's in css and I don't think it is best practice. I would suggest you to use id's only for javascript and classes for css.
Good luck in your future projects.
Marked as helpful - @fkrunic@Fayozxon
Just a quick reminder: the favicon is still vite logo, i would recommend chaning it.
- @Muhammadjewel@Fayozxon
For naming variables you can use Kevin Powell's method.
Instead of
--color-primary-bg, you should write repeatitive words first, (also suggest short forms) like--clr-bg-primaryor--clr-txt-secondaryI would also suggest you to use variables not just for colors or shadows but also font sizes.
Marked as helpful - @Rhinozer0s@Fayozxon
Hello, Rhinozer0s. Congratulations on your first project! It is great but you should adjust the size of the cards (maybe by giving some padding) and the shadow (in the design it looks softer) and also, you I suggest you using semantic elements on HTML like h3 instead of span or div for a text. Good luck on your next project!
- @shakhboz-shukhrat@Fayozxon
I was also doing this project (although i don't have acces to this challenge, i copied from preview) and I am using VueJS and Bootstrap, Sass. You can find the site preview and source code on telegram: @fayozhons
