Skip to content
OverviewSolutions
28
Comments
9

katrine9176

@katrine9176540 points

Life is beautiful isn't it? I'm 13y/o, in the future I wanna become a fullstack dev. I love coding

I’m currently learning...

React.js

Latest solutions

Latest comments

  • katrine9176•540
    @katrine9176Posted almost 2 years ago

    Hello! Your solution looks really good! 🤗

    Here is some advice:

    • usually it's better to name HTML file as index.html,
    • use semantic tags, learn about that here -> article.

    Hope it was helpful and happy coding! 😁

    1
  • katrine9176•540
    @katrine9176Posted almost 2 years ago

    Hello! Your solution looks very good! 😁

    Here is some advice:

    • put your CSS styles in style.css,
    • to connect style.css with HTML file add in <head> - <link rel="stylesheet" href="css/style.css">,
    • add CSS reset, learn about that more here -> article,
    * {
  margin: 0;
  padding: 0;
  box-sizing: border-box;
}
    • to make code better to read, consider downloading nice extension, if you use VSC I recommend prettier -> prettier, and for HTML default formatter,
    • update screenshot on frontendmentor :)

    Hope it was helpful and happy coding! 😃

    Marked as helpful
    0
  • katrine9176•540
    @katrine9176Posted almost 2 years ago

    Hello! Your solution looks really good! 😁

    Here is some advice:

    • don't put your CSS styles in HTML file, instead of that create style.css file and there put them,
    • to connect style.css with HTML add in <head> - <link rel="stylesheet" href="css/style.css">,
    • I recommend you to use semantic tags, learn about that here -> article,
    • an article about IDs and CSS classes -> article,
    • in CSS reset you don't have to put %:
    * {
  margin: 0;
  padding: 0;
  box-sizing: border-box;
}

    Learn about that more here -> article.

    Hope it was helpful and happy coding! 😀

    Marked as helpful
    1
  • katrine9176•540
    @katrine9176Posted almost 2 years ago

    Hi! Your solution looks really good! 😄

    Here is some advice:

    • in * put also:
    margin: 0;
padding: 0;
    • every page has to have one <h1> tag as a main headline,
    • I recommend you to use semantic tags, learn about it here -> article

    Hope it was helpful and happy coding! 😀

    Marked as helpful
    1
  • katrine9176•540
    @katrine9176Posted almost 2 years ago

    Hello your solution looks really good! 😃

    Here is some advice:

    • don't put CSS styles in index.html, instead of that create style.css and there put your styles,
    • to connect style.css with HTML file, add in <head> - <link rel="stylesheet" href="style.css">,
    • add a CSS reset, learn about that here -> article, to do that add:
    * {
  margin: 0;
  padding: 0;
  box-sizing: border-box;
}
    • every page has to have one <h1> tag as a headline,
    • to center a card, try:
    body {
        display: flex;
	min-height: 100vh;
	align-items: center;
	justify-content: center;
}
    • I recommend you to use semantic tags, learn about it here -> semantic tags

    Hope it was helpful and happy coding! 😁

    Marked as helpful
    0
  • katrine9176•540
    @katrine9176Posted almost 2 years ago

    Hello your solution is really good! 😃

    Here is some advice:

    • in * add only:
    margin: 0;
padding: 0;
box-sizing: border-box;

    the rest put in body,

    • add <h1> as a headline,
    • I recommend you to use semantic tags, learn about that here -> article

    Hope it was helpful and happy coding 😁

    Marked as helpful
    0
View more comments