Skip to content

Вторая ветка переделанная#2

Open
misha004 wants to merge 3 commits intomasterfrom
second
Open

Вторая ветка переделанная#2
misha004 wants to merge 3 commits intomasterfrom
second

Conversation

@misha004
Copy link
Copy Markdown
Owner

No description provided.

Comment thread css/style.css Outdated
line-height: 20px;
color: #4A4A4A;
padding: 10px 0 10px 50px;
margin-left: -20px;
Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

Старайся по минимуму использовать отрицательные маржины, я понял в чем у тебя возникала сложность. Как дашь права на репу, я пушну коммит, посмотришь, как можно было попробовать сделать иначе

Comment thread css/style.css
.features__list-item + .features__list-item{
border-left: 1px solid #DFE5EC;
}
input[type=radio] {
Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

Так лучше не писать, т.к. если бы были еще элементы input с type=radio, то ты бы их не видел, лучше создать отдельный класс для нужных тебе элементов и там прописывать стили.

Так же для скрытия достаточно написать одно св-во visibility или display, не надо оба сразу

Comment thread css/style.css

.features__list-item--link{
.filters-ticket-type__text {
display: inline-block;
Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

Не понятно зачем здесь это сво-во

Comment thread css/style.css
width: 167px;
}

input[type=radio]:checked + .filters-ticket-type__text{
Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

Как писал выше, после создания класса для радио, надо переписать

Comment thread css/style.css
transition: 0.5s;
}

.filters-ticket-type__text + input[type=radio] + .filters-ticket-type__text {
Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

Как писал выше, после создания класса для радио, надо переписать

Comment thread index.html
<a class="features__list-item--link" href="">Оптимальный</a>
</li>
</ul>
<section class="ticket-section">
Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

Класс ticket-section не используется

Comment thread index.html
</ul>
<section class="ticket-section">
<form class="filters-ticket-type">
<input class="filters-ticket-type__input" type="radio" id="option-one" name="selector" checked>
Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

Класс filters-ticket-type__input не используется

Comment thread index.html
</div>

<div class="ticket__text">
<div class="ticket__text-block">
Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

Класс ticket__text-block не используется

Comment thread css/style.css
.ticket__btn:hover{
.ticket__btn:hover {
background-color: #1e85da;
transition: .3s;
Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

Где то используется запись 0.3s, а где то .3s, это не критично, но, желательно писать одинаково

Comment thread css/style.css Outdated
.nav{
.transfers-filter {
width: 232px;
height: 252px;
Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

Высоту лучше не задавать блокам с контентом, т.к контент со временем может поменять, и контент будет вылазить за блок, что не совсем гибко

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.

2 participants