Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Styling #3

Open
wants to merge 8 commits into
base: master
Choose a base branch
from
Open

Styling #3

wants to merge 8 commits into from

Conversation

lex111
Copy link
Member

@lex111 lex111 commented Aug 20, 2018

No description provided.

@lex111 lex111 requested a review from katienazarova August 20, 2018 18:25
@lex111
Copy link
Member Author

lex111 commented Aug 20, 2018

@katienazarova посмотри, пожалуйста.

Copy link
Collaborator

@LinchSmyth LinchSmyth left a comment

Choose a reason for hiding this comment

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

Посмотрел полностью, нашел пару десятков спорных на мой взгляд моментов разной степени важности.


In this part, you will learn about styling-related concerns in detail including loading styles, refreshing styles during development, separating CSS, eliminating unused CSS, and autoprefixing.
В этой части вы подробно узнаете о проблемах, связанных с стилем, включая загрузку стилей, обновление стилей во время разработки, разделение CSS, исключение неиспользуемого CSS и автоматическое добавление браузерных префиксов.
Copy link
Collaborator

Choose a reason for hiding this comment

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

связанных с стилем ...

правильнее будет:

связанных со стилями ...

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

связанных с каскадными таблицами стилей ...


The matched files can be processed through loaders like [file-loader](https://www.npmjs.com/package/file-loader) or [url-loader](https://www.npmjs.com/package/url-loader), and these possibilities are discussed in the *Loading Assets* part of the book.
Соответствующие файлы могут обрабатываться через загрузчики, таких как [file-loader](https://www.npmjs.com/package/file-loader) или [url-loader](https://www.npmjs.com/package/url-loader), и данная возможность обсуждается далее в части *Загрузка ресурсов*.
Copy link
Collaborator

Choose a reason for hiding this comment

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

не

через загрузчики, таких как

a

через загрузчики, такие как


*style-loader* injects the styling through a `style` element. The way it does this can be customized. It also implements the *Hot Module Replacement* interface providing for a pleasant development experience.
*style-loader* внедряет стилизацию через элемент `style`. То, как он это делает, можно настроить по-своему. Этот загрузчик также реализует интерфейс *горячей перезагрузки модулей*, обеспечивающий приятный опыт при разработки.
Copy link
Collaborator

Choose a reason for hiding this comment

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

внедряет стилизацию через элемент

я думаю стоит уточнить, что здесь идет речь о "HTML-элементе", или "теге":

внедряет стилизацию через тег style


```bash
npm install css-loader style-loader --save-dev
```

Now let's make sure webpack is aware of them. Add a new function at the end of the part definition:
Теперь давайте убедимся, что webpack знает о CSS-файлах. Добавить новую функцию в конец файла с определениями:
Copy link
Collaborator

Choose a reason for hiding this comment

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

не "Добавить", а "Добавим" или "Добавьте"

@@ -53,17 +53,17 @@ leanpub-end-insert
]);
```

The added configuration means that files ending with `.css` should invoke the given loaders. `test` matches against a JavaScript-style regular expression.
Добавленная конфигурация означает, что файлы, заканчивающиеся на `.css`, должны вызывать указанные загрузчики. `test` сопоставляет стилевой файл с регулярным выражением JavaScript.
Copy link
Collaborator

Choose a reason for hiding this comment

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

я думаю лучше будет файл стилей вместо стилевой файл


W> Using PurifyCSS loses CSS source maps even if you have enabled them with loader specific configuration due to the way it works underneath.
W> Использование PurifyCSS теряет карты кода CSS, даже если вы включили их с определенной конфигурацией загрузчика из-за того, как он работает под ним.
Copy link
Collaborator

Choose a reason for hiding this comment

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

я думаю более верно:

Использование PurifyCSS приводит к потере карт кода CSS, даже если вы включили их с определенной конфигурацией загрузчика, из-за своей внутренней обработки.


The idea of [critical path rendering](https://developers.google.com/web/fundamentals/performance/critical-rendering-path/) takes a look at CSS performance from a different angle. Instead of optimizing for size, it optimizes for render order and emphasizes **above-the-fold** CSS. The result is achieved by rendering the page and then figuring out which rules are required to obtain the shown result.
Идея [отрисовки критического пути](https://developers.google.com/web/fundamentals/performance/critical-rendering-path/) рассматривает производительность CSS с другой точки зрения. Вместо оптимизации размера, он оптимизирует порядок отрисовки и выделяет CSS **в верхней части страницы (above-the-fold)**. Результат достигается путем отрисовки страницы, а затем выяснения, какие правила необходимы для получения позитивного результата.
Copy link
Collaborator

Choose a reason for hiding this comment

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

на хабре, "отрисовка критического пути" уже переведена как "критический путь рендеринга", и является более верным переводом. Я добавил это в словарь, так что просмотри где еще ты использовал этот термин.

Copy link
Member Author

Choose a reason for hiding this comment

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

Ок, только использовать будем вместо рендеринга - отрисовку, хорошо?

* Исключить неиспользованный CSS можно с помощью PurifyCSS. Он выполняет статический анализ с источником.
* Эта функциональность может быть включена через *purifycss-webpack*, и плагин следует применять *после* `MiniCssExtractPlugin`.
* В лучшем случае PurifyCSS может исключить большинство, если не все, неиспользуемых CSS-правил.
* Отрисовка критического пути — это еще одна методика CSS, которая подчеркивает сначала отрисовку CSS в верхней части страницы. Идея состоит в том, чтобы сделать что-то как можно быстрее, а не ждать загрузки CSS.
Copy link
Collaborator

Choose a reason for hiding this comment

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

Критический путь рендеринга - это еще одна методика CSS, которая выделяет CSS в начало страницы. Идея состоит в том, что бы отобразить что-то как можно быстрее, вместо ожидания загрузки CSS.


In the next chapter, you'll learn to **autoprefix**. Enabling the feature makes it more convenient to develop complicated CSS setups that work with older browsers as well.
В следующей главе вы узнаете **autoprefix**. Включение возможности упрощает написание вручную сложных настроек CSS, предназначенные для работы со старыми браузерами.
Copy link
Collaborator

Choose a reason for hiding this comment

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

узнаете об


Включение этой возможности ...


предназначенных

@@ -43,7 +43,7 @@ leanpub-end-insert
]);
```

