Open
Conversation
innabelaya force-pushed the dist-qs branch 3 times, most recently from 0920cda to 6b47f4f Compare 
Collaborator Author
| cc @godfreyd |
Contributor
| @innabelaya , привет. Запланировал, посмотрю. |
godfreyd reviewed Jun 6, 2018
| Этот урок включает в себя: | ||
| | ||
| > **Важно!** Данный вариант поставки не позволяет использовать все преимущества БЭМ-проекта: [уровни переопределения](../../../method/key-concepts/key-concepts.ru.md#Уровень-переопределения), [миксы](../../../method/key-concepts/key-concepts.ru.md#Микс) и возможность [точечной сборки проекта](../../../method/build/build.ru.md#Определение-списка-БЭМ-сущностей). Для максимально эффективного использования библиотеки, воспользуйтесь поставками [source или compiled](https://ru.bem.info/platform/libs/bem-components/6.0.0/#source-compiled). | ||
| * [Быстрый старт](#Быстрый-старт). Как подключить библиотеку и вставить блок на страницу |
| | ||
| > **Важно!** Данный вариант поставки не позволяет использовать все преимущества БЭМ-проекта: [уровни переопределения](../../../method/key-concepts/key-concepts.ru.md#Уровень-переопределения), [миксы](../../../method/key-concepts/key-concepts.ru.md#Микс) и возможность [точечной сборки проекта](../../../method/build/build.ru.md#Определение-списка-БЭМ-сущностей). Для максимально эффективного использования библиотеки, воспользуйтесь поставками [source или compiled](https://ru.bem.info/platform/libs/bem-components/6.0.0/#source-compiled). | ||
| * [Быстрый старт](#Быстрый-старт). Как подключить библиотеку и вставить блок на страницу | ||
| * [Руководство](#Руководство-Работа-с-блоками-библиотеки). Как работать с блоками библиотеки |
| Откройте файл `hello.html` в браузере. | ||
| | ||
| Чтобы добавить блок на страницу, необходимо: | ||
| ## Руководство. Работа с блоками библиотеки |
Contributor
There was a problem hiding this comment.
Давай уйдем от двух предложений в заголовке.
| Зайдите в описание блока на сайте [bem.info](https://ru.bem.info/platform/libs/bem-components/6.0.0/) и выберите подходящий пример: | ||
| * [input](https://ru.bem.info/platform/libs/bem-components/6.0.0/touch-phone/input/#Модификатор-type-10) | ||
| * [button](https://ru.bem.info/platform/libs/bem-components/6.0.0/touch-phone/button/#Кнопка-отправки-формы-модификатор-type-в-значении-submit-1) | ||
| В этой части урока мы создадим форму приветствия, подключим в нее блоки `input` и `button` и научим их взаимодействовать друг с другом: имя пользователя при нажатии на кнопку будет отображаться в приветствии. |
Contributor
There was a problem hiding this comment.
- «В этой части урока мы создадим форму приветствия» — вводная неконсистентная с вводной быстрого старта. Для меня было открытие про часть урока, я пошел искать предыдущую, а ее явно нет.
- «научим их взаимодействовать друг с другом» — давай не будем одушевлять блоки и избавимся от местоимения.
- мы уже обсуждали с тобой на внеклассном чтении, что здесь есть явное нарушение последовательности. Откуда взялось имя пользователя (какого пользователя, пользователя чего, где о пользователе сказано ранее)? Почему имя должно отобразиться при нажатии?
| ### Вставьте блок на страницу | ||
| | ||
| Чтобы подключить скрипты и стили библиотеки в проект, скопируйте ссылки на бандлы в теги `<link>` и `<script>`: | ||
| * Откройте описание блока (напрмер, [input](https://ru.bem.info/platform/libs/bem-components/6.0.0/touch-phone/input) на сайте [bem.info](https://ru.bem.info/platform/libs/bem-components/6.0.0). |
Contributor
There was a problem hiding this comment.
- Последовательные действия оформляются нумерованным списком https://support-testing.qloud.yandex.ru/support/paradigm/tags/lists.html#lists__ol.
- Наверное, не стоит вставлять версию библиотеки в URL, есть риск, что в версиях выше может что-то измениться.
- напрмер → например.
- «на сайте bem.info» — пояснение можно убрать, а ссылку поставить под «описание блока».
| modules.require('i-bem-dom__init', function(init) { init(); }); | ||
| Чтобы проверить, что блоки `input` и `button` отобразились на странице, откройте файл `hello.html` в браузере. | ||
| | ||
| ### Добавьте блокам функциональность |
Contributor
There was a problem hiding this comment.
- +1 про глагол.
- Что значит «функциональность» непонятно.
| ``` | ||
| | ||
| Чтобы изменять поведение блоков, используется фреймворк [i-bem.js](https://ru.bem.info/platform/i-bem/), который является частью библиотеки [bem-core](https://ru.bem.info/platform/libs/bem-core/4.2.0/). Чтобы писать код на i-bem.js, дополнительно подключать на страницу предсобранные бандлы библиотеки `bem-core` не нужно, они уже включены в сборку. | ||
| Чтобы изменять поведение блоков, используется фреймворк [i-bem.js](https://ru.bem.info/platform/i-bem/), который является частью библиотеки [bem-core](https://ru.bem.info/platform/libs/bem-core/4.2.0/). Дополнительно подключать его на страницу не нужно, предсобранные бандлы библиотеки `bem-core` уже включены в сборку. |
Contributor
There was a problem hiding this comment.
- «Чтобы изменять ..., используется» — здесь все верно?
- «уже включены в сборку» — в какую сборку?
| Проект в [JSFiddle](https://jsfiddle.net/inna__neige/0hLmLmzn/). | ||
| | ||
| ### Генерация HTML в браузере | ||
| ## Альтернативный способ подключения блоков |
Contributor
There was a problem hiding this comment.
Это что за раздел? Изначально было заявлено про быстрый старт и руководство.
| Проект в [JSFiddle](https://jsfiddle.net/inna__neige/df6uuw7u/). | ||
| | ||
| ## Результат | ||
| ## Какой способ лучше? |
Contributor
There was a problem hiding this comment.
Есть знак вопроса, см. следующий раздел.
| Если в новой версии библиотеки обновленные шаблоны изменят разметку, исправлять шаблоны вручную не потребуется. Но генерируемая в браузере разметка хуже индексируется поисковыми системами. | ||
| | ||
| | ||
| ## Что дальше |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters. Learn more about bidirectional Unicode characters
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
2 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
No description provided.