ABCDEF
1
р12.1. Форматирование@
2
 -Комментарии студентов к ответу → → →
3
-
4
type-
5
html-and-css.md
6
7
В пункте "БЭМ" (2.1 html-and-css-extended) необходимо всю структуру переформатировать? Т.е. убрать все id, присвоить в каждом блоке БЭМ классы, переиначить css. Если делать как надо, потребуется серьезно редактировать js-код. Или достаточно переименовать существующие классы?
8
В задании указано " навести порядок внутри файлов ... (только html и css файлы)". Но без небольшого редактирования js файла не обойтись. Не будет ли это нарушением?Не будет. В задании есть даже предупреждение, что это сделать нужно будет
9
font-family: 'Roboto', sans-serif;
10
Не будет. В задании есть даже предупреждение, что это сделать нужно будетМожно или нужно?
11
Нужно ли создавать ветку clean-code-s1e1 или правки вносить в master? Как сабмитить задание: ссылка на PR или деплой на gh-pages?в ветку и создать PR в мастер - ссылку на этот PR
12
Примеры в правиле 2.1 содержат 3 или 4 пробела в отступе, что не соответствует 1.1. Или 3-4 пробела в HTML нормальная практика?2 пробела - 1
13
Задание можно выполнять не по порядку следования пунктов в руководствах, а в том порядке, который удобно? Например, сначала пункт (и коммит) 3.1., потом 2.1.?можно в удобном порядке
14
Предлагаю заменить для начала на 'edit-mode' для единообразия.
15
Что делать с editMode?id для использования элемента JS, для стилизации - классы
16
стоит ли оставлять элемент Go Shopping в статусе editModeможнoА3ю11
17
вопрос по п 3.5 "отступы в блоках': Всегда ставьте отступы для любого содержимого в блоке. Это только когда блок в блоке или все css свойства всегда отделять отступом? ul {
font-size: 12px;
}

