-
Notifications
You must be signed in to change notification settings - Fork 41
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
无法在 airpub 中使用多说评论小组件 #22
Comments
还有几个疑问 1.40行构建方式是否不支持更换主题路径,只会读取/chill/下的模板文件…哪怕不能改,也希望这个文件夹可以叫theme什么的……当然这只是细节问题 2.已经实测40行方式不支持翻页,点翻页按钮无效。。。已经用云端默认主题,只改了id的index.html测试,均无法正常翻页 3.多说评论数经常读不出,原因不知,我其它网站正常,唯独用airpub的不行。代码正确。 4.在另一个帖子里提到的p标签输出问题(guoyu.me和airpub.io的差别) 主题已经基本完工了,就差后台和正文部分的CSS了,伸手怒求拆分的,自己写好累的而且后台完全摸不透。loading和login样式貌似可玩性也不错,所以建议基础css文件有正文排版和后台部分就好了 ^^ 另外……丢个主题预览链接:http://www.brdev.org |
5.和4差不多的一个问题。airpub.io和40行的输出方式,img标签外面会自动嵌套p标签,很不方便写样式……TUT 响应式部分还没写不知道会不会有更多问题,暂时先坑着…… |
多说评论数读不出的问题是否是因为代码的执行顺序?两个数据都是从多说服务器发出,但是读取评论数的请求早于data-thread-key的请求,所以没有读取成功呢? 以上纯脑补……最新评论也容易读不出,但是成功率比评论数略高 |
回北京了,我一个一个解释一下吧。
|
你的主题开源了吗?如果开源了我看看代码是如何工作的~ 这样更有利于解决问题。 |
@turingou 还没有制作完毕,admin部分的css还没有复制过来,所以后台现在没样式。 另外和这个版本保持同步的页面是:http://www.brdev.org |
至于标签的输出问题我可能没解释清楚,就是airpub.io和我所引用CDN后建立的页面,所打出的文章,在每个段落p标签外边,都会包围section标签(下面还有3个div标签),而guoyu.me是直接输出p标签,外面没有包裹那么多东西。个人建议后者更合适,我接触过的blog系统中都是使用后者的方式输出的 |
@banrikun 嗯,再确认一下你现在正在 |
引用的这两个JS文件 |
@banrikun 我看看具体是什么版本的问题。看来给打包后的文件打个版本好还是有必要的。。 |
@banrikun 找到了。这上边最新的版本是 而第一篇文章的图片之所以没有这个标签作为父层,是因为那个文章不是我自己编写的,是多说的爬虫抓取的正文,所以包裹在 这个问题,当时其实是想用 CSS 来做一个 workaround,用包含选择器来做判断,以此定义文章中插入图片的样式,现在看起来这种方法应该不太现实,还是应该在 filter 中处理好最后输出的 DOM 结构,这个问题会在 |
@turingou 辛苦了,现在这样不是很影响一般使用,只是个人有点在这种细节上比较纠结……囧 另外就是那个读取评论数和评论数的问题,貌似更新版本后并没有解决……依然读不出 0_0 |
@banrikun 关于读取评论数的代码,你具体是写在哪里? |
@turingou
审查元素获得的信息好像没问题
|
@banrikun 我看到代码了。原因是这样的,多说的 解决办法是我们在 airpub 中在相应的路由触发之后,手动告知 duoshuo embed.js 检查是否有钩子存在,然后填充 DOM。我标记这个为一个 bug |
@turingou 感谢解答,坐等新版本。 |
@banrikun 不客气,应该感谢你 :) |
@turingou 刚好今天给另一个站折腾instantclick的时候,遇到了同样的问题。由于instantclick会使得全站不刷新,从而无法触发多说,于是很期待解决方案~我相信也会有其他人需要的^^ |
@banrikun Airpub 会在 |
@turingou 好效率!感谢XD 我去折腾啦 |
https://github.com/airpub/chill/blob/master/admin.html
这个文件中,editor-ninja已经被拆分开来了 - -
我下载的这个默认的主题,按照上面的结构制作新主题,结果后台一直不加载输入框。。。。。。。。。
然后下载了 http://airpub.qiniudn.com/themes/chill/admin.html 这个文件……发现
。。。。。。。。。。。。。。。。。。。。
The text was updated successfully, but these errors were encountered: