All comments

• 

  AReactDeveloper• 340

  @AReactDeveloper

  Submitted 2 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

  30

  

  martbudr• 90

  @martbudr

  Posted 2 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

  This comment has received 1 replyShow
• 

  Karim Essam• 250

  @KaEssam

  Submitted 3 months ago

  

  Testimonials grid section

  1

  60

  

  martbudr• 90

  @martbudr

  Posted 3 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
• 

  kendo-desu• 70

  @kendo-desu

  Submitted 3 months ago

  

  Grid Cards Challenge

  3

  70

  

  martbudr• 90

  @martbudr

  Posted 3 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

  @SergioBrandaoF

  Submitted 3 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

  40

  

  martbudr• 90

  @martbudr

  Posted 3 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