Skip to content
OverviewSolutions
7
Comments
7
DarkCityTreno
@DarkCityTreno

All comments

  • Melissa Martins•150
    @MellTinsSubmitted 4 months ago
    What are you most proud of, and what would you do differently next time?
    • It was a relatively easy challenge, but I had difficulty defining how many columns there would be
    What challenges did you encounter, and how did you overcome them?
    • semantic html, it sometimes confuses me to know which tags to use
    What specific areas of your project would you like help with?
    • What is the next logical step now?

    Testimonials Grid

    1
    DarkCityTreno•80
    @DarkCityTrenoPosted 4 months ago

    Looks a bit bigger than the original design but other than that nice job! ^_^

    0
  • Weverson Godinho•290
    @GodinhoweversonSubmitted 4 months ago
    What are you most proud of, and what would you do differently next time?

    I'm most proud of the responsive design, but next time I will refactor my code to reduce redundancy and use CSS Grid instead of Flexbox.

    What challenges did you encounter, and how did you overcome them?

    I encountered challenges on mobile devices with breakpoints and maintaining container sizes. However, after consulting W3Schools, I was able to overcome them.

    Four card feature section

    1
    DarkCityTreno•80
    @DarkCityTrenoPosted 4 months ago

    Hello @Godinhoweverson Idk if I should comment on other people's code as a beginner myself, but it forces me to so imo you should try more semantic tags instead of generic div, and putting css file into a separate folder seems a bit weird, other than that it's responsive and looks great! Cheers bro!

    0
  • DarkCityTreno•80
    @DarkCityTrenoPosted 4 months ago

    Hey @EliPinheiro I would humbly recommend to work on mobile version too, and use semantic HTML tags instead of regular divs, also probably better to use classes instead of id. Take it all with a grain of salt though since I'm a very beginner myself. Cheers!

    0
  • DarkCityTreno•80
    @DarkCityTrenoPosted 4 months ago

    Looks clean :o

    0
  • DarkCityTreno•80
    @DarkCityTrenoPosted 4 months ago

    Oh the buttons changing the background color on hover is something I should've done as well xD. Nicely done friend!

    1
  • DarkCityTreno•80
    @DarkCityTrenoPosted 4 months ago

    Hey there. I'm a very beginner myself but found and easy yet a bit cheese way to center the card. Try it if you want to

    body {
  display: grid;
  min-height: 100vh;
  place-content: center;
}
    0
  • AlbertoSocorro•80
    @AlbertoSocorroSubmitted 4 months ago
    What are you most proud of, and what would you do differently next time?

    Estoy orgulloso de lo bien que lo hecho para ser la primera vez. Lo único que cambiaría sería, tal vez, el orden en que hice las cosas, que no fue demasiado eficiente.

    What challenges did you encounter, and how did you overcome them?

    El principal obstáculo que tuve fue centrar verticalmente el div, lo cual acabé consiguiendo gracias a un blog.

    What specific areas of your project would you like help with?

    En principio ninguna, ya que estoy satisfecho con el resultado final. Únicamente necesito reforzar mis conocimientos sobre la propiedad "display".

    qr-code-component-main

    1
    DarkCityTreno•80
    @DarkCityTrenoPosted 4 months ago

    Seems like there's the initial starter files in your repository, unless I'm dumb(which might be the case).

    0