Skip to content

Add files - #2

Open
erofeevskiy-ia wants to merge 1 commit into
masterfrom
dz_2
Open

Add files #2
erofeevskiy-ia wants to merge 1 commit into
masterfrom
dz_2

Conversation

@erofeevskiy-ia

Copy link
Copy Markdown
Collaborator

ver 0.4

@DmitryMakhnev DmitryMakhnev left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Пофиксить с учётом разбора дз

Comment thread todo.html
</div>

<div class="todo-filter" data-filter="all" aria-label="Фильтр : показ всех">
<button class="todo-filter __active" > <!-- Модификатор??-->

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Достаточно одного модификатора, чтобы показать стейт. В остальные места прокидывать не надо

Comment thread todo.html
3 items left
</div>

<div class="todo-filter" data-filter="all" aria-label="Фильтр : показ всех">

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Почему нет общего контейнера?

Comment thread todo.html

<div class="todo-item"> <!--модификатор??-->
<div class="todo-item_ready-mark">
<input type="checkbox" id="todo-item_ready-mark3" class="todo-item_ready-mark" value="cb_html"

Copy link
Copy Markdown

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
}

.todo_footer_1 {
width: 444px;

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

нельзя хардкодить ширину, см разбор домашнего задания по css

Comment thread css/style.css
@@ -0,0 +1,242 @@

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

нужно сбросить стили на на используемые элементы, в том числе и на . см reset.scss в репе со стабом препроцессора

Comment thread css/style.css
overflow: hidden;
border:none;
background: inherit;
width:70%;

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

% !== честный адапитв
см https://cloud.mail.ru/public/JdWC/71ADpXe2W

Comment thread css/style.css



input[type=checkbox] {

Copy link
Copy Markdown

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
@@ -0,0 +1,242 @@

Copy link
Copy Markdown

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
}

.todo-item_remove{
-ms-filter: "progid:DXImageTransform.Microsoft.Alpha(Opacity=0)";

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Такие хаки для ie не нужны

Comment thread css/style.css

-webkit-border-radius: 50px;
-moz-border-radius: 50px;
border-radius: 50px;

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

width / 2

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