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

Windows compatible #2

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

Windows compatible #2

wants to merge 4 commits into from

Conversation

pahaz
Copy link
Contributor

@pahaz pahaz commented Jan 10, 2015

Now, window users can use this branch.
WARNIG: pull request was sent in deep night.

&__content {
margin-left: 300px;
}
.layout__aside {
Copy link
Member

Choose a reason for hiding this comment

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

Can you speak russian?

Почему вы хотите писать именно так? Нам нравится использовать возможности Stylus.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Yes.

К сожалению, я не нашел списка используемых в ваших проектах соглашений и рекомендаций.
Для начинающих разработчиков явный синтаксис будет более понятен. (+ https://github.com/bevis-ui/bevis-stub/blob/master/blocks/layout/layout.styl#L1-L9 использует подобный синтаксис)
Я посчитал, что project-stub имеет больший приоритет в данном случае.

Должен быть какой-то канонический пример или список рекомендаций (в данном случае я использовал project-stub).

Copy link
Member

Choose a reason for hiding this comment

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

Согласен с вами. Я сам виноват, что ввожу в заблуждение разношёрстным кодом :)

@mdevils
Copy link

mdevils commented Jan 12, 2015

👍

@@ -1 +0,0 @@
../node_modules/chai/chai.js
Copy link
Contributor Author

Choose a reason for hiding this comment

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

В winows есть проблема с символическими ссылками. :(

@pahaz
Copy link
Contributor Author

pahaz commented Jan 14, 2015

Позже, я подробнее посмотрю внесенные изменения и верну удаленные блоки.
(pull was sent in deep night)

Сейчас, эту ветку могут использовать пользователи windows.

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.

None yet

3 participants