[WIP] Dist. Review changes #701
Open
Conversation
This file contains 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
innabelaya
force-pushed
the
dist-qs
branch
3 times, most recently
from
0920cda
to
6b47f4f
Compare
|
cc @godfreyd |
|
@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). | ||
| * [Быстрый старт](#Быстрый-старт). Как подключить библиотеку и вставить блок на страницу | ||
| * [Руководство](#Руководство-Работа-с-блоками-библиотеки). Как работать с блоками библиотеки |
|
|
||
| Чтобы добавить блок на страницу, необходимо: | ||
| ## Руководство. Работа с блоками библиотеки |
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` и научим их взаимодействовать друг с другом: имя пользователя при нажатии на кнопку будет отображаться в приветствии. |
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). |
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` в браузере. | ||
|
|
||
| ### Добавьте блокам функциональность |
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` уже включены в сборку. |
There was a problem hiding this comment.
- «Чтобы изменять ..., используется» — здесь все верно?
- «уже включены в сборку» — в какую сборку?
|
|
||
| Проект в [JSFiddle](https://jsfiddle.net/inna__neige/0hLmLmzn/). | ||
|
|
||
| ### Генерация HTML в браузере | ||
| ## Альтернативный способ подключения блоков |
There was a problem hiding this comment.
Это что за раздел? Изначально было заявлено про быстрый старт и руководство.
| ``` | ||
|
|
||
| Проект в [JSFiddle](https://jsfiddle.net/inna__neige/df6uuw7u/). | ||
|
|
||
| ## Результат | ||
| ## Какой способ лучше? |
There was a problem hiding this comment.
Есть знак вопроса, см. следующий раздел.
| Если в новой версии библиотеки обновленные шаблоны изменят разметку, исправлять шаблоны вручную не потребуется. Но генерируемая в браузере разметка хуже индексируется поисковыми системами. | ||
|
|
||
|
|
||
| ## Что дальше |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
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.