Отступы для правил внутри каждого css-блока
Блок в блоке тоже выделяется отступами
18
добавить class
19
в строке font-family: Lato, sans-serif; как записывать latoШрифты пишутся в соотстветствии с их именем в объявлении или же как они называются в бразере (н/п Times News Roman)В роли fantasy шрифта на скриншоте оригинала красуется Papyrus. Входит в состав Windows/Mac
20
а что насчет универсальных селекторов? их можно оставить? БЭМ + правило именования можно одним коммитом (так как одно покрывает другое)ччч
21
Нет пробела между селектором и фигурной скобкой button{ , 4 пробела после ">" в li > input[type="text"] --- какие это пункты правил?Read the Manual

Связующие операторы стилей тоже лучше выделять пробелами, но если вы не сделали - не ошибка, ибо в правилах этого не было		какой мануал ?
22
Нужны ли отступы для <head> и <body>?лучше делать
23
Нужно ли добавлять отступы между блоками в css?не обязательно
24
Подскажите, какой шрифт сюда вставлять?

.aaa .more_inf {
font-family: fantasy, cursive;
}		Оставьте тот, который стоит. Если на Linux едут "символы" - выберете любой на свое усмотрение.В роли fantasy шрифта на скриншоте оригинала красуется Papyrus. Входит в состав Windows/MacНиже этот же вопрос повторяется
25
1.1 и 3.5 - если отступы сделаны в 1.1, что тогда делать в 3.5? Или он про пустые строки?read the manual
1.1 про табы, если табов нет, то и не будет коммита с 1.1 (ничего страшного)		Что значит не будет коммита? Там же изначально нет никаких отступов, как может не быть коммита с 1.1?
26
В результате изменений в tag <p id='topSectionForAddingElementToList'> вложили <div class="task-row-wrapper">. В браузере ломается tag <p> и создается еще один пустой <p></p> со своими margin. В продвинутом уровне 1.1 Семантика <p> заменяю на <section>, макет немного съезжает. Вопрос: Надо ли подгонять макет стилями? можно "Надо ли подгонять макет стилями? " "можно" ??????
27
Доброй ночи, такой вопрос: отредактировала отступы и сделала два коммита по этому пункту, но в коммите забыла указать номер пункта. Нужно теперь изменить названия коммиттов или можно продолжить дальше и в следующих коммитах уже писать номер пункта, по которым делала изменения?Git Graph
28
Я правильно понял, что нарушение функциональности это 0 баллов за задание?если работало и сломалось после изменений студента, то да
29
я все изменения выпонил в ветке мастер. могу ли я создать ветку gh-pages для деплоя и сделать пул реквест оттуда в мастер, не будет ли это ошибкой? не хочется переделывать с нуляСоздайте ветку из мастера (
git checkout -b clean-code-s1e1 master (убедитесь, что тут есть все коммиты!)
git push origin clean-code-s1e1
git checkout master
git reset --hard HEAD~n , где n - число коммитов необходимое удалить
git push origin master -f		Огромное спасибо!
Получилось!!!!! 		Спасибо огромное, получилось. Только не понял почему не давал удалить сразу все коммиты из masterблагодарю, сразу и не поняла, что коммиты не там рамещала. всё получилось
30
Ещё одна группа вопросов по требованию "В одном коммите правки по одному пункту правил", но в разрезе оценивания работы. Влияет ли несоблюдение этого пункта на оценку? В критериях не увидел этого. Если влияет, то как (насколько в баллах, какова формула)? Если влияет, то это означает, что ревьюерам нужно будет проверять это требование? Если влияет и уже попортил историю коммитов публикацией в удалённом репозитории коммита со "смешанными" правками, то как быть?Это требование для зачета.
Смотри ответ выше
31
Смежный вопрос по требованию "Один пункт на один коммит". В задании есть пункты по отступам и по пробелом/табам, которые по идее решаются автоматически на уровне IDE/форматтера (Prettier). Собственно вопрос (вопросы) звучат так: нужно ли пересаживаться с условного VS Code на условный блокнот (отключать Prettier) для выполнения задания и во избежание того, что автоформаттер автоматом решит несколько пунктов за раз (т.е. сделает невозможным выполнение пункта "В одном коммите правки по одному пункту"?id для использования элемента JS, для стилизации - классы
32
33
Смежный вопрос по требованию "Один пункт на один коммит". В задании есть пункты по отступам и по пробелом/табам, которые по идее решаются автоматически на уровне IDE/форматтера (Prettier). Собственно вопрос (вопросы) звучат так: нужно ли пересаживаться с условного VS Code на условный блокнот (отключать Prettier) для выполнения задания и во избежание того, что автоформаттер автоматом решит несколько пунктов за раз (т.е. сделает невозможным выполнение пункта "В одном коммите правки по одному пункту"?Один из краеугольных камней таска. Необходимо стремиться к выполнению условия "один пункт на один коммит", иначе обязательно найдутся проверяющие, которые снизят баллы за невыполнение этого требования. В начале таска так и написано: "На данном этапе не предполагается использование инструментов по автоматическому форматированию кода". То есть не следует использовать автоформатеры, Prettier сделать disable на время выполнения таска. Всё править ручками, выполнять ТОЛЬКО ТО, что требуется в каждом пункте, тогда не будет конфуза после уже сделанных дюжины коммитов.
34
Здравствуйте, есть несколько вопросов:
35
1). p > button:hover { color: #0fc57c } не работает так как сверху есть button:hover, чинить?Можете, если хотите :)
36
2). нет правила для удаления пустых строк в блоках cssок
37
3). li > input тоже не подпадает под правило (пробелы перед input)
38
4). нет правила для отступа между блоками в cssок
39
5). Перевести в БЭМ нужно только существующие классы? или надо переделать структуру полностью по БЭМ? Тогда css тоже переписывать под БЭМ?Как считаете нужным
40
6). id="new-task-input"class="task", правила для добавления пробела тоже нет, делать? И что тогда в коммите писать? Тут вообще вопрос, допустимы ли коммиты не по пунктам задания?Да, допустимы. Главное именуйте правильно
41
Fcommi
42
редактировать js можно
43
есть нюанс с js всеравно. мы в js создаём элемент через createElement("input")
мы к этому инпуту не вяжем никакой класс через js. в итоге если мы с css удаляем все селекторы по имени тега, наш новый созданный элемент из js не получит никаких стилей. нюанс
тут или оставлять селектор имени тега или знатно менять js. нужно добавлять через классЛист, ремувать, это тема этого задания?)) как делать дальше - не ясно)
44
есть нюанс с js всеравно. мы в js создаём элемент через createElement("input")
мы к этому инпуту не вяжем никакой класс через js. в итоге если мы с css удаляем все селекторы по имени тега, наш новый созданный элемент из js не получит никаких стилей. нюанс
тут или оставлять селектор имени тега или знатно менять js. нужно добавлять через классЛист, ремувать, это тема этого задания?)) как делать дальше - не ясно)		Да
45
в рулах написано: Избегайте использование имен классов или идентификаторов с селекторами тега элемента.
дальше говорится о том что допускается оставление селекторов тега элемента в некоторых ситуациях.
в форме для проверки сказано в требовании к пункту : теговые селекторы не используются
не соответствие. я оставлю - при проверке балл снимут. 		Да
46
Можно ли в форму для кросс-чека добавить больше отдельных конкретных пунктов для штрафных баллов, вместо одного ссылающегося на сам таск?Принято
47
Для выполнения пункта по БЭМ весь html и соотвестсвенно связанные классы в css и js перебивать или достаточно пару основных блоков перевести в БЭМ?весь
48
как уйти от li > * в соответствии БЭМ? просто наклепать классов для li-шек и для каждого из них поочереди задавать классы? после чего при экшене - ремувать и добавлять другие?
49
деплой не нужен будет, в Cross-Check Submit , будем добавлять ссылку на пулреквест?Да. Деплой можно делать, но сдавать нужно все равно ссылку на пул-реквестУважайте проверяющих и облегчайте им жизнь. И они к вам тоже с понимание отнесутся :)
50
Можно ли увидеть изменения в тз, по поводу бэм, что раз я начиная с пункта 3.1. использую бэм то мне не нужно комитить все остальные пункты 3.2 3.3 продвинутый 2.1Но они должны быть описаны в коммите с БЭМ правилом
51
Как правильно проверять это правило "В одном коммите - одно правило"? Нужна подробная инструкия для ревьюеровПо сообщению в коммите (там же описан покрываемый пункт правила)
52
Вопрос на будущее: как оценивать если отвалилась часть функционала?Так как это указано в ТЗ. В текущем задании, если что-то отвалилось - 0 баллов
53
А что насчет универсальных селекторов? их можно оставить?
54
Является ли ошибкой делать отступ для строчных элементов внутри блочных? Например при конструкции вложенности div > a? Просто это в разы улучшает читаемость кодаНет, не является) Можете добавить
55
Как выполнить пункт задания 1.1 ? В файле с html вообще нет никаких отступов, а в css при замене табов на 2 пробела VSCode проставляет 4 пробела (делает 2 отступа вместо одного). А гит вообще не видит этих изменений. Что коммитить ? Руками все исправлять даже не думаю, это не путь программиста и не этому это задание должно нас научить.Если на правило нет случаев, то не будет на него коммита. Это нормЕсли правильно понял вопрос, в settings ставим Editor: Detect Indentation отключить галочку, Editor: Insert Spaces поставить галочку, shift+tab удаляем табы, потом ставим таб(то есть 2 пробела по нашим настройкам)
56
Пункт 1.2 пример
<!-- Recommended -->
<img src="google.png" alt="Google"> С ваших слов Google должно быть с маленькой. Так же не ясно, что делать с атрибутом value в инпут. Если в примере нет ошибки, то оставлять, если нет, то прописывать ловер кейс и как-то менять на капиталайз значения через цсс или js		value - то, что будет видеть User. Это не служебное значение. Оставляйте с большой
57
Правило '2.4 Атрибут 'type'' из html-and-css выполняется во всем проекте: CSS-стили и JS-скрипты подключены без использования данного атрибута. Нужно ли удалять "type" в <link href="https://fonts.googleapis.com/css?family=Lato:300,400,700" rel="stylesheet" type="text/css">, ведь подключается не CSS файл, а CSS просто присутствует в url?Подключается css файл
58
Можно ли строчные элементы разделять, пернося на новую строку? (в рамказ п.2.1)
<input type="text" value="go shopping" class="task"><button class="edit">Save</button><button class="delete"><img src="./remove.svg"></button> - наверно не очень читабельна?		можно
59
Нужно ли оставлять ли type="text/css" для подключаемых шрифтов? Чем руководствуемся убирая type для css/js файлов и оставляя для шрифтов?
60
Задание ещё будет правиться, или это уже окончательная версия?новых требований точно не будет. Только что уточнения
61
Уточните пожалуйста название ветки в которой выполняем задание, название ветки в которую делаем PR, что сабмитится - ссылка на репозиторий, PR, деплой, нужно ли мержить коммиты?Делаем в мастер или другую главную ветку. Название ветки с заданием - в описании задания
62
Потому что не обязателен.
63
Для пункта 1.1 сделал отступы в два пробела все что находилось внутри html. Внутренним тегам форматирование не делал. Считается ли это за выполнение пункта 1.1?
P.S. форматирование/разбиение на блоки сделал в пункте 2.1
Скрин по пункту 1.1: https://prnt.sc/10wullv		1.1 про использование tab-ов
64
Пункты 3.1, 3.2 и 3.3 начального уровня все покрываются БЭМ - означает ли это, что их можно разместить одним коммитом? Да
65
Если один коммит буден назван не согласно правилам, т.е. в начале нет refactor или fix, остальное написано правильно, могут ли за это снять все баллы?Да
66
Можно ли при выполнении задания по методологии БЭМ использовать только блоки и модификаторы блоков?Если сможете обосновать такое решение. При этом натягивать правило там, где оно не применимо, точно не стоит
67
Если после ренейма селекторов, верстка поехала на 5-10px, считается за ошибку?В части семантики мы должны поменять некоторые теги на более подходящие, и к примеру label в блоке add task выглядит как другие заголовки а они являются h3 , но после переименования верстка сдвигается за счет того что у этого label есть свой отдельно прописаный отступ, можно ли подкорректировать стиль? или мы только с названиями работаем
68
можно попросить видеоинструкцию по проведению кроссччека таска https://rolling-scopes-school.github.io/stage0/#/stage1/tasks/clean-code/clean-code-s1e1
69
селектор p > button:hover не работает. Это ошибка?Моя ошибка. Можете не исправлять
70
pay bills в значении "Edit" слетает
71
Подскажите, какой шрифт сюда вставлять?

