@ApplePieGiraffe
Posted
Greetings, yet again, Raymart Pamplona! π
Sorry, I've missed a couple of your last few challenges (I've just been a little busy lately)! π I just saw this one and came to say well done on this challenge! π Overall, everything looks pretty on point! π
One thing I noticed, though, is that the image of the desktop computer in the "Keep Track Of Your Snippets" section gets very squeezed in the desktop layout as the width of the screen decreases. Making sure that that image retains its original aspect ratio (to prevent it from distorting when its size changes) would be a good idea. π
And one other super tiny thingβIDK why, but the Facebook icon in the footer of the page doesn't seem to change its fill color and keep it when hovered over (like the other icons next it). π§
That's all! π
As alwaysβkeep coding (and happy coding, too)! π
@pikapikamart
Posted
@ApplePieGiraffe Thank you for reviewing :>>. That's fine being busy and since I saw in the article that you are working here now in FEM, that is really something and congrats to you for it^^
Yes hahaha well I decided to fixed the height of the image, I don't why but it can be refactored though and for the footer icon, well hahaha the reason is that, the I declared the hover state on the path hahaah that is why when you hover on it, on the hollow part of the svg, it will scale but not colored, because it is declared in path.
Thank you for reviewing and will refactor that later once I finished my last FEM challenge ^^
@ApplePieGiraffe
Posted
@pikamart
No problem! π Keep it up! π