-
Notifications
You must be signed in to change notification settings - Fork 1
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
base: master
Are you sure you want to change the base?
Conversation
@katienazarova посмотри, пожалуйста. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Посмотрел полностью, нашел пару десятков спорных на мой взгляд моментов разной степени важности.
manuscript/06_Styling.md
Outdated
|
||
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 и автоматическое добавление браузерных префиксов. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
связанных с стилем ...
правильнее будет:
связанных со стилями ...
т.к. имеется ввиду не стиль как характерная особенность чего-то
, а стили как каскадные таблицы стилей
, поэтому возможно лучше будет даже так:
связанных с каскадными таблицами стилей ...
manuscript/styling/01_loading.md
Outdated
|
||
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), и данная возможность обсуждается далее в части *Загрузка ресурсов*. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
не
через загрузчики, таких как
a
через загрузчики, такие как
manuscript/styling/01_loading.md
Outdated
|
||
*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`. То, как он это делает, можно настроить по-своему. Этот загрузчик также реализует интерфейс *горячей перезагрузки модулей*, обеспечивающий приятный опыт при разработки. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
внедряет стилизацию через элемент
я думаю стоит уточнить, что здесь идет речь о "HTML-элементе", или "теге":
внедряет стилизацию через тег
style
manuscript/styling/01_loading.md
Outdated
|
||
```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-файлах. Добавить новую функцию в конец файла с определениями: |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
не "Добавить", а "Добавим" или "Добавьте"
manuscript/styling/01_loading.md
Outdated
@@ -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. |
There was a problem hiding this comment.
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, даже если вы включили их с определенной конфигурацией загрузчика из-за того, как он работает под ним. |
There was a problem hiding this comment.
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)**. Результат достигается путем отрисовки страницы, а затем выяснения, какие правила необходимы для получения позитивного результата. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
на хабре, "отрисовка критического пути" уже переведена как "критический путь рендеринга", и является более верным переводом. Я добавил это в словарь, так что просмотри где еще ты использовал этот термин.
There was a problem hiding this comment.
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. |
There was a problem hiding this comment.
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, предназначенные для работы со старыми браузерами. |
There was a problem hiding this comment.
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 для того, как показано ниже: |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
убери для того
@LinchSmyth спасибо, исправил замечания. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
вот теперь отлично! Только пара не критичных помарок.
manuscript/styling/01_loading.md
Outdated
@@ -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`). |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
что-нибудь другой
-> какой-нибудь другой
manuscript/styling/01_loading.md
Outdated
@@ -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*. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
"испоЛЬЛЬзование"
|
||
Благодаря этому процессу `MiniCssExtractPlugin` поставляется с накладными расходами во время этапа компиляции. Он пока не работает с горячей перезагрузкой модуля (HMR). Учитывая, что плагин используется только для продакшена, это не проблема. | ||
Из-за этого процесса `MiniCssExtractPlugin` накладывает дополнительные затраты на этапе компиляции. Он пока не работает с горячей заменой модулей (HMR). Учитывая, что плагин используется только для продакшена, это не проблема. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Я думаю, лучше добавить "Однако" или "НО" перед
Учитывая, что плагин используется только для продакшена, это не проблема.
@LinchSmyth спасибо, поправил и это. |
No description provided.