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

use ppt template #4

Merged
merged 3 commits into from
Sep 1, 2015
Merged

use ppt template #4

merged 3 commits into from
Sep 1, 2015

Conversation

Jayin
Copy link
Collaborator

@Jayin Jayin commented Aug 31, 2015

请review

@huangruichang
Copy link
Collaborator

LGTM

@fritx
Copy link
Owner

fritx commented Aug 31, 2015

哦 不要放在web/ppt_里 那个是复制到product里的
这个模板只是build的时候用到 放在web文件夹外的其他地方~

@fritx
Copy link
Owner

fritx commented Aug 31, 2015

👍 牛逼 这速度

@huangruichang
Copy link
Collaborator

@fritx 不用管我的评论 👍

@fritx
Copy link
Owner

fritx commented Aug 31, 2015

你们认为呢 或者就这样也ok?

@huangruichang
Copy link
Collaborator

@Jayin @fritx 感觉亮爷说得有道理,刚才回去看了下,那段拼出来的 html 就用一次,那就招亮爷说的放到别的地方?

@fritx
Copy link
Owner

fritx commented Aug 31, 2015

😄 居然递归了

@Jayin
Copy link
Collaborator Author

Jayin commented Aug 31, 2015

Right.
不过我认为构建的时候不需要copy整个web/,复制web/ppt_就可以了
So, ppt.handlebars移到web/下合理一些

@fritx
Copy link
Owner

fritx commented Aug 31, 2015

不是 那个web是预留的 那是一个结构 ppt_只是一部分
web/**都是被复制过去的对象

@fritx
Copy link
Owner

fritx commented Aug 31, 2015

另外 我个人比较支持使用hbs后缀 不用搞那么冗余 嘿嘿

@Jayin
Copy link
Collaborator Author

Jayin commented Aug 31, 2015

so,你建议放哪儿好?

支持使用hbs后缀

OK

@fritx
Copy link
Owner

fritx commented Aug 31, 2015

好 目前希望简单点 统一模板 可以直接放根目
或者你希望支持多模板选择 也可以建个文件夹放 预留空间 似乎都是通过js控制内容 一套模板吧

@fritx
Copy link
Owner

fritx commented Sep 1, 2015

或者放web 然后把复制那里改成只复制ppt_ 目前没问题 你来选 谢啦!

Jayin added a commit that referenced this pull request Sep 1, 2015
@Jayin Jayin merged commit 8a96fea into dev Sep 1, 2015
@Jayin Jayin deleted the seperate-template branch September 1, 2015 02:24
@fritx
Copy link
Owner

fritx commented Sep 1, 2015

nice thanks! 不过 handlebars后缀真的好长 😓 never mind..

@fritx
Copy link
Owner

fritx commented Sep 1, 2015

个人建议 提交PR的时候
尽量使用原编码规范 比如分号缩进 是对原作者的尊重
一般来说 他的每一个决定 都有合理的理由
btw: 'separate' instead of 'seperate'

@Jayin
Copy link
Collaborator Author

Jayin commented Sep 1, 2015

分号缩进

加个editorconfig?

@fritx
Copy link
Owner

fritx commented Sep 1, 2015

哦 我反对这个东西 我认为多余了
(其实我认为@sinderxxx的许多小项目有太多无关的文件 违背了我个人的协调美学)
比较表层的编码规范在改的时候是人都能看得出来
如果需要这种 我会采用暴力一点的standard 但我目前不会搞这套
只能说看"自觉"吧 毕竟这对部分人来说属于高要求

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.

3 participants