Skip to content
OverviewSolutions
2
Comments
3
Paritosh Sahni
@Paritosh7

All comments

  • Sergei Tsvetkov•25
    @tsvetkovsergeySubmitted almost 4 years ago

    Any feedback appreciated :) Check my JS. I create and remove floating box image every time view alternate between mobile and desktop. Maybe I should keep the image and just use css display:none property instead?

    Simple accordion card using Flexbox and JS

    3
    Paritosh Sahni•80
    @Paritosh7Posted almost 4 years ago

    Pretty dope! Everything seems to work pretty nicely. I have also made this project but nowhere close to your precision! Great job!

    1
  • Paritosh Sahni•80
    @Paritosh7Posted almost 4 years ago

    Hey Janegca! That's a cool README.md you have for this project!

    0
  • Paritosh Sahni•80
    @Paritosh7Posted almost 4 years ago

    Hey Gorkem!

    Your site looks spot on for a Desktop view. I think you should try to work on making the wesbite more responsive. Maybe, try to build the website for a mobile view too. Also, I saw that you have created 2 columns, col-1 and col-2 and have given them fixed margins. Another way would be to add both these divs in a single div and give margin-left and margin-right (outer div) an auto value. This would have the made the page more responsive. The email validation is spot on !

    Keep coding!

    2