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

Showcase #104

Open
blond opened this issue Apr 7, 2015 · 7 comments
Open

Showcase #104

blond opened this issue Apr 7, 2015 · 7 comments

Comments

@blond
Copy link
Member

blond commented Apr 7, 2015

Давайте заиспользует showcase из bem-components вместо непонятного бандла.

@tadatuta
Copy link
Member

tadatuta commented Apr 7, 2015

@blond в целом да, но есть проблема: showcase предполагает кучу кастомных блоков, которые в project-stub совершенно не нужны

// cc @sipayRT

@sipayRT
Copy link
Contributor

sipayRT commented Apr 7, 2015

там всего 4 блока новых. все остальное - это доопределения существующих. В общем, я за то, чтобы добавить

@tadatuta
Copy link
Member

tadatuta commented Apr 7, 2015

@sipayRT меня смущает то, что для всех, кто использует project-stub по делу, а не для посмотреть на витрину, потребуется эти блоки удалять. может быть можно заинлайнить стили прямо в bemjson?

@sipayRT
Copy link
Contributor

sipayRT commented Apr 7, 2015

@tadatuta можно и заинлайнить - их там вроде не много

@blond
Copy link
Member Author

blond commented Apr 7, 2015

+1 к заинлайнить!

@sipayRT
Copy link
Contributor

sipayRT commented Apr 7, 2015

@tadatuta @blond я так понимаю, что эта таска достается мне? :)

@tavriaforever
Copy link
Member

showcaseman

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

5 participants