chore(widget): tests de la retención tras reordenar (#240)

已关闭
  • 哪里不对？
#241 3 条评论 0 个 reaction 已指派 1 人 在 GitHub 查看

维护者通常 1 天内回复

@DonatoSilva 已经在做这个了。

开始于 2026年11月5日。

  • #242 来自 @DonatoSilva —— 已合并

评估

这个 Issue 还没有评估数据。

描述

status:approved type:chore
Contexto

Seguimiento de #240 (arreglo de #222). El comportamiento es correcto y está integrado; esta issue solo mejora sus tests. No es un bug.

Problema

  1. Funciones sin lógica propia. holdAfterReorderDrop, releaseOnStripLeave y canPointerOpen (src/lib/anchor-panel.ts) envuelven un booleano (x => x, _ => false, h => !h), así que sus tests no pueden fallar. Los que comprueban la aridad (canPointerOpen.length, hoverPanel.length, clickPanel.length) y «never changes the panel state» (src/lib/anchor-panel.test.ts) no fijan ningún comportamiento.

  2. La lógica con bordes reales no tiene test. La comprobación de «dentro de la ventana» al soltar un reordenado está en línea en src/Widget.tsx:1219:

    e.clientX >= 0 && e.clientY >= 0 && e.clientX < window.innerWidth && e.clientY < window.innerHeight

    Un error de < frente a <= dejaría la retención activa sin que nada la libere, y el hover no volvería a abrir el panel hasta salir de la tira.

Propuesta
  • Dejar la retención como un useRef<boolean> en Widget.tsx y quitar las tres funciones envoltorio y sus tests.
  • Extraer pointerInsideViewport(x, y, width, height) como función pura y cubrir los bordes en ambos ejes: 0, -1, width - 1 y width (y lo mismo con height).
  • Mantener el test de ProviderGauge sobre el enrutado de mouseenter a onPointerEnter, que sí aporta.
Criterios
  • Sin cambio de comportamiento: mismas condiciones de retención y liberación que #240.
  • npm test, npm run typecheck y npm run build en verde.
Fuera de alcance

Cambiar la regla de #222, el temporizador de 400 ms (REORDER_GUARD_MS) o el cableado de foco y clic.

主要语言
TypeScript
星标
10
派生
2
平均合并
8 小时 30 分钟
30 天内合并 PR
119

环境准备

这个项目没有提供开发容器、Dockerfile 或贡献指南，环境需要你自己搭建：先看它的 README，通用步骤见我们的新手贡献指南。

从这里开始

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

egdev6/quorb 的其他 Issue

查看 egdev6/quorb 的全部 Issue

相似的 Issue

更多 TypeScript Issue

把新 issue 发到你的邮箱

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