.aaa .more_inf {
font-family: fantasy, cursive;
}		Оставьте тот, который стоит. Если на Linux едут "символы" - выберете любой на свое усмотрение.
72
Если настроить WebStorm согласно всем критериям задания и нажать cmd opt + l, то сразу будет выполнено 3 или 4 пункта. Задание после такого не будет принято? Оно подразумевает ручную работу? Не жедательно в ходе него использовать IDE? Для пункта 1.1 сделал отступы в два пробела все что находилось внутри html. Внутренним тегам форматирование не делал. Считается ли это за выполнение пункта 1.1?
P.S. форматирование/разбиение на блоки сделал в пункте 2.1
Скрин по пункту 1.1: https://prnt.sc/10wullv
73
в пункте 3.3 Лаконичность названий идентификаторов и классов, указано:
Для идентификаторов и классов используйте настолько длинные имена, насколько нужно, но настолько короткие, насколько возможно.

У меня возникает, вопрос по части того, что оценка по этому пункту будет крайне субъективной. Так как нет четкого критерия приемки данного пункта.
Зная себя, во время проверки чужих работ - в моменте я не всегда могу продумать все возможные варианты короткого или длинного названия для класса или айди. А как заметили выше: "Задача не может быть принята, если все условия выше не выполнены".

И тут как по мне появляется поле для самодеятельности. Чего при оценке я бы избегал.
Как быть?)		С удовольствием, но как объяснять свою позицию Student 1 ?
74
/Не будет.
75
Можете. Тогда будет коммит без ссылки на правило. Это не запрещено
76
c1Можно
77
Помогите разрешить противоречие: в задании дан скриншот внешнего вида приложения (синие чекбоксы и ссылка определенного вида). Если я сейчас до всех изменений (нулевой вариант из задания) открываю приложение в файерфоксе, то ссылка соответствует заданию, а чекбоксы не соответствуют. Если открываю в хроме, то, наоборот чекбоксы соответствуют, а ссылка не соответствует. Что мы в итоге должны получить на выходе?Сделайте на свое усмотрение. Можете привести приложение к достойному виду, максимально близкому к изначальному. pixel-perfect не требуется
78
Привет!) Выполняя задание 2.1 extended (БЭМ) затрагиваются пункты 3.1. единый стиль, 3.2. значимые наименования, 3.3. лаконичность. На стриме говорилось, что можно в данном случае несколько пунктов, если речь про БЭМ, но в условии прописано, что можно только ДВА. Если сделать все рационально (один раз, выполняя один пункт - БЭМ), сразу же подправить JS, это допустимо? Эти пункты ведь тесно связаны один с другим. Окончание commit message: … and BEM methodology in HTML and CSS according to rules 3.1, 3.2, 3.3 in html-and-css.md and rule 2.1 in html-and-css-extended.md, fix JS to keep working with new classes namesИ три можно, если это обоснованно. Я не делала упор на числе два, а упор был на том, что вы сможете обосновать это на ревью.
79
Пул реквест в master делать свой или владельца репозитория??Работа веделся в СВОЕМ репозитории. В нем создаете ветку clean-code-s1e1 и PR делаете в master тоже своего репо
80
Можно ли ревертить коммиты?можно
81
Стоит ли переделывать всю задачу, если сделал всё в ветке masters?даНе обязательно, можно создать новую ветку из местер, так чтобы перенеслись все коммиты и в ветке мастер удалить все лишние коммиты
82
Для пункта 1.1 сделал отступы в два пробела все что находилось внутри html. Внутренним тегам форматирование не делал. Считается ли это за выполнение пункта 1.1?
P.S. форматирование/разбиение на блоки сделал в пункте 2.1
Скрин по пункту 1.1: https://prnt.sc/10wullv		создавать дополнительные html элементы можно
83
после выполнения заданий 1.1 и 2.1 продвинутого уровня, некоторые селекторы в css изменились, а некоторых просто не стало. Можно ли "прибраться" в css (упорядочить селекторы) для удобочитаемости в п.2.1? можно
84
Pay bills в значении "Edit" слетаетНе повторяйте вопросы. Слетает - поправьте
85
Добрый день, практически полностью выполнил задание. Весь функционал приложения работает как и в исходном варианте.
Однако, в связи с изменением кода внешний вид приложения немного изменился (в основном это касается вертикальных расстояний сверху и снизу от input-ов). Опытным путем выяснил, что это, вроде как происходит из за того что к исходному коду добавляется некий "user agent stylesheet", к моему же коду он уже не добавляется.

