Skip to content

Conversation

@4exova
Copy link

@4exova 4exova commented Nov 28, 2014

No description provided.

Quickstart.md Outdated
Copy link
Owner

Choose a reason for hiding this comment

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

в доках должны заголовки начинаться со второго уровня. С первым уровнем не будут создаваться якоря на них

@4exova 4exova force-pushed the quickstart branch 3 times, most recently from 0d19d36 to f30b3d9 Compare November 28, 2014 12:41
Quickstart.md Outdated
Copy link
Collaborator

Choose a reason for hiding this comment

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

предалагаю консольные команды оформлять так:

git clone https://github.com/bem/project-stub.git start-project  
cd start-project      
npm install

Copy link
Collaborator

Choose a reason for hiding this comment

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

нужно завязаться на конкретную ревизию, чтобы при изменениях в project-stub следование мануалу приводило к предсказуемым результатам

Quickstart.md Outdated
Copy link
Owner

Choose a reason for hiding this comment

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

Насколько я понимаю, страница работать не будет, потому что для блока hello ничего еще не написано. На этом этапе будет просто отображаться все то, что мы описали в BEMJSON.

@4exova 4exova force-pushed the quickstart branch 2 times, most recently from 4176ff3 to acda9b1 Compare January 19, 2015 11:13
Quickstart.md Outdated

Choose a reason for hiding this comment

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

Картинка -- идея правильная, только предлагаю:

  1. Сделать ее аккуратнее (сейчас она обрезана, выглядит не очень; можно уменьшить размер снимаемого окна, чтобы строчка наверху и кнопки внизу так резко не обрубались; черный угол наверху слева тоже не добавляет аккуратности).
  2. Если в MD поддерживаются каты, можно было бы спрятать картинку, чтобы не пугать объемом статьи.

Copy link
Collaborator

Choose a reason for hiding this comment

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

node_modules/.bin/npm start -> npm start

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