Mobile first using grid "Four card feature section".

Community feedback

• 

  Szymon Rojek•4,540

  @SzymonRojekPosted almost 4 years ago

  Hi Hagureshinma,

  Very nice solution, I like your RWD and effect on :hover :D

  I have checked your HTML, a few tips from me:

  • the text at the top it will be good to put inside of the header tag;
  • under the header you can create the main tag and inside these 4 divs;
  • using two h1 it is not a good practice, only one per page => also we can not skip the headings, we have to put them gradually;
  • I think you don't have to use ul > li between 4 boxes, split them => one div for each box;
  • inside of these div you can use the h2 and paragraph plus img;
  • I'd recommend learning BEM naming convention (at the moment your classes are not readable and descriptive);
  • add transition on :hover.

  Greetings :D

  1