@pikapikamart
Posted
Hey, great work. The content seems bigger but preferences right.
In terms of layout, desktop is fine and it resizes well when going in mobile state, thought it just get very narrow when almost reaching the breakpoint.
One concern is that, the learn more button, you should not use div
for that. Since that learn more is supposed to be a anchor to redirect a user, you should use a
tag for it and not div. Using a
tag will make your markup more semantic. Changing that will be really good
Apart from those mentioned, good work^
@c-tregunna
Posted
@pikamart Thank you. You are right, not sure how I missed using the <a>
tag. Will get that altered and look again at the layout, as realized it was a little bigger than intended.
Thank you for the feedback