@PhoenixDev22
Posted
Hello @d4wk0m ,
I have some suggestions regarding your solution :
-
It's better to have the styling in a separate file.
-
you need to add an interactive element around the image. Anything with a hover style in a design means it's interactive.
-
Section is not meant to be used anytime . section is for a bigger chunk of content often titled by <h2>.
-
For any decorative images, each
img
tag should have empty alt="" andaria-hidden="true"
attributes to make all web assistive technologies such as screen reader ignore those images as inalt="Icon clock" alt="Icon View"
,alt="Ethereum"
-
the eye image doesn't really need to be in the html, you could do it with css. If you want it to stay in html it needs to be aria-hidden or role presentation with empty alt.
-
you can use an unordered list
<ul>
forid="tabela">
and in list item<li>
there would be<img>
and<p>
.
I would do for the avatar's part:
<div class="paragraph2 ">
<img src="./images/image-avatar.png " class="author" alt="Avatar">
<p > Creation of <a href="#">Jules Wyvern</a></p>
</div>
-
never style on IDS. That's not what they're for.
-
You should use
em
andrem
units .Bothem
andrem
are flexible, scalable units. -
Never have font size in
px
. Usingpx
won't allow the user to control the font size based on their needs.
Hopefully this feedback helps.
Marked as helpful