Skip to content
OverviewSolutions
6
Comments
5
Anderson Moreira Silva
@anderjackie

All comments

  • Simsim99•70
    @Simsim99Submitted about 1 month ago
    What are you most proud of, and what would you do differently next time?

    im proud of the fact that i am learning, what i would do differently is not overthink stuff, i need to keep it simple to avoid wasting time!

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

    Starting with the phone layout first was confusing since it is my first time doing it, i really do not like doing mobile first but i have to. The picture element was another hurdle but after a couple of youtub vids explaining it, i managed to do it! I learned alot!

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

    going from mobile to desktop layout

    preview card using HTML and CSS

    1
    Anderson Moreira Silva•70
    @anderjackiePosted about 1 month ago

    Your design is perfect! Really good.

    About the responsiveness I would put the restraints of size and dimensions in the components instead of the div.all. This way you guarantee that if you wanna use that in other designs you can just put it there instead of redesign the container around it.

    Marked as helpful
    0
  • João Vitor Princesval Caetano•240
    @PrincesvalSubmitted 2 months ago
    What are you most proud of, and what would you do differently next time?

    I'm really happy with the final result, especially the result on mobile devices.

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

    The last section: how to create that table effect? The way I found was to use a separate ul for each item and the tag hr at the bottom of each one.

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

    A simple way to format images for mobile devices.

    Responsive recipe page using ul and ol

    1
    Anderson Moreira Silva•70
    @anderjackiePosted about 2 months ago

    Hey man, great job! You did fantastic, it is really close to the design.

    I have a couple of comments but you can take them as you please

    1. I can see you reset your stylesheet but you can use a more complete reset that is super helpful for further development. I based mine on this one reset-css.
    2. To use the table tag in your html you can style it with border-collapse: collapse. This way you can have the effect of the design.
    3. I believe you solution for the image was pretty clever. Another way to do it is let the container have a padding for bigger screens and remove it for smaller screens, but make all the other components of the recipe put their own padding for smaller screens. this way.

    Keep the good work =)

    0
  • Saumyadip Biswas•50
    @biswasSaumyadipSubmitted about 2 months ago
    What challenges did you encounter, and how did you overcome them?

    Initially, I was unable to write proper media queries so I took help from sources like MDN, StackOverflow, and css-tricks.com.

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

    Some feedback on writing SCSS and CSS structure will be good.

    Responsive Social links profile using Vite and SCSS

    2
    Anderson Moreira Silva•70
    @anderjackiePosted about 2 months ago

    Your solution looks amazing. Almost perfect with the design. Very good. It is a nitpick but I would separate the reset css rules from the design style in 2 separate files. I believe it is a good separation of concerns.

    1
  • zdnnnnn•40
    @zidannn24Submitted 9 months ago
    What are you most proud of, and what would you do differently next time?

    It is important for me to receive feedback from the community regarding user responsiveness.

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

    So far, I haven't encountered any difficulties while working on this project. However, if there are still areas where I can improve, please provide me with feedback.

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

    Is my code clean? And are there any unnecessary elements?

    Blog preview card using HTML & CSS Flexbox

    1
    Anderson Moreira Silva•70
    @anderjackiePosted 2 months ago

    Hello!

    I love the animation in your solution. Very nice touch.

    I believe everything looks good but I would remove a little bit of the tags in your html, like starting after the body with a main directly in your sections set the class directly in it. No need for the inner div's.

    In the style as well put only the classes and tags that are related to your solution. It's easier to review ;)

    0
  • Anderson Moreira Silva•70
    @anderjackiePosted 3 months ago

    Hello!

    Your page is really good. It's super close to the design purposed.

    I think the only thing I noticed that could be improved is the lateral padding for the text. In the design is a little bit bigger than the one you coded.

    Maybe add another div only for the text to help you get that style.

    0