To confirm that the setup works, we have to add something to autoprefix. Adjust the CSS:
Чтобы проверить, что настройка работает, нам следует что-нибудь добавить, измените CSS для того, как показано ниже:
Copy link
Collaborator

Choose a reason for hiding this comment

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

убери для того

@lex111
Copy link
Member Author

lex111 commented Aug 22, 2018

@LinchSmyth спасибо, исправил замечания.

Copy link
Collaborator

@LinchSmyth LinchSmyth left a comment

Choose a reason for hiding this comment

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

вот теперь отлично! Только пара не критичных помарок.

@@ -84,13 +84,9 @@ leanpub-end-insert
...
```

Выполните «npm start» и перейдите на «http: // localhost: 8080», если используете порт по умолчанию и откройте * main.css * и измените цвет фона на нечто вроде `lime` (` background: lime`) ,
Выполните `npm start` и перейдите по URL-адресу `http://localhost:8080`, если используете порт по умолчанию, а затем откройте *main.css* и измените цвет фона на что-нибудь другой, например на `lime` (`background: lime`).
Copy link
Collaborator

Choose a reason for hiding this comment

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

что-нибудь другой -> какой-нибудь другой

@@ -100,14 +96,10 @@ T> В приложении *CSS-модули* обсуждается подхо

![Less](images/less.png)

Меньше - процессор CSS, оснащенный функциональностью. Использование Less не требует больших усилий с помощью webpack, поскольку менее загружаемый имеет дело с тяжелым подъемом. Вы должны установить меньше, также учитывая, что это зависимость от меньшего загрузчика.
[Less](http://lesscss.org/) — CSS-препроцессор, оснащенный функциональностью. Испольльзование Less не требует больших усилий с webpack, поскольку загрузчик [less-loader](https://www.npmjs.com/package/less-loader) выполняет всю тяжелую (черновую) работу. Вам нужно установить [less](https://www.npmjs.com/package/less), поскольку от него зависит *less-loader*.
Copy link
Collaborator

Choose a reason for hiding this comment

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

"испоЛЬЛЬзование"


Благодаря этому процессу `MiniCssExtractPlugin` поставляется с накладными расходами во время этапа компиляции. Он пока не работает с горячей перезагрузкой модуля (HMR). Учитывая, что плагин используется только для продакшена, это не проблема.
Из-за этого процесса `MiniCssExtractPlugin` накладывает дополнительные затраты на этапе компиляции. Он пока не работает с горячей заменой модулей (HMR). Учитывая, что плагин используется только для продакшена, это не проблема.
Copy link
Collaborator

Choose a reason for hiding this comment

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

Я думаю, лучше добавить "Однако" или "НО" перед

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

@lex111
Copy link
Member Author

lex111 commented Aug 23, 2018

@LinchSmyth спасибо, поправил и это.

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