songhn233

songhn233

An algorithm enthusiast and front-end newbie.

Member Since 2 years ago

Hangzhou, China

Experience Points
44
follower
Lessons Completed
27
follow
Lessons Completed
643
stars
Best Reply Awards
33
repos

989 contributions in the last year

Pinned
⚡ ⛵ A simple, responsive Hexo theme
⚡ 📚 All my posts archive
⚡ 🌈 Shiki plugin for milkdown. Add support for shiki highlight.
⚡ My Personal Homepage
⚡ 🌰 My algorithm training record
⚡ 现代 JavaScript 教程(The Modern JavaScript Tutorial)
Activity
Jan
15
1 week ago
started
started time in 1 week ago
Activity icon
issue

songhn233 issue comment theme-kaze/hexo-theme-kaze

songhn233
songhn233

图片无法显示或无法get

Describe the bug 描述问题 我插入的图片不显示,但是有实体可以点开,于是我建了个新的hexo测试,图片显示无异常和获取,然后改kaze-theme以后图片能正常显示但是点开是"Cannot GET /2022/01/13/test/null". 文件夹结构均为

source -test.md test abc.jpg

两次场景图片插入方式均为{% asset_img abc.jpg url_vs_uri %}, hexo的config.yml已经修改为post_asset_folder: true

To Reproduce 如何复现 安装hexo和kaze后新建post后插入图片即可

Environment 使用环境

  • Hexo Version 使用 Hexo 版本
  • Theme Version 使用主题版本

hexo: 6.0.0 hexo-cli: 4.3.0 os: win32 10.0.22000 node: 16.13.1 v8: 9.4.146.24-node.14 uv: 1.42.0 zlib: 1.2.11 brotli: 1.0.9 ares: 1.18.1 modules: 93 nghttp2: 1.45.1 napi: 8 llhttp: 6.0.4 openssl: 1.1.1l+quic cldr: 39.0 icu: 69.1 tz: 2021a unicode: 13.0 ngtcp2: 0.1.0-DEV nghttp3: 0.1.0-DEV

  • Browser 浏览器版本(可选) 96.0.4664.110

Additional context 额外信息 图片显示空白的kaze项目和对照组kaze仅改了背景色主题和post_asset_folder: true,纯对照组hexo没有任何修改,图片获取显示均正常。

songhn233
songhn233

Describe the bug 描述问题 我插入的图片不显示,但是有实体可以点开,于是我建了个新的hexo测试,图片显示无异常和获取,然后改kaze-theme以后图片能正常显示但是点开是"Cannot GET /2022/01/13/test/null". 文件夹结构均为

source -test.md test abc.jpg

我这里复现的结果是点击可以正常预览图片,但是在文章中无法显示。初步判断是 asset_img 的一些机制导致 lazyload 失败了。

建议是可以先回避使用这个特性,其他方式都可以做到类似的效果。而且这个特性依赖标签插件,实际上如果后面要迁移 markdown 文件的话这种依赖特定主题 / 引擎的扩展总归是有迁移成本的,尤其是放在图片这样基本的需求上。

可以直接在 source 中用 img folder 管理图片,也可以嵌套文件夹来实现类似的管理分类效果。

或者使用自建图床,之后用 url 方式引用图片。

started
started time in 1 week ago
Activity icon
issue

songhn233 issue comment javascript-tutorial/zh.javascript.info

songhn233
songhn233

fix: translation in 1-js/06-advanced-functions/05-global-object/article.md

目标章节:1-js/06-advanced-functions/05-global-object/article.md

当前上游最新 commit: 610fb9366e8bb4b43661f4dd564fe012cf053b91 本 PR 所做更改如下:

文件名 参考上游 commit 更改(理由)
article.md 修改翻译不准确问题
songhn233
songhn233

看起来没有问题,这个 pr 我先关了。

pull request

songhn233 pull request javascript-tutorial/zh.javascript.info

songhn233
songhn233

fix: translation in 1-js/06-advanced-functions/05-global-object/article.md

