P
Karen 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
Tharun kumar S• 100
@Tharunkumarsubburaj
Posted
@klb-dev thanks for your comment i will correct them in the upcomming project and on this also
0
