Skip to content
OverviewSolutions
13
Comments
5
P
Aynur Demir
@Maanlicht91

All comments

  • Aynur Demir•150
    @Maanlicht91Posted 2 days ago

    Congrats on completing this challenge. I'm happy to see people using scss which is very useful for designing.

    0
  • Aynur Demir•150
    @Maanlicht91Posted 4 days ago

    Congrats on completing this challenge. I liked your class name style Jonas taught me this technique especially its useful for scss and sass. Keep going to improve your coding skill.

    0
  • Aynur Demir•150
    @Maanlicht91Posted 7 days ago

    Nice one. Probably you didn't give attention on class names because of mini simple project. Otherwise giving all tags class name for css and id for js is best option and very clear code. This way is hard to understand what is p.one or p.grey if you had huge style codes but ofc this is a simple basic project. No harm to do like this

    Marked as helpful
    0
  • Aynur Demir•150
    @Maanlicht91Posted 7 days ago

    Nice try. But I suggest you to start with this code for most css:

    • { margin: 0; padding: 0; box-sizing: border-box; } body { font-family: "Figtree", sans-serif; }

    if you want container(wrapper) in the mid of page you can add some properties in body also:

    body { font-family: "Figtree", sans-serif; display: flex; flex-direction: column; align-items: center; justify-content: center; min-height: 100vh; }

    0
  • Aynur Demir•150
    @Maanlicht91Posted 7 days ago

    Nice and clean code. Looks like you just needed to use text-align:center for text part. Otherwise I liked your code design.

    0