Skip to content
OverviewSolutions
6
Comments
3

Maksim Deviatilov

@maksimcoder185 points

Welcome to my GitHub page. My name is Max and I'm 17 year old student who adores coding and web-development.

Latest solutions

Latest comments

  • Maksim Deviatilov•185
    @maksimcoder

    Hello, nice work you've done! I've only 1 thing I saw. When creating a button with outline style it's better to use border rather than outline. Additionally, you might have noticed that button's size is like bouncing when hovering it that happens due to the absence of border property on not hovered button state. So what I suggest exactly: 1 - for outline button style use borders 2 - to make styling look natively use border: *px solid transparent on not hover state and on hover state use border style you wish.

    Thanks for reading, happy coding!

    Marked as helpful
  • Maksim Deviatilov•185
    @maksimcoder

    Hello, Rayane, I was looking through your work after your comments on mine and spotted some issues. In mobile (375px) width, middle card top borders have radius, while two other cards don't. This is a little thing but still I wanted you to know it. 😁 Also, some margins, according to the mobile-design file, should be done. That's all I wanted to share with you, I take your work as a great example. Have a nice weekend!

  • Maksim Deviatilov•185
    @maksimcoder

    Hello, Aayushi, I've looked through your JS code and have some tips for you as I would do if I were you.

    1. When you create a function createtodoElement(val) use Class in order to automate and avoid repeating your code.
    2. When you're adding an eventListener on footer buttons, use cycles or forEach method to automate the process. (activeBtn.classList.remove("active"); completedBtn.classList.remove("active"); => footerBtns.forEach(item => item.classList.remove('active');) I hope this tips will help you in future even if you knew them before. By the way, I'm currently working on this project and that's awesome that you did it!