Improve heading semantics

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

维护者通常 9 天内回复

@stephaniehobson 已经在做这个了。

开始于 2026年11月4日。

  • #475 来自 @stephaniehobson —— 未关闭

评估

难度
1/5
预计耗时
1 小时以内
新手友好度
50/100
Issue 类型
缺陷
描述清晰度
描述清楚
活跃度
停滞
技术栈
javascript

调研方向

Open src/components/views/MetricOverview.js at the referenced heading and compare it with the surrounding heading structure shown in the issue. Change the heading level only if needed to remove the h3-to-h5 jump, then verify the metrics page no longer triggers the aXe heading-level warning.

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

描述

to do

aXe is telling me on https://metrics.mozilla.com/protected/usage-report-demo/dashboard/user-activity that "Heading levels should only increase by one". I think this is because we have an <h3> 👉 <h5>, but not an intermediate <h4> tag.

<h3 id="metrics-heading">Metrics</h3>
...
<h5 class="striped"><span>Yearly Active Users</span></h5>

I reckon we should possibly bump this from an H5 to an H4:

https://github.com/mozilla/ensemble/blob/9ebbb3dba9c0b82a674b3482a7288c46b20de0c2/src/components/views/MetricOverview.js#L91

主要语言
JavaScript
星标
21
派生
7
平均合并
7 天 6 小时
30 天内合并 PR
5

环境准备

  • 没有 Dockerfile 或 Docker Compose 文件
  • 没有 Pull Request 模板
  • 阅读贡献指南

从这里开始

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

mozilla/ensemble 的其他 Issue

查看 mozilla/ensemble 的全部 Issue

相似的 Issue

更多 JavaScript Issue

把新 issue 发到你的邮箱

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