Add file identity icons and animated transcript tool activity - #329
Conversation
5e8a0ea to
9595f23
Compare
|
imo the tree segment lines from the left are animated nicely, but it adds a bit too much clutter to the minimal aesthetic. |
b6f6a96 to
c578194
Compare
|
CI investigation for head
The evidence strongly suggests this PR did not introduce the iOS failure. It does not distinguish a sampling/timing issue from a real intermittent iOS animation/layout defect: the assertion measures presentation-layer geometry. Suggested next step: inspect the retained |
wingleeio
left a comment
There was a problem hiding this comment.
No blocking findings. Reviewed icon resolution/assets, Markdown decoration, and tool activity/fold lifecycle. The full UI suite passes locally: 923 tests on the PR head and 923 after integrating current main (2704615). Changed-file formatting passes apart from pre-existing formatting reached through child modules; diff checks pass. The original iOS failure is unrelated to this desktop change: apps/ios has the identical Git tree 5124e4f6d1a05b32e8892ca4aa168b109ddea229 in this PR, main, and #341, where the same testToolGroupsRevealAndCollapseThroughIntermediateHeights test passes (job 103807982096). All original desktop, browser, sync, and networking checks pass. The failed iOS job has also been rerun.
Summary
Validation
cargo test -p zeron-ui --lib --quiet: 896 passed.rustfmt --edition 2024 --check crates/ui/src/transcript.rsandgit diff --check: passed.cargo fmt -p zeron-ui -- --checkstill reports existing unrelated formatting drift in UI examples, browser, and terminal files.Compatibility and attribution
Symbols artwork and its filename/folder associations are pinned to
296ef1b62287fb2315cb5651e552e09e8c8e1de8; its MIT license and notice are included in the bundled assets andTHIRD_PARTY_NOTICES.md. Icons are decorative alongside their text labels. No engine, protocol, data migration, or settings changes.