Skip to content

Conversation

@DenysStrilchuk
Copy link
Owner

No description provided.

@DenysStrilchuk DenysStrilchuk marked this pull request as ready for review June 30, 2022 11:23
@abalanuk
Copy link
Collaborator

abalanuk commented Jun 30, 2022

Please move all images into assets folder and do not forget to update references/paths to images inside css file

index3.html Outdated
<img src="assets/logo.jpg">
</div>
<div class="menu">
<span>Дерево</span>
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Please try with ul+lis

Copy link
Owner Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

done

…sition: relative' style to position child relatively to his parent container.Refactoring header with with ul+li.
index3.html Outdated
<h4>
<a href="">Домашня сторінка</a>
</h4>
<h5>
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This is not correct. Do not use h5 as a wrapper for a bunch of other elements

Copy link
Collaborator

@abalanuk abalanuk Jul 21, 2022

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Look how they do it: div -> ul -> li -> a
Here the link/anchor is not by semantic meaning is really heading, so why to wrap it by h4, just add for instance one class to add styles for all "first-items" in each column: Домашня сторінка, Про нас, Пидтримка

Copy link
Owner Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Done

@DenysStrilchuk DenysStrilchuk requested a review from abalanuk July 21, 2022 17:51
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants