Jani-B• 270
@Jani-B
Submitted
P
Karen L Byrd
@klb-dev
All comments
- PKaren L Byrd• 80
@klb-dev
Posted
Looks amazing very similar to the original design.
0 - Monica I.• 200
@monicaivanov
Submitted
What are you most proud of, and what would you do differently next time?
This was not a difficult project but it required a lot of change in the properties as the design had different details. Im proud I did find a solution to all and Im happy with the result. Hopefully as I keep learning I will improve my code solutions.
What challenges did you encounter, and how did you overcome them?
I had to remind myself the list and tables properties (something I rarely use). w3school has saved me many times.
What specific areas of your project would you like help with?
Any suggestions on what I could have done better or simpler I will greatly appreciate. Thank you.
PKaren L Byrd• 80@klb-dev
Posted
Your page looks very sleek. You followed most of the design, just some color of the content. You have black, but I believe the color is stone-600 (hsl(30, 10%, 34%). It just stands out in black but does not look horrible. Great job, keep it up.
0 - Alex Camacho• 245
@alexcamachogz
Submitted
PKaren L Byrd• 80@klb-dev
Posted
I love that you added your picture and location. The code looks sleek and works well.
0 - Tharun kumar S• 100
@Tharunkumarsubburaj
Submitted
PKaren L Byrd• 80@klb-dev
Posted
Great job on the design and coding so far. Your efforts are evident and appreciated. The code is clean, but why not use <button> or <a> for the learning button? You placed it in <p>. The heading should be an <h1> as well. These tags are done for screen readers and people with visibility issues. Your avatar image does not show up even though it looks correct. Placing a '.' before the first '/' would make it work.
CSS looks good; maybe start with mobile first and then have fewer media queries to produce more DRY code—great job with styling.
Marked as helpful
0 - Eslam Mohamed• 190
@Eslam-Mohamed-G
Submitted
PKaren L Byrd• 80@klb-dev
Posted
NA
0