Собственно напрашивается логичный вопрос, должен ли я редактировать исходные стили чтобы подогнать макет пиксель в пиксель к исходнику?
Будет ли считаться ошибкой если я оставлю все как есть?

UPD: Ради интереса посмотрел визуально пару работ других учеников, и ни у одного пиксель в пиксель макет не сошелся. Исходя из этого думаю данный вопрос будет волновать не только меня одного.		Нет, если выглядит норм - не редактируйтеБлагодарю за ответ, хорошего вам дня :)
86
Добрый день, практически полностью выполнил задание. Весь функционал приложения работает как и в исходном варианте.
Однако, в связи с изменением кода внешний вид приложения немного изменился (в основном это касается вертикальных расстояний сверху и снизу от input-ов). Опытным путем выяснил, что это, вроде как происходит из за того что к исходному коду добавляется некий "user agent stylesheet", к моему же коду он уже не добавляется.

Собственно напрашивается логичный вопрос, должен ли я редактировать исходные стили чтобы подогнать макет пиксель в пиксель к исходнику?
Будет ли считаться ошибкой если я оставлю все как есть?

UPD: Ради интереса посмотрел визуально пару работ других учеников, и ни у одного пиксель в пиксель макет не сошелся. Исходя из этого думаю данный вопрос будет волновать не только меня одного.		Можнопосмотрите сразу пункт 2.5, возможно надо будет объединить пункты в коммите. Перенесла при этом, потом увидела, что в 2.5 написано..
87
 Нет, конечнo
