feat: show current zoom level while zooming#9689
Conversation
Co-Authored-By: Kait <100063447+gurpreetkaits@users.noreply.github.com> Co-Authored-By: Oz <oz-agent@warp.dev>
aa32027 to
dae1131
Compare
|
@oz-for-oss[bot] I'm starting a first review of this spec-only pull request. You can view the conversation on Warp. I completed the review and posted feedback on this pull request. Comment Powered by Oz |
There was a problem hiding this comment.
Overview
This spec PR adds product and technical specs for a transient, non-interactive zoom-level HUD shown after user-initiated zoom actions. The product spec covers goals, non-goals, UX behavior, boundaries, validation, and open design questions; the tech spec maps the behavior to the existing zoom model, workspace action paths, overlay rendering approach, tests, risks, and follow-ups.
Concerns
- No blocking concerns found for completeness, clarity, feasibility, issue alignment, internal consistency, or high-level security.
Verdict
Found: 0 critical, 0 important, 0 suggestions
Approve
Comment /oz-review on this pull request to retrigger a review (up to 3 times on the same pull request).
Powered by Oz
Co-Authored-By: Oz <oz-agent@warp.dev> Co-Authored-By: Zach Lloyd <zachlloyd@users.noreply.github.com>
Closes #9576
Summary
Validation
cargo test --manifest-path /workspace/warp/Cargo.toml -p warp adjusted_zoom_levelcargo fmt --manifest-path /workspace/warp/Cargo.toml -- app/src/workspace/view.rs app/src/workspace/view/zoom_level_hud.rscould not run becausecargo-fmt/rustfmtis not installed in this environment.