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

Add docs to stylus tech #99

Merged
merged 1 commit into from
Jul 31, 2015
Merged

Add docs to stylus tech #99

merged 1 commit into from
Jul 31, 2015

Conversation

tavriaforever
Copy link
Contributor

Resolved #62
//cc @blond @sipayRT

@tavriaforever tavriaforever self-assigned this Jul 30, 2015
@@ -3,32 +3,76 @@ enb-stylus

[![NPM version](http://img.shields.io/npm/v/enb-stylus.svg?style=flat)](http://www.npmjs.org/package/enb-stylus) [![Build Status](http://img.shields.io/travis/enb-make/enb-stylus/master.svg?style=flat&label=tests)](https://travis-ci.org/enb-make/enb-stylus) [![Build status](https://img.shields.io/appveyor/ci/blond/enb-stylus.svg?style=flat&label=windows)](https://ci.appveyor.com/project/blond/enb-stylus) [![Coverage Status](https://img.shields.io/coveralls/enb-make/enb-stylus.svg?style=flat)](https://coveralls.io/r/enb-make/enb-stylus?branch=master) [![Dependency Status](http://img.shields.io/david/enb-make/enb-stylus.svg?style=flat)](https://david-dm.org/enb-make/enb-stylus)

Поддержка Stylus для ENB. Пакет содержит технологии:
* `enb-stylus/techs/stylus`
Пакет предоставляет [ENB](https://ru.bem.info/tools/bem/enb-bem/)-технологию для сборки `CSS` и `Stylus` файлов в проектах построенных по [методологии БЭМ](https://ru.bem.info/method/).
Copy link

Choose a reason for hiding this comment

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

запятая перед "построенных"

Copy link
Contributor Author

Choose a reason for hiding this comment

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

ok


### Совместное использование Stylus и CSS

В проекте допускается совместное использование блоков, которые используют одну из технологии `.css` или `.styl`. Однако в рамках одного блока не могут одновременно использоваться обе технологии. Если стили блока реализованы и в CSS, и в Stylus, по умолчанию будет использоваться файл с расширением `.styl`.
Copy link
Member

Choose a reason for hiding this comment

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

совместное использование блоков

Что это значит?

Copy link
Member

Choose a reason for hiding this comment

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

В проекте допускается совместное использование блоков, которые используют одну из технологии .css или .styl.

Может так?

В проекте допускается совместное использование .css и .styl файлов.

Copy link
Member

Choose a reason for hiding this comment

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

Если стили блока реализованы и в CSS, и в Stylus, по умолчанию будет использоваться файл с расширением .styl.

«по умолчанию» тут лишнее слово, потому что на это никак нельзя повлиять.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

@blond изменил

@blond
Copy link
Member

blond commented Jul 31, 2015

🆙


## Работа технологии

В БЭМ-методологии стили к каждому блоку хранятся в отдельных файлах в директориях блоков.
Copy link
Member

Choose a reason for hiding this comment

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

я бы написал «По БЭМ-методологии» и здесь можно поставить ссылку https://ru.bem.info/method/filesystem/

@blond
Copy link
Member

blond commented Jul 31, 2015

🆙

<head>
<meta charset="utf-8"/>
...
<!--[if gt IE 9]>
Copy link
Member

Choose a reason for hiding this comment

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

<!--[if gt IE 9]><!--><link rel="stylesheet" href="index.css"/><!--<![endif]-->
<!--[if lte IE 9]><link rel="stylesheet" href="index.ie.css"/><![endif]-->

@blond
Copy link
Member

blond commented Jul 31, 2015

🎉

blond added a commit that referenced this pull request Jul 31, 2015
@blond blond merged commit dad6cc1 into master Jul 31, 2015
@blond blond deleted the issue-62 branch July 31, 2015 14:33
@blond blond removed the review label Jul 31, 2015
@sipayRT
Copy link

sipayRT commented Jul 31, 2015

надо Коляну картинку отослать)

@innabelaya
Copy link

дада, разделяю радость :) В enb-bh это было так: enb/enb-bh#95 (comment)

@tavriaforever
Copy link
Contributor Author

Алилуя! Use Befooz! Люблю вас господа)

@tavriaforever tavriaforever removed their assignment Mar 21, 2016
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.

5 participants