Huge updates #27

xiazeyu · 2017-10-28T04:40:02Z

Fixed:

仅生成选择的模型(#14) 模型列表移除(#14)
增加模型左右设置(#10)(#19)(#21)(#23) 增加移动端模型更多功能(#12)
New README
Deleted package-lock.json to solve dependencies problems
Updates dependencies
Tried to fix (#26) Uncaught TypeError.

然后 @EYHN 你3个月前发的#15 现在还有效吗？
有空想把您的pr 3.0 merge到下一个pr中。

可用性与稳定性请大大仔细验证 xiazeyu未经仔细的测试，初步保证可用性
然后一些issues如果解决了可以关闭。
觉得可以请等待最终完成然后发布npm新版本

…hexo-fs to 0.2.2

xiazeyu · 2017-10-28T04:42:49Z

@EYHN 还有，我看你3.0的PR里改了好多，除了commit里提到的内容，很多都是语法风格之类的修改吗?
--已查阅并更新

xiazeyu · 2017-10-28T04:54:13Z

好吧，你的${config.hiddenOnMobile ? ...:...}实现貌似比我的简单很多，你可以接着更改QwQ
--已查阅并更新

EYHN · 2017-10-28T15:40:40Z

readme.zh-CN.md

+<%- live2d({
+	model: "nipsilon"
+}) %>
+```


--已查阅并更新
代码中加入了相关内容，请查阅
应在此补充设置优先顺序

EYHN · 2017-10-28T15:42:30Z

index.js

-  "nico": "nito/nico.model.json",
-  "nito": "nito/nito.model.json",
-  "nipsilon": "nito/nipsilon.model.json",
-  "nietzsche": "nito/nietzche.model.json",


nito系列 (i.e. ni-j ,nico,nito, nipsilon, nietzche) 都用不了了

--已查阅并更新
请仔细再次检查确保无误

EYHN · 2017-10-28T15:43:05Z

index.js

-
-registerDir('live2d/assets/', path.resolve(__dirname, './assets'));
-registerFile('live2d/script.js', path.resolve(__dirname, './dist/bundle.js'))
-
 hexo.extend.helper.register('live2d', function (config) {


这里的 config 参数是否应该删除

嗯，意思是这个hexo.helper只要传递一堆字符串就可以了吗？那个function(config)里的config变量是没用的？

xiazeyu · 2017-10-29T03:42:22Z

ejs不可用：
这个就不知道了，你repo原本的README就是这样写的= =我也不会ejs
要不然我删掉？

这里的 config 参数是否应该删除：
nodejs是异步的吗？这里我还在研究hexo的api原理
我的理解是
hexo.extend.generator.register
的作用是把文件丢到publish目录下，而
hexo.extend.helper.register('live2d', function (config)
的作用是告诉hexo应该在网页里加什么html代码
但是我还是beginer嘛，所以：
1，对function 后config是什么意思不理解。这句代码用的是你原先的代码
2，获取到的config数组（大概这么叫吧）是局部变量，无法在registerDir('live2d/assets/', path.resolve(__dirname, path.join('./assets/', config.model)));
中使用，因此在index.js中我增加了73-80行的代码

nito系列 (i.e. ni-j ,nico,nito, nipsilon,nietzche) 都用不了了
这个我更改了assets的文件结构，直接把nito系列单独拖出来了一个文件夹，README里的说明没检查，不过应该是可以使用的

话说如果我要更改pr的comment的话，直接git push我fork的repo，然后pr上做什么操作直接跟进到我repo的最近提交呢？

xiazeyu · 2017-10-29T03:46:41Z

index.js

+    <script type="text/javascript">
+    (function(){
+      if(device.mobile()){
+        if(${config.mobileShow}){


常见作者的pr“3.0”

xiazeyu · 2017-10-29T03:47:39Z

readme.zh-CN.md

+```
+
+
+- `model`: 模型名称 默认值: z16


测试是否可用

xiazeyu · 2017-10-29T03:50:00Z

index.js

+
+
+var config = Object.assign(


更好地传递config

xiazeyu · 2017-10-29T03:51:19Z

index.js

-
-registerDir('live2d/assets/', path.resolve(__dirname, './assets'));
-registerFile('live2d/script.js', path.resolve(__dirname, './dist/bundle.js'))
-
 hexo.extend.helper.register('live2d', function (config) {


嗯，意思是这个hexo.helper只要传递一堆字符串就可以了吗？那个function(config)里的config变量是没用的？

xiazeyu · 2017-10-30T13:19:09Z

@EYHN 我正在改写代码，然后请看看我上面的反馈，谢谢。

EYHN · 2017-10-30T13:29:06Z

我是指如果在ejs传入模型设置会导致一些问题。
同第一条
nito系列因为路径问题无法使用，你自己试试就知道了
push就可以自动更新了

xiazeyu · 2017-10-30T14:04:04Z

ejs中传入模型会导致错误如何解决？emmm,或者是改成swig传入模型设置？
对模板不是很熟悉，我觉得这种问题还是您多给点建议或者亲自更改比较好。

xiazeyu · 2017-10-30T14:05:33Z

@EYHN
好吧，请再次确认程序没问题，我下次commit再做document的问题吧。
建议你直接看修改过的index.js，diff可能比较乱，因为做了挺大的结构调整

xiazeyu · 2017-10-30T15:04:09Z

参照了
https://hexo.io/api/helper.html
修复了config问题
以不再能够在模板中传递设置的代价换来了只上传选定模型的改进。

xiazeyu · 2017-11-02T14:10:36Z

目前所有改动均已完成，请检查。
@EYHN
如果可以merge，请顺便更新版本号发布npm，谢谢。

xiazeyu added 8 commits October 27, 2017 17:52

Tried to fix (#26) Uncaught TypeError

4ad70f5

Deleted package-lock.json to solve dependencies problems, and update …

a992565

…hexo-fs to 0.2.2

Write new README

4a79abe

增加模型左右设置(#10)(#19)(#21)(#23) 增加移动端模型更多功能(#12)

b3041f3

Update READMEs

d5a6c9b

仅生成选择的模型(#14) 模型列表移除(#14) 请注意README中的路径要求！

58453ec

Update READMEs

1ae7b69

optimize attaching models' file structure

9045f96

EYHN suggested changes Oct 28, 2017

View reviewed changes

xiazeyu commented Oct 29, 2017

View reviewed changes

Fix bugs and improve coding

f22ce35

Fixed config issue

48a84c7

xiazeyu added 4 commits November 2, 2017 20:23

Improve file structure

e18da32

Rebuild script

7570106

Update README

e8332a2

Add custom model path at root directory

5fc7796

change bottomOffset to verticalOffset

94560f7

EYHN approved these changes Nov 3, 2017

View reviewed changes

update version

97ad909

EYHN merged commit cb36854 into EYHN:master Nov 3, 2017

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

Huge updates #27

Huge updates #27

xiazeyu commented Oct 28, 2017

xiazeyu commented Oct 28, 2017 •

edited

Loading

xiazeyu commented Oct 28, 2017 •

edited

Loading

EYHN Oct 28, 2017

xiazeyu Oct 30, 2017

EYHN Oct 28, 2017

xiazeyu Oct 30, 2017 •

edited

Loading

EYHN Oct 28, 2017

xiazeyu Oct 29, 2017

xiazeyu commented Oct 29, 2017

xiazeyu Oct 29, 2017

xiazeyu Oct 29, 2017

xiazeyu Oct 29, 2017

xiazeyu Oct 29, 2017

xiazeyu commented Oct 30, 2017

EYHN commented Oct 30, 2017 •

edited

Loading

xiazeyu commented Oct 30, 2017

xiazeyu commented Oct 30, 2017 •

edited

Loading

xiazeyu commented Oct 30, 2017

xiazeyu commented Nov 2, 2017 •

edited

Loading

		```


		- `model`: 模型名称默认值: z16

Huge updates #27

Huge updates #27

Conversation

xiazeyu commented Oct 28, 2017

xiazeyu commented Oct 28, 2017 • edited Loading

xiazeyu commented Oct 28, 2017 • edited Loading

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

xiazeyu Oct 30, 2017 • edited Loading

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

xiazeyu commented Oct 29, 2017

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

xiazeyu commented Oct 30, 2017

EYHN commented Oct 30, 2017 • edited Loading

xiazeyu commented Oct 30, 2017

xiazeyu commented Oct 30, 2017 • edited Loading

xiazeyu commented Oct 30, 2017

xiazeyu commented Nov 2, 2017 • edited Loading

xiazeyu commented Oct 28, 2017 •

edited

Loading

xiazeyu commented Oct 28, 2017 •

edited

Loading

xiazeyu Oct 30, 2017 •

edited

Loading

EYHN commented Oct 30, 2017 •

edited

Loading

xiazeyu commented Oct 30, 2017 •

edited

Loading

xiazeyu commented Nov 2, 2017 •

edited

Loading