Skip to content
OverviewSolutions
7
Comments
4
martbudr
@martbudr

All comments

  • AReactDeveloper•340
    @AReactDeveloperSubmitted 4 months ago
    What are you most proud of, and what would you do differently next time?

    i would try to get the down arrow working also i would try to get the social media panel look identical i would polish and review my CSS next time i would focus more on accessibility best practices

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

    just getting that user box to be that small I struggled with the image size and position if you have any tips or tricks on best way to get that work with few lines of codes id appreciate it thank you

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

    aligning things and positioning them without something else breaking in the site

    Article preview CSS with a sprinkle of js just start the js

    1
    martbudr•90
    @martbudrPosted 4 months ago

    The solution looks good, both on mobile and desktop. The code is clear, one thing that I just noticed was that I think that there is no need to add class to 'main' object - as there should be only one object of this type in the file

    1
  • martbudr•90
    @martbudrPosted 4 months ago

    The html file is not readable at first sight, due to a huge amount of divs - maybe it could be somehow changed. The css file on the other hand is easy to read. The site looks good both on desktop and mobile. The solution is very similar to the design. One thing that I noticed was just the lack of border surround Daniel Clifford's photo (I think that this border should be in the color of quotation sign)

    0
  • martbudr•90
    @martbudrPosted 4 months ago

    The solution works both on desktop and mobile. The layout looks good on different screen sizes. The code is written in a readable way. The solution looks very similar to the design.

    1
  • SergioBrandaoF•130
    @SergioBrandaoFSubmitted 4 months ago
    What are you most proud of, and what would you do differently next time?

    Having learned how to implement art direction -- that is, how to responsively use multiple pictures for the same purpose within a page.

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

    Getting my head around using srcset.

    Product Preview Card

    1
    martbudr•90
    @martbudrPosted 4 months ago

    As for the CSS, the white background for the section with text could be added, and also button:active state, for it to change the color. Also, the site doesn't work for mobile when inspected in the browser, so it would be worth it to checking out. Overall, the solution looks really similar to the reference design, so good job :)

    0