目标章节:1-js/06-advanced-functions/05-global-object/article.md

当前上游最新 commit: 610fb9366e8bb4b43661f4dd564fe012cf053b91 本 PR 所做更改如下:

文件名 参考上游 commit 更改(理由)
article.md 修改翻译不准确问题
pull request

songhn233 pull request javascript-tutorial/zh.javascript.info

songhn233
songhn233

fix(web-components/slots): querySelectorAll

fix issue: #990

pull request

songhn233 merge to javascript-tutorial/zh.javascript.info

songhn233
songhn233

fix: translation in 1-js/06-advanced-functions/03-closure/article.md

目标章节:1-js/06-advanced-functions/03-closure/

当前上游最新 commit

本 PR 所做更改如下:

文件名 参考上游 commit 更改(理由)
article.md a23882d 使其通顺
push

songhn233 push songhn233/zh.javascript.info

songhn233
songhn233

fix(web-components/slots): querySelectorAll

commit sha: f903239569e1d0f3081fc1e5452440ec4f6d4ef4

push time in 1 week ago
Activity icon
created branch

songhn233 in songhn233/zh.javascript.info create branch fix-web-component

createdAt 1 week ago
push

songhn233 push songhn233/zh.javascript.info

songhn233
songhn233

feat: 更新第一部分简介和 JavaScript 基础知识

songhn233
songhn233

Merge branch 'master' into update-part-1

songhn233
songhn233

Merge branch 'master' into update-part-1

songhn233
songhn233
songhn233
songhn233

Merge branch 'master' into update-part-1

songhn233
songhn233
songhn233
songhn233

