Szymon Rojek• 4,540
@SzymonRojek
Posted
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
