Hacktoberfest 2026: le issue che i maintainer hanno segnato per ottobre, aperte e adatte ai principianti. Sfoglia le issue Hacktoberfest

组件内部 zIndex 抬升子树排序键,根级浮层(scrim/下拉菜单)失去命中与绘制优先级

Chiusa Adatta ai principianti
#95 0 commenti 0 reazioni 0 assegnatari Vedi su GitHub

I maintainer di solito rispondono entro 2 giorni

Nessuno ha ancora preso questa issue.

Valutazione

Difficoltà
2/5
Tempo stimato
Meno di un'ora
Idoneità per principianti
78/100
Tipo di issue
Bug
Chiarezza
Specificata chiaramente
Stato di attività
Attiva
Stack tecnologico
cpp
Ambito
desktop-dev

Direzione di ricerca

Inizia da components/slider.h, dove il rettangolo .hit viene costruito con uno zIndex fisso nel codice, e verifica come rebuildOrderedElements in core/dsl.h ordina i figli in base a subtreeMaxZIndex. Il test di regressione è tests/unit/slider_subtree_z_order.cpp; compilalo ed eseguilo per primo, così da vedere l'asserzione che fallisce. È completato quando page->subtreeMaxZIndex vale 0 e l'overlay viene ordinato dopo la pagina, in modo che il test passi senza modificare gli altri comportamenti dello slider.

Scritto dal modello di indicizzazione a partire dal testo della issue.

Descrizione

现象

宿主应用(节点编辑器画布)中,页面内容包含滑条组件后,合成在其后的根级浮层——全窗口 scrim 与下拉菜单面板——完全失去命中优先级:点击 scrim/菜单项时,事件落到页面底层的输入区(画布 mouseArea),浮层被意外收起、菜单项永远无法选中。绘制序同样被压到页面之下。

宿主真机 A/B 实证(同一画布、同一浮层代码,唯一变量是页面内是否存在滑条参数节点):

  • 页面含滑条节点 → 点击浮层菜单项的坐标,事件穿透到画布输入区;
  • 页面不含滑条节点(无参数的节点)→ 同一坐标由 scrim 正确拦截并收起浮层。

根因

命中测试与绘制的遍历顺序按 orderedChildren / orderedRoots 排序,排序键是 subtreeMaxZIndex(core/dsl.h 的 rebuildOrderedElements,升序排序、命中逆序遍历)。

而 components/slider.h 的内部热区自带硬编码 zIndex:

ui_.rect(id_ + ".hit")
    .size(width_, height_)
    .states(...)
    .zIndex(10)          // ← 抬升整个滑条子树的排序键
    .interactive()
    .sliderInputFrom(id_)
    .build();

滑条栈内 .hit 的输入优先级本就由合成序保证(栈内最后一个子元素,逆序命中时最先被测试),这个 zIndex(10) 没有栈内收益,却把页面子树的 max z 抬到 10——所有默认 z=0 的根级浮层在根级兄弟排序中被排到页面之后(命中先测试页面、绘制后画页面),浮层即失效。

两个连带推论:

  1. 由于排序只看 z 不看合成序,浮层即便晚于页面合成也无法挽回;
  2. 同理,滑条所在子树还会压过其后合成的兄弟子树中 z=0 的交互元素(如后画到滑条上方的卡片控件),不只是根级浮层场景。

最小复现(单元级)

root stack → page stack(内含一个 components::slider)→ overlay rect(onClick,z 默认 0,后合成)。
ui.layout(...) 后:page->subtreeMaxZIndex == 10,且 root 的 orderedChildren 中 overlay 被排到 page 之前——逆序命中遍历先测 page。修复后两者均为 0、稳定序保持合成序(overlay 在后,逆序命中先达)。

(已随修复提交回归测试 tests/unit/slider_subtree_z_order.cpp:修复前该测试在第一条断言失败——slider inflates page subtreeMaxZIndex to 10。)

影响

凡页面内容包含滑条,宿主的模态浮层(下拉、右键菜单、scrim)、以及页面上后合成的 z=0 交互控件,命中与绘制都被滑条子树压过。组件库若未来有更多组件内部使用 zIndex,同类风险会扩大。

建议修复

移除 slider.h .hit 的 .zIndex(10)(其栈内优先级由合成序保证,行为不变)。随后提 PR,附上述回归测试。

Lingua principale
C++
Stelle
2k
Fork
177
Merge medio
19h 58m
PR unite (30g)
9

Preparare l'ambiente

Questo progetto non fornisce container di sviluppo, Dockerfile né guida per i contributori, quindi l'ambiente è a tuo carico: parti dal suo README e consulta la nostra guida al primo contributo per i passaggi generali.

Come iniziare

  1. Leggi tutta la issue e poi la guida ai contributi del progetto.
  2. Commenta sulla issue per dire che te ne occupi tu — evita che due persone facciano lo stesso lavoro.
  3. Fai un fork del repository e lavora su un branch.
  4. Apri una pull request che faccia riferimento al numero della issue.

Altre issue di sudoevolve/EUI-NEO

Tutte le issue di sudoevolve/EUI-NEO

Issue simili

Altre issue su C++

Ricevi le nuove issue nella tua casella

Un breve riepilogo di issue GitHub adatte ai principianti.