All comments

• 

  Mirjax2000• 680

  @Mirjax2000

  Submitted 6 months ago

  What are you most proud of, and what would you do differently next time?

  1. Here i have to think how to prepare HTML for breakpoint switching.
  2. Playing with flexbox. That was fun. I tryed some things to do without flex box and i made tons of code and the just one parrametr in flex box and it works as well. One line of code. Nice.

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

  Thinking ahead. Write some notes before started.It takes time but in the end it is shorter if you stick with the plan.

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

  Still BEM naming. Id like to usem better in the SASS with & parrent selector. Still strugelling with naming classes in HTML. Parrent of parrent makes me trouble.

  

  Four_card_feature - responsive until 200px

  1

  30

  

  Oliver Villalobos• 90

  @olivervillalobos

  Posted 4 months ago

  I would suggest to read an article about when to use sections and articles, the cards are components that you could place anywhere on a given website, so using articles is not the best choice (This is my opinion), I liked the approach you used of dividing it in three sections (left,center,right).

  Great Job!

  Marked as helpful

  0
• 

  Linh Nguyen• 150

  @auri222

  Submitted 4 months ago

  What are you most proud of, and what would you do differently next time?

  • Using basic grid CSS and flexbox
  • Responsive with media queries in CSS

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

  I have some problems with image width and height which can't fit in the div if I set the width a little bit wider. For example, if I set the width to 350px the image will flow out of the div. So my solution here is to stick with 300px width.

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

  I need to learn more about grid CSS and how to control the image better.

  

  Responsive product preview card with HTML and CSS

  1

  50

  

  Oliver Villalobos• 90

  @olivervillalobos

  Posted 4 months ago

  As what I can see, everything looks perfect, the only thing I would change is the font size units, change it to rem or em units, better if you use calc.

  0
• 

  Engr. Fisayo Michael Animashaun (MNSE, FMP®, COREN)• 330

  @fmanimashaun

  Submitted 5 months ago

  What are you most proud of, and what would you do differently next time?

  I am quite excited that I can produce the design relying only on the design picture without any Figma file. I am close as possible to the intended design with few off measurements due to approximation.

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

  I was a bit thrown off by the list-type alignment but I was able to use before pseudo selector to handle that. I am learnt a new thing about using custom list-style

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

  none

  

  Recipe page with SASS and BEM

  1

  60

  

  Oliver Villalobos• 90

  @olivervillalobos

  Posted 4 months ago

  Check sizes, do not ise pixels, responsive is a little off.

  0
• 

  irene• 120

  @irene-panis

  Submitted 4 months ago

  What are you most proud of, and what would you do differently next time?

  most proud of utilizing min, a css function i haven't really used before

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

  putting anchor tags within list items while making the whole list item clickable - had to look this up. also read up on min() because i wanted the card to be slightly wider but be able to shrink on mobile.

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

  i styled the list items to look like buttons then put anchor tags inside them w/ display block and height/width 100% to make them clickable; im pretty sure this is better than using buttons but lmk if i'm wrong and also if the usage of the list was necessary at all

  

  Social links profile using flexbox and min

  2

  40

  

  Oliver Villalobos• 90

  @olivervillalobos

  Posted 4 months ago

  Nice job!!!

  The only thing I suggest is to add a minimum width. If you keep the default font size, which is 16px on most browsers, it's totally fine. However, if you modify it, for example, to 32px for the browser's font size, when you inspect the code and reduce the width size, you will see a bunch of boxes shrink and behave inappropriately.

  0
• 

  CodeCuriousCat• 20

  @ANTHEIAM

  Submitted 4 months ago

  What are you most proud of, and what would you do differently next time?

  I learnt about clamp() function, first time I make the fonts responsive.

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

  The flexbox again, I still don't know when to use align-items, justify-content ...

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

  Nothing for now.

  

  Blog preview card with flexbox and clamp()

  2

  50

  

  Oliver Villalobos• 90

  @olivervillalobos

  Posted 4 months ago

  Hey first of all nice work!

  index.html File:

  1. I would remove the

  0
• 

  Botan Cosar• 10

  @Botan-Cosar

  Submitted 4 months ago

  What are you most proud of, and what would you do differently next time?

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

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

  

  QR code component

  2

  60

  

  Oliver Villalobos• 90

  @olivervillalobos

  Posted 4 months ago

  Nice solution.

  0