-
Notifications
You must be signed in to change notification settings - Fork 8
added new doc quickstart #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
base: master
Are you sure you want to change the base?
Conversation
Quickstart.md
Outdated
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.
в доках должны заголовки начинаться со второго уровня. С первым уровнем не будут создаваться якоря на них
0d19d36 to
f30b3d9
Compare
Quickstart.md
Outdated
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.
предалагаю консольные команды оформлять так:
git clone https://github.com/bem/project-stub.git start-project
cd start-project
npm installThere 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.
нужно завязаться на конкретную ревизию, чтобы при изменениях в project-stub следование мануалу приводило к предсказуемым результатам
Quickstart.md
Outdated
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.
Насколько я понимаю, страница работать не будет, потому что для блока hello ничего еще не написано. На этом этапе будет просто отображаться все то, что мы описали в BEMJSON.
4176ff3 to
acda9b1
Compare
Quickstart.md
Outdated
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.
Картинка -- идея правильная, только предлагаю:
- Сделать ее аккуратнее (сейчас она обрезана, выглядит не очень; можно уменьшить размер снимаемого окна, чтобы строчка наверху и кнопки внизу так резко не обрубались; черный угол наверху слева тоже не добавляет аккуратности).
- Если в MD поддерживаются каты, можно было бы спрятать картинку, чтобы не пугать объемом статьи.
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.
node_modules/.bin/npm start -> npm start
No description provided.