Skip to content

fix: 流量超额、离线时段、手机标题等显示问题;清理过时注释与未用代码 - #21

Merged
stqfdyr merged 1 commit into
mainfrom
chore/code-review
Oct 2, 2026
Merged

stqfdyr merged 1 commit into
mainfrom
chore/code-review

Conversation

@stqfdyr

@stqfdyr stqfdyr commented Oct 2, 2026 •

Copy link
Copy Markdown
Collaborator

全量读了一遍默认主题,修几处显示问题,清理与代码不符的注释和用不到的代码。

修复

  • 流量超过额度时卡片显示真实百分比(如 212%),进度条仍止于满格;原来封顶显示 100%
  • 资源图在节点离线的时段断开,原来把离线前后两个点连成一条直线,看起来像这段时间一直有数据。点间隔超过常规间隔(取间隔的中位数,偶数个时取较小的那个)两倍才断开,所以上报间隔较长的 agent、偶尔缺一个点都不会断
  • 详情页节点名放不下时,状态和 agent 徽章换到下一行;原来手机上名字被两个徽章挤到只剩两三个字
  • 列表页点站名不再重复写入历史记录,原来每点一次就要多按一次返回
  • 时间范围、图表页签、探测图例按钮加上 aria-pressed

清理

  • 删掉没用到的 tw-animate-css、--popover、--input,以及 Badge、Button 里用不到的变体(做法与已精简的 Card 相同),CSS 从 30.5 kB 降到 23.7 kB
  • 五个提示框共用一份设置;去掉多余的 clock 包装和 axisTop 不会走到的分支,axisBytes 复用 unitOf
  • 注释改为与现状一致:--ok/--warn 的用途、入口包体积、断线后的轮询间隔、时间格式化的说明、规格区说明、削峰窗口的取样间隔
  • README:历史窗口上限改为 hub 的保留天数,/api/me 补上保留天数,npm test 的覆盖范围

测试

  • npm run lint、npm test(UTC 与纽约时区)、npm run build 通过;新增百分比不封顶与断线的用例,改回封顶、阈值改为一倍、中位数改为最小值或较大的中位数时各自变红
  • 预览环境浏览器实测:
    • 徽章与按钮在浅色、深色、列表页、详情页与悬停状态下的计算样式与改动前逐项相同
    • 额度 2 GB、已用 4.18 GB 的节点显示 212%
    • 预览 agent 停过三次,1 天窗口的每条曲线断成 4 段,空档里悬停不显示数值;其余时段没有误断
    • 详情页标题在 320、390、1280 宽下,短名、常见长度、超长名都没有横向溢出;常见长度的名字在手机上完整显示,桌面仍是一行
    • 在列表页连点三次站名,历史记录条数不变;从详情页回到列表后,按两次返回依次回到详情页和列表首页
    • 延迟图的提示框、丢包标注、图例隐藏正常

- 流量超过额度时显示真实百分比(如 212%),进度条仍止于满格
- 资源图在节点离线的时段断开,不再画成一条直线;间隔超过常规间隔(取中位数)两倍才断开,上报间隔较长的 agent 不受影响
- 详情页节点名过长时,状态与 agent 徽章换到下一行,手机上名字不再被挤成两个字
- 列表页点站名不再重复写入历史记录,返回键一次就能离开
- 时间范围、图表页签与探测图例按钮标明按下状态
- 删掉未使用的 tw-animate-css、--popover 与 --input,以及 Badge、Button 里用不到的变体,CSS 从 30.5 kB 降到 23.7 kB
- 五个提示框共用一份设置;去掉多余的 clock 包装与 axisTop 的死分支,axisBytes 复用 unitOf
- 注释与 README 对齐现状:配色变量的用途、包体积、轮询间隔、刻度格式、规格区说明、历史窗口上限
@stqfdyr
stqfdyr force-pushed the chore/code-review branch from 1ca27ac to 46d88d3 Compare October 2, 2026 05:29
@stqfdyr
stqfdyr merged commit f2ff2d2 into main Oct 2, 2026
1 check passed
@stqfdyr
stqfdyr deleted the chore/code-review branch October 2, 2026 05:54
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant