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

[Bug]: source 中填写 bangumi 或 bgm 都会出现只能获取第一页的数据,bgmv0 则出现排版错误 #499

Closed
windowBR opened this issue Aug 16, 2024 · 16 comments
Labels
bug Something isn't working

Comments

@windowBR
Copy link
Contributor

问题描述及如何重现此问题

当我使用 bgm 或者 bangumi 时仅获取到了 Bangumi 的第一页数据:
image
image

当我使用 bgmv0 时能获取正常,但出现了排版问题,疑似是我主题没法用进度条:
image
image

报错信息(建议使用截图,请尽量截取全部把错误信息)

插件版本

1.10.1

环境信息(使用hexo -v命令查看)

hexo: 6.3.0
hexo-cli: 4.3.0
os: win32 10.0.22631
node: 18.20.4
acorn: 8.11.3
ada: 2.7.8
ares: 1.28.1
base64: 0.5.2
brotli: 1.0.9
cjs_module_lexer: 1.2.2
cldr: 44.1
icu: 74.2
llhttp: 6.1.1
modules: 108
napi: 9
nghttp2: 1.61.0
nghttp3: 0.7.0
ngtcp2: 1.3.0
openssl: 3.0.13+quic
simdutf: 5.2.4
tz: 2024a
undici: 5.28.4
unicode: 15.1
uv: 1.44.2
uvwasi: 0.0.19
v8: 10.2.154.26-node.37
zlib: 1.3.0.1-motley
@windowBR windowBR added the bug Something isn't working label Aug 16, 2024
@windowBR
Copy link
Contributor Author

目前我希望的是想试试在使用 bgmv0 的同时不使用追番进度条,但我在 readme 里似乎没找到这个功能的开关?

@HCLonely
Copy link
Owner

progress: false关闭进度条

@windowBR
Copy link
Contributor Author

progress: false关闭进度条

这我当然试过,但我希望关掉的是上图这个进度条,这个选项关掉的是下图的进度条,压根不是一个东西啊。。。
image
image

HCLonely added a commit that referenced this issue Aug 18, 2024
>> 新增:`progressBar`选项,可配置是否显示追番进度条(#499)
@HCLonely
Copy link
Owner

v1.10.2版本已修复

@windowBR
Copy link
Contributor Author

v1.10.2版本已修复

感谢修复,另外我刚刚更新到了最新 1.10.2 版本,再次进行了相关选项调整后发现了排版错误并非进度条引起。
经测试发现原因为在 同时使用 bgmv0 且开启我的评论(showMyComment: true) 的情况下才会出现错误,错误状态如该 issue 最开始的截图所示。

@HCLonely
Copy link
Owner

已修复

@windowBR
Copy link
Contributor Author

windowBR commented Aug 29, 2024

已修复

再次感谢修复,但仍然发现了新的问题,我直接上图吧
首先是 bgmv0源 打开评论栏的有评论和无评论的截图:
image
然后是使用 bgm源 其余设置不变:
image

问题描述:使用 bgmv0源 并开启评论栏时,首先是无论是否撰写了评论,都会显示一个框;然后有评论的条目也存在一些问题,首先是评论栏的框与封面重合,其次是没有显示个人打分。而 bgm源 的评论框行为则非常符合直觉,有评论的才显示框,并且条目高度增高以正常放置评论栏并显示评分。

对于排版的 bug 并不确定是否是由于主题引起,bgmv0 下的评论栏重合情况也不是所有条目都一定重合,仍然存在不重合的条目;但使用 bgm源 时则所有条目全部正常,所以个人推测原因还是出在对于 bgmv0 并未做好适配。

@windowBR
Copy link
Contributor Author

windowBR commented Aug 29, 2024

补充:对比了各个番剧条目后似乎找到了封面与评论框重合的原因,原因出在 bgmv0源 显示的简介与 bgm源 截断位置不同。bgmv0源 在许多条目的番剧简介截断的位置在我的主题下显示为两行,造成了评论栏位置上移了一行。将浏览器窗口大小缩窄使简介变为三行字后排版恢复正常也验证了这个想法。而 bgm源 则基本不会出现这个问题,截断位置基本都是显示为三行简介。

附图为 bgmv0源 将浏览器窗口缩窄后的表现
image

顺带一提在 bgm源 下如果简介原文全长都只有两行字甚至一行字的情况下也会出现类似的问题
image
但是我在官方提供的 demo 页面似乎并未发现这种情况
image

如果这是我主题 bug 那不予修复也可以理解,但以下问题仍然希望得到改善

  1. 简介截断位置的逻辑更改为与 bgm源 类似以缓解这种问题
  2. 添加个人打分的数据
  3. 无评论、打分时不显示评论框

或许会问既然这样我直接用 bgm源 应该是最好的方法,为什么还要提这么多意见。我先提前解释一下,因为我同样想显示 我的游戏,而 我的游戏 根据 readme 的描述是没法使用 bgmv0 以外的数据源的,所以要是能解决这个问题我会非常感激。

@HCLonely
Copy link
Owner

已修复,再试试

@windowBR
Copy link
Contributor Author

再次感谢修复

排版问题相比上一版有缓解,但仍未完全解决
对于无评论、无评分的作品仍然未去除评论栏的显示

下图为浏览器默认最大化窗口大小下,简介分别显示出两行及三行、有评论和评分/有评分无评论/无评分和评论的情况
image
下图为同样窗口下简介仅显示一行的情况(简介全长仅能占一行)
image

@HCLonely
Copy link
Owner

  1. 排版样式已修复;
  2. 无评论、无评分的作品去除评论栏的显示需要重新更新数据hexo bangumi -u

@windowBR
Copy link
Contributor Author

目前发现的其余问题均已解决
简介不足三行的排版问题仍然存在
image

已使用当前版本部署到博客上,如果能帮助到 debug 的话可以来看看
页面链接

@HCLonely
Copy link
Owner

主题兼容性问题,可以添加自定义样式解决

.bangumi-info {
    min-height: 180px;
}

@windowBR
Copy link
Contributor Author

有点不好意思但我其实不太懂这个该怎么改。

姑且说一下我找过的地方吧
我先是按照 README 最后的路径去 hexo 里面的 /node_modules/hexo-bilibili-bangumi/src/lib/templates/theme/ 添加了个nexmoe.css 并填入你提供的代码后,分别 hexo cleanhexo generatehexo s 进行测试但发现 bangumi-info 仍然是 160px
image

然后发现你之前的修复提交多次更改了 /src/lib/templates/theme/fluid.css 就尝试去这文件最后加上你提供的代码并测试,情况仍未改变
最后还尝试了将 /src/lib/templates/index.css 第 48 行 .bangumi-info 中的 min-height 修改为 180px 似乎也没见生效

知识不到位尝试摸索但没能成功摸索出来,看来还是需要来求助一下😢,实在麻烦您了

@HCLonely
Copy link
Owner

参考你主题的配置说明https://docs.nexmoe.com/v4.2/config/style.html

@windowBR
Copy link
Contributor Author

感谢协助,虽然用的不是这个方法但我最终解决了

因为经测试这个方案并没有作用,会被插件的设置覆盖掉(如下图),但最终我还是成功找到了解决方案
image

其实还是我知识不足的问题,就像我上面说的

我先是按照 README 最后的路径去 hexo 里面的 /node_modules/hexo-bilibili-bangumi/src/lib/templates/theme/ 添加了个nexmoe.css 并填入你提供的代码后,分别 hexo clean、hexo generate、hexo s 进行测试但发现 bangumi-info 仍然是 160px

我修改后压根没重新 build,而且直接再 hexo 下这个环境下也没法运行 build(虽然我也不知道原因但总之就是报错),因此其实改了个寂寞。

于是我还是完整的走了遍流程:

  1. 将项目 fork 并进行了修改然后 build。
  2. 我直接将 build 后的整个项目文件夹复制并覆盖了 hexo 的 node_modules 里面的原文件夹然后进行测试,结果是一切完美
    我感觉这么直接整个文件夹覆盖是不是会留下什么隐患,但这里我是真不太清楚应该如何操作,希望能得到些指导
    明天我再研究下 pr 相关吧,到时候再交一个 pr 来适配一下这个主题 bug

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

No branches or pull requests

2 participants