Skip to content
OverviewSolutions
3
Comments
23

pablodev

@PabloodevSão Paulo, Brazil240 points

improving in everything that I can.

I’m currently learning...

Hi, I'm Pablodev 🤓🖖 I'm currently trying to become a fullstack dev, but not just any dev, one who beats his chest and chooses the companies he works for! I'm currently learning the front-end triad, I've already learned the basics of HTML and CSS and now, I'm learning JS. here start my history

Latest solutions

Latest comments

  • pablodev•240
    @Pabloodev

    Hello, I don't have much to say about your solution, I really liked how it ended, it was a little more similar to mine, I just think your github folder could be more organized, I know it's annoying to do this for every solution, but I think it's a good practice.

    Congratulations on your solution!

  • pablodev•240
    @Pabloodev

    Good man, congratulations on the solution!

  • pablodev•240
    @Pabloodev

    ey @Safi200, whats up?

    Well, I have some suggestions for you and I hope they help you!

    • You used display flex to style and insert each card in its position, but when entering the live site URL I noticed that there were some cards that were out of place, when looking at the root solution, it is possible to see that an approach with display: grip could give you bring more code clarity.

    • Your CSS reset is covered in comments that I believe are not necessary, one of the topics written in the book "clean code" is to eliminate comments, "they are smelly, delete them", because anyway, we all know intuitively that This is a reset, right?

    • This in this case is not a tip but feedback, I really liked the way you carried out the responsiveness, it was so calm and smooth! Congratulations, maybe this is due to the fact that I use display flex in the card div

    • Document your code, I know it's boring to do code documentation, the README is very cumbersome, but believe me, make it routine after finishing a solution, code documentation is something very important, companies will notice how organized you can be be with your codes!

    • I couldn't figure out why, but the cards are becoming distant when I put them on my second monitor, the card profile is exceeding the screen size, as if something was unfinished

    Anyway, it was a great solution, maybe I'll learn from you how to achieve smooth responsiveness, keep coding!

  • pablodev•240
    @Pabloodev

    Hello Viniciu, how are you?

    When I entered your website, everything was ok, but when reducing the screen to a mobile device, I noticed that your footer was on top of some elements, perhaps this problem is because you are using the absolute position in both elements ( main and footer), you might want to take a look at this, anyway, congratulations on the solution!

    if this comment helped you in any way, don't forget to mark the comment as useful, thank you

    Marked as helpful
  • P
    Hexerse•460
    @Hexerse
    What are you most proud of, and what would you do differently next time?

    I was proud of not using too many static qualities. As well as researching on how to do forms and validations.

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

    I faced challenges like how to do a form validation, email validation. How to make the website reactive what ideas I needed to use for the errors etc.

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

    If there is anything that you think might be useful please write it down ! I want to write good and efficient code !

    newsletter-sign-up-with-success

    1
    pablodev•240
    @Pabloodev

    Whats up @Hexerse?

    Congratulation for your sulution...Its pretty good!

    I'll try to give you some tips that might help you.

    • if you removed the attribution from your code, it would be nice to remove the script tag from the top of your HTML, I was a little confused when I entered kkkk.
    • Maybe it would be nice to tidy up your repository and make it minimalist, with just what is necessary, the design folders are still there, as well as the README-template
    • In addition to coding, keeping your lines clean and easy to understand, showing how organized you are, in technology companies this is very important taking into account that you will work in a time of development, so always try to think about how organized you are (take a look at your read me there on github)!
    • Your CSS ended up with almost 300 lines which gives the impression that it was a lot of work, maybe you should take a look at your code and analyze if there is any redundancy or if you could do it in a less complicated way, thinking like this, Little by little you will improve your lines of code!

    Anyway, congratulations on your solution, I know that many observations may seem boring, but treat them as sincere help because I spent a while looking hahaha, if I helped you, don't forget to mark my answer as helpful! thank you bro

    Marked as helpful
  • Christina•240
    @codercreative
    What are you most proud of, and what would you do differently next time?

    I am happy the way the app turned out.

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

    The positioning of the social media overlay was very challenging.

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

    I am happy to hear ANY feedback for improvement. Thank you in advance!

    Article Proview Component built with HTML, CSS, JavaScript

    2
    pablodev•240
    @Pabloodev

    Hi @codercreative, how are you? I loved your solution!

    Here are some tips

    • Tidy up your github repository, I know that sometimes it seems boring, since we are always making solutions, why do I always tidy up the documentation and my folders? Speaking for myself, I always like to observe how organized the developer is with their code documents.

    • Your index is so cute, really good! (I know it's not a tip but positive feedback lol)

    • One thing I love to do after finishing my CSS styling and run the entire code with my eyes again and reduce redundancies, your code has more than 400 lines so maybe it would be cool to try to optimize your lines, this makes your lines more readable and with easy maintenance

    • The last thing is that maybe it would be cool to center the content in the center of the screen, when I open it in the browser it is half way up on the screen, giving the impression that it is undergoing maintenance, maybe it would be cool to align it using the flex display on your main content and align-items: center, but remember that the body needs to be at height: 100vh!

    These are the tips I found, congratulations on your solution, keep it up Cris! ❤️❤️

    Marked as helpful
View more comments