Deduplicate `showSpinner()` invocation alongside `loadEditor()`

未关闭
#417 0 条评论 0 个 reaction 已指派 0 人 在 GitHub 查看

维护者通常 1 天内回复

还没有人认领这个 Issue。

评估

难度
2/5
预计耗时
1-3 小时
新手友好度
48/100
Issue 类型
重构
描述清晰度
基本清楚
活跃度
冷清
技术栈
javascript
领域
mobile-dev

调研方向

首先定位 showSpinner() 和 loadEditor() 入口点,并检查 showSpinnerPhase() 在 loadEditor() 中的调用方式。如果确认存在重复调用,则将其移除,然后验证编辑器在加载期间仍会显示其 spinner,并且现有检查均通过。

由索引模型根据 Issue 内容生成。

描述

[Type] Code Quality Good First Issue

This may be duplicative since showSpinnerPhase() is invoked within loadEditor(). Remove it?

Originally posted by @dcalhoun in https://github.com/wordpress-mobile/GutenbergKit/pull/326#discussion_r2829592044

主要语言
JavaScript
星标
29
派生
7
平均合并
2 天 12 小时
30 天内合并 PR
79

环境准备

从这里开始

  1. 先读完整个 Issue,再读项目的贡献指南。
  2. 在 Issue 下留言说明你要接手 —— 这能避免两个人做同样的事。
  3. Fork 仓库,在一个分支上完成修改。
  4. 提交 Pull Request,并在描述里引用这个 Issue 编号。

wordpress-mobile/GutenbergKit 的其他 Issue

查看 wordpress-mobile/GutenbergKit 的全部 Issue

相似的 Issue

更多 JavaScript Issue

把新 issue 发到你的邮箱

精选适合新手参与的 GitHub issue 摘要。