krishna• 40
@coder483
Submitted
please text me the suggestions
Juan Pablo Arias Duque
@boudgnosis
All comments
- Juan Pablo Arias Duque• 320
@boudgnosis
Posted
@coder483 Hi! The hover effects you could put a transition so that the change is not so abrupt, also to the appearance of the filter and the small eye.
-
You should also add the main tag and an h1 to make the HTML a little more semantic.
-
You didn't add the title or the favicon, these are things that can be forgotten, but they should be there.
-
I recommend you not to apply styles inside the HTML since this is not done anymore nowadays and another recommendation would be to import the fonts to the HTML because if they go directly to the CSS they consume more resources since first you have to load the CSS file for the fonts to appear.
-
lastly you have a style for the H1 tag that is not being applied to anything, you should correct that.
I hope I have not been too rude in my recommendations, I am not a native English speaker, if you need help just ask.
Good luck with the corrections.
0 -