88
подскжите, допустимы ли комиты такой формулировки(пример из задания рекомендательный?):

refactor: apply 1.2 альтернатива для мультимедиа (продвинутый уровень)
refactor: apply 1.1 отступы		Все должно быть на английском и согласно правилам составления коммита, остальное - ваша фантазияКажется в задании забыли упомянуть что все должно быть на английском....
89
 По-хорошему надо, потому что guidelines несколько файлов. Но если по соержанию коммита легко определить, к какой доке он относится, то ничего страшного (думаете с позиции удобства проверки ревьюерам)
90
Помогите разобраться: есть у нас input'ы и label'ы с классом task. Логика ясна - если у нескольких элементов одинаковые стили, то во избежание дублирования даем этим стилям один класс и просто добавляем к нужным элементам. Но по БЭМ, получается, мы должны каждому инпуту и лэйблу добавить свой класс относительно блока, где они находятся, и будет куча разных классов с одними и теми же стилями?
91
Нужно ли форматировать JS-файл, а именно: добавлять пробелы в присваиваниях, избавляться от var, убирать/добавлять отступы и т.д.?JS-файл не проверяется, но форматировать его очень желательно, так как работать с ним всё-равно придётся, а форматированный он воспринимается гораздо легче.
92
Подскажите как быть с именованием классов для кнопок edit и delete?
Мы присваиваем всем кнопкам в разных блоках общий класс button, чтобы избавиться от тегового селектора. Дальше нужно отдельно прописать класс для кнопки edit и delete. Эти кнопки не являются элементами для нашего блока button, по идее они элементы блока li (в моем примере у li class="item"), в котором лежат. Тогда теоретически класс для кнопок будет выглядеть так:
class ="button item__edit-button"
class ="button item__delete-button" - Это не по БЭМу
Они так же не являются модификаторами блока button, потому что это 2 разные кнопки, а не модификация одной кнопки,
соответственно записать как ниже, мы тоже не можем:
class ="button button_edit-button"
class ="button button_delete-button"
Подскажите как быть или где неправильно поняла
93
Подскажите как проверять задания в части БЭМа? Каждый сделает как понял и будут споры относительно того можно было оборачивать в блок или надо было сделать элементом, или модификатором и т.д. и соответственно баллы будут снижаться исходя из того, как чловек понял БЭМ
94
При обновлении задания, появился тег p который добавляет сам браузер, и соответственно у него есть свои размеры, и как с этим быть, в версии где тег p больше не является оберткой ? У него там margin: 16px
95
Финальный вариант должен выглядеть, как форкнутая версия или же как скриншот, ибо там как минимум отличается шрифт у ссылки?
96
Подскажите пожалуйста,
[type="checkbox"] {
margin: 0 10px;
}
это теговый селектор? (Ведь тег не указан)		[type="checkbox"] - это селектор атрибута (attribute selector)
97
По поводу атрибута alt для кнопки Delete. В моем понимании, если стили не загрузятся, то пользователь увидит пустую кнопку (т.к. текста в ней нет), а что с ней делать - не ясно. Значит в данном случае необходимо написать alt="button Delete". Правильно я понимаю?
98
В ТЗ есть: "Изучите и используйте максимально возможности вашей IDE для рефакторинга (ниже есть ссылки для VS Code)" - ссылок нетСсылки в другом файле. Вот она https://code.visualstudio.com/docs/editor/refactoring
99
Есть два вопроса: 1. обязательно ли менять уникальный теговый селектор body на класс .body, 2. Если задание уже было сделано полгода назад, не будет ли ошибкой, что коммиты такие древние и можно ли добавить коммит после существующих как дополнительные правки по какому-либо критерию
100
Нет, штрафные баллы не ссуммируются, минимальный балл не может быть ниже 0.