Skip to content
OverviewSolutions
29
Comments
4
Shchetkov Maksim
@ArmsAndArrows

All comments

  • Shchetkov Maksim•470
    @ArmsAndArrowsPosted over 1 year ago

    Hi! I like your solution. One thing I can offer - when error message displays - it pushes image down. Toggling class that shows image could be placed not to the whole container but only to the message. And message could have reserved space for it or absolute position to relative position of its parent. Cheers!

    Marked as helpful
    1
  • Shchetkov Maksim•470
    @ArmsAndArrowsPosted over 1 year ago

    Hi. Great work! It's a quite accurate implementation.

    I can suggest you to start building grids by mobile first step - it's much easier to reassign grid properties with media queries as the size keeps growing. Also it's great to avoid using pixels except really small things like borders.

    And I checked code on github it seems like at line 127 of css file you started writing responsive queries, but you didn't close whole query at the end with a curly bracket.

    I hope these suggestions would help you a little at your professional career =)

    Marked as helpful
    1
  • RavanCod•250
    @vanessie2424Submitted over 1 year ago

    So, I'm done and making the mobile view of this project was one of the parts I found difficult. Also, I would like to know how to be able to select one at a thing. If someone could help, I would be grateful. Thank you.

    FAQ ACCORDION CARD using HTML, CSS AND JS

    1
    Shchetkov Maksim•470
    @ArmsAndArrowsPosted over 1 year ago

    Hi! I like your implementation, but I could suggest some points of research in case of coding js:

    1)When you declare a new variable it's a good practice to keep the most of them in the beginning of your code. The reason for using variable inside a function is only when that variable should be used only inside the scope of function.

    2)When you use a variable that not supposed to be changed try to use "const" instead of var. It will prevent from future mistakes

    3)You could upgrade your project if you take a research about event listeners:e.g document.addEventListener("click", callback function). So you could make element interactive.

    4)For accordeon you need to practice with array looping( regular one or with method called "forEach". Check on mozilla MDN or on youtube. For example: you could have same class for all dropdown elements and using loop you could apply classes to all of them making it disable showing content and select conditionally that one that needs to be open.

    5)To get even better results, you can look into classList methods, this could help you write less code and it would let you apply animation with less pain.

    I hope that somehow could help you! =)

    Marked as helpful
    0
  • Shchetkov Maksim•470
    @ArmsAndArrowsSubmitted over 1 year ago

    First time I tried to use tailwind. I don't have figma file so it could be a little bit different

    I'd like to hear from tailwind pro's: 1)Is it possible to use BEM with tailwind? 2)How do you usually add custom classes, is it via @apply only or just as simple class in input.css?

    Thank you!

    Age Calculator App TailwindCSS

    2
    Shchetkov Maksim•470
    @ArmsAndArrowsPosted over 1 year ago

    Logic is fixed for now. Added number animation.

    0