All comments

• 

  sheralkaren•50

  @sheralkarenSubmitted about 1 month ago

  

  css html

  1

  50
  

  Dharmendra Rathod•210

  @rathoddharmendraPosted about 1 month ago

  It looks great. Keep up the good work

  0
• 

  Karen Robertson•190

  @indigoroseSubmitted about 2 months ago

  

  Product preview card with responsive css

  1

  10
  

  Dharmendra Rathod•210

  @rathoddharmendraPosted about 2 months ago

  Hey,

  Overall site looks really cool. One suggestion

  • you can reduce the margin for top of card to better position the content in center https://github.com/indigorose/frontend_mentor_hosting/blob/main/product_preview_card/style.css#L53C10-L53C21

  keep up the good work.

  0
• 

  Dharmendra Rathod•210

  @rathoddharmendraSubmitted about 2 months ago

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

  responsiveness breaks after 1440px . Yet to figure out

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

  Building responsiveness for all screen while using flex and grid.

  

  Built using HTML and CSS

  1

  30
  

  Dharmendra Rathod•210

  @rathoddharmendraPosted about 2 months ago

  The page responsiveness doesn't really work on all the screen size with images without adding multiple media queries breakpoint. Is there a better way? Is anyone facing the same problem with grid?

  0
• 

  sheralkaren•50

  @sheralkarenSubmitted about 2 months ago

  

  html css

  1

  21
  

  Dharmendra Rathod•210

  @rathoddharmendraPosted about 2 months ago

  Looks pretty good

  Marked as helpful

  0
• 

  Sohrab Hossain•40

  @planner-sohrabSubmitted about 2 months ago

  

  Recipe page

  1

  10
  

  Dharmendra Rathod•210

  @rathoddharmendraPosted about 2 months ago

  Here are few of my observation :

  • Page can be aligned to center. As of now, it looks more inclined to left.
  • Page behaves well within 375px screen size

  0
• 

  Nalish•40

  @NalishSubmitted about 2 months ago

  

  Creating a social links card

  1

  20
  

  Dharmendra Rathod•210

  @rathoddharmendraPosted about 2 months ago

  @Nalish, code looks really good. One suggestion to add responsiveness below 800px, as you can notice it;s behaving differently below this point. Keep up the good work!

  Marked as helpful

  0
• 

  Atakula Pranav Reddy•40

  @PranavReddy2703Submitted about 2 months ago

  

  Responsive webpage using CSS

  1

  40
  

  Dharmendra Rathod•210

  @rathoddharmendraPosted about 2 months ago

  Overall page looks really good.

  One suggestion that I can make regarding the responsiveness below 375px. -> You can use percentage instead for better responsiveness. https://github.com/PranavReddy2703/blog-preview-card/blob/main/style.css#L118C16-L118C22

  Marked as helpful

  0
• 

  Basma-Bensadi•20

  @Basma-BensadiSubmitted about 2 months ago

  

  first front-end mentor challenge

  1

  10
  

  Dharmendra Rathod•210

  @rathoddharmendraPosted about 2 months ago

  I really liked that you use multiple divs to manage each group better. Few of the improvements I noticed are:

  • Using sizing on image twice - once as img attribute and on img selector in CSS
  • missed to add color: grey on class="text"
  • suggestion: using relative sizing like percentage or rem is better for responsiveness overall than px

  As I said, overall the code works really good. I hope my review helps you :)

  0