Skip to content

feat(style): record own visibility after cascade - #2

Open
lanyue-llk wants to merge 4 commits into
moli-update-main-20261018from
codex/moli-style-provenance
Open

lanyue-llk wants to merge 4 commits into
moli-update-main-20261018from
codex/moli-style-provenance

Conversation

@lanyue-llk

@lanyue-llk lanyue-llk commented Sep 29, 2026 •

Copy link
Copy Markdown

Why

Moli's Markdown export distinguishes an inherited page-loading veil from an explicit hidden subtree. Only the resolved cascade can tell whether the element establishes its own visibility value after variable substitution and cascade rollback.

Change

Record HAS_OWN_VISIBILITY when the winning declaration is applied. This is a read-only style fact; consumers do not parse stylesheets or repeat the cascade.

The base is moli-update-main-20261018, the Stylo revision already pinned by Moli main (973988b9). The diff contains only the flag and its cascade assignment. The earlier animation-opacity API has been removed because current document export does not use paint effects to select source content.

Consumer and validation

Moli #1337 consumes the flag in a single style-observation batch. Its page-export tests cover inherited visibility, explicit hidden descendants, variable substitution, cascade rollback, disclosure panels and ordinary article siblings. All 28 Markdown renderer tests and affected-crate Clippy pass against this revision; the published historical benchmark remains bound to its original binary.

@lanyue-llk

Copy link
Copy Markdown
Author

已将本 PR 的两项修改完整归并到 #1,并保留独立提交与原有验证记录。后续评审和 Moli 依赖统一以 #1 为入口,关闭此重复 PR。

@lanyue-llk lanyue-llk closed this Sep 30, 2026
@lanyue-llk lanyue-llk reopened this Sep 30, 2026
@lanyue-llk

Copy link
Copy Markdown
Author

更正此前“已归并到 #1、#2 重复”的说明:复核后确认两项 PR 解决的是不同问题。#1 是 preferred text scale 环境支持;本 PR #2 提供 Moli #919 实际需要的 resolved visibility / final opacity content provenance,不能由 #1 替代。

本 PR 已同步 Stylo moli 当前提交,最新 head 为 1032a2947ec2f7cee1027d07084203be87e17643,Linux debug/release、macOS debug、Windows debug 与汇总门禁均通过。Moli #919 已精确锁定该提交。此前评论作废,以当前 PR body 与本条更正为准。

@lanyue-llk
lanyue-llk force-pushed the codex/moli-style-provenance branch from 1032a29 to e2baa14 Compare October 22, 2026 04:11
@lanyue-llk
lanyue-llk changed the base branch from moli to moli-update-main-20261018 October 22, 2026 04:12
@lanyue-llk lanyue-llk changed the title feat(style): expose resolved content provenance feat(style): record own visibility after cascade Oct 22, 2026
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