fix: 删除 9-regular-expressions/11-regexp-groups/article.md 中多余英文原文 (#968)

songhn233
songhn233

fix: 1-js/02-first-steps/15-function-basics/article.md createForm(..) // 创建表单 (#970)

fix: '表格' -> '表单'

songhn233
songhn233

fix: 1-js/99-js-misc/01-proxy/article.md 优化翻译 (#975)

songhn233
songhn233

feat: update translation 2-ui/3-event-details/6-pointer-events/article.md #867 (#955)

  • update(pointer-events): improve, close #867

  • Create article.md

  • Update article.md

Co-authored-by: LeviDing [email protected]

songhn233
songhn233

fix: built-in trans (#961)

  • fix: built-in trans

  • Update article.md

  • Update article.md

Co-authored-by: LeviDing [email protected]

songhn233
songhn233

fix: minor typo error in 9-regular-expressions/17-regexp-methods/article.md (#977)

  • 调用 regexp.exec(str) 会返回第一个匹配项,并将紧随其后的位置保存在属性 regexp.lastIndex 中。
  • 下一次同样的调用会从位置 regexp.lastIndex 开始搜索,返回下一个匹配项,并将其后的位置保存在 regexp.lastIndex 中。
  • ...以此类推。
  • 如果没有匹配项,则 regexp.exec 返回 null,并将 regexp.lastIndex 重置为 0

排版小问题,无序列表中有两个 - 后面忘了空格,导致排版错误。

songhn233
songhn233

Merge branch 'master' into update-part-1

songhn233
songhn233

feat: update translation of 2-ui/4-forms-controls/3-events-change-input/article.md (#963)

  • update(ui/events-change-input): sync upstream

  • Update article.md

Co-authored-by: LeviDing [email protected]

songhn233
songhn233
songhn233
songhn233

Merge branch 'master' into update-part-1

commit sha: a0340f5d58dafd95c2a3ca69a67071366cf0a64a

push time in 1 week ago
Activity icon
issue

songhn233 issue comment javascript-tutorial/zh.javascript.info

songhn233
songhn233

第二部分 UI事件--4指针事件描述错误

第二部分 UI事件--4指针事件中事件:pointercancel一节中阻止浏览器的默认行为来防止 pointercancel 触发。

我们需要做两件事:中的1的的第二条描述“这种方式也适用于鼠标事件”应改为“这种方式也适用于指针事件”

songhn233
songhn233

“这种方式也适用于鼠标事件” 的意思是在这里通过设置 ondragstart 来阻止浏览器默认行为触发 pointercancel,同时对于鼠标事件这种方式也是有效的。

Activity icon
issue

songhn233 issue comment javascript-tutorial/zh.javascript.info

songhn233
songhn233

变量作用域,闭包章节 “if 内的函数” 练习题错误

应是:hello, John,而非error。

let phrase = "Hello";

if (true) {
  let user = "John";

  function sayHi() {
    alert(`${phrase}, ${user}`);
  }
}

sayHi();
songhn233
songhn233

教程中代码 默认启用严格模式

在非严格模式下不受块级作用域的限制所以会出现正常运行的情况

pull request

songhn233 merge to javascript-tutorial/zh.javascript.info

songhn233
songhn233

fix: translation in 1-js/06-advanced-functions/03-closure/article.md

目标章节:1-js/06-advanced-functions/03-closure/

当前上游最新 commit

本 PR 所做更改如下:

文件名 参考上游 commit 更改(理由)
article.md a23882d 使其通顺
open pull request

songhn233 wants to merge javascript-tutorial/zh.javascript.info

songhn233
songhn233

fix: translation in 1-js/06-advanced-functions/03-closure/article.md

目标章节:1-js/06-advanced-functions/03-closure/

当前上游最新 commit

本 PR 所做更改如下:

文件名 参考上游 commit 更改(理由)
article.md a23882d 使其通顺
songhn233
songhn233
如果一个函数是在另一个函数中创建的,该函数就被称为“嵌套”的。
pull request

songhn233 merge to javascript-tutorial/zh.javascript.info

songhn233
songhn233

fix: translation in 1-js/06-advanced-functions/03-closure/article.md

目标章节:1-js/06-advanced-functions/03-closure/

当前上游最新 commit

本 PR 所做更改如下:

文件名 参考上游 commit 更改(理由)
article.md a23882d 使其通顺
started
started time in 1 week ago
Jan
14
1 week ago
started
started time in 1 week ago
started
started time in 1 week ago
started
started time in 1 week ago
pull request

songhn233 merge to javascript-tutorial/zh.javascript.info

songhn233
songhn233

Update article.md

Getter ----->getter

目标章节:例如 1-js/01-getting-started/1-intro

当前上游最新 commit:此处填写本项目英文版 https://github.com/javascript-tutorial/en.javascript.info 的最新 commit,例如 https://github.com/javascript-tutorial/zh.javascript.info/commit/b03ca00a992a73aaf213970e71f74ac1c04def33

**本 PR 所做更改如下:Getter ----->getter 将Getter修改为getter

文件名 参考上游 commit 更改(理由)
article.md a23882d 修改部分错误

注意,参考上游 commit 是指你所修改的文件,在英文仓库中同名文件的对应 commit,即你此次提交的修改的依据。如果本 PR 你只是提交一个文字或者语句优化,并非根据上游英文仓库的修改而提交的更新,则请填无。

songhn233
songhn233

这里为什么要修改成小写呢

Jan
13
2 weeks ago
started
started time in 1 week ago
Jan
12
2 weeks ago
started
started time in 2 weeks ago
Jan
11
2 weeks ago
started
started time in 2 weeks ago
pull request

songhn233 merge to javascript-tutorial/zh.javascript.info

songhn233
songhn233

docs: 更新运算符优先级

目标章节:1-js/02-first-steps/08-operators/article.md

当前上游最新 commit:无

本 PR 所做更改如下:

文件名 参考上游 commit 更改(理由)
article.md a23882d 更新优先级
songhn233
songhn233

上游 还没有同步,从 MDN 看这个修改是正确的。

建议给原仓库先提一个 PR,然后我们这边同步。

Jan
10
2 weeks ago
started
started time in 2 weeks ago
started
started time in 2 weeks ago