feat(rallocator_cli): scroll long stack frames instead of wrapping - #667
feat(rallocator_cli): scroll long stack frames instead of wrapping#667martintmk wants to merge 3 commits into
Conversation
There was a problem hiding this comment.
Pull request overview
This PR updates the rallocator_cli HTML snapshot template styling so long stack-frame lines scroll horizontally instead of wrapping, improving readability of stack traces in generated reports.
Changes:
- Switch
.stackrendering from wrap-anywhere to horizontal scrolling (overflow-x: auto) and prevent per-frame wrapping (white-space: nowrap). - Add thin/custom scrollbar styling for
.stack(Firefoxscrollbar-*and WebKit pseudo-elements). - Ensure grid children in
.hotspot-stackscan shrink (min-width: 0) to avoid overflow issues.
💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.
Codecov Report✅ All modified and coverable lines are covered by tests. Additional details and impacted files@@ Coverage Diff @@
## main #667 +/- ##
=======================================
Coverage 100.0% 100.0%
=======================================
Files 503 503
Lines 57407 57407
=======================================
Hits 57407 57407 ☔ View full report in Codecov by Harness. 🚀 New features to boost your workflow:
|
Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com> Copilot-Session: 80ca15a7-29b1-45d6-851f-501e3c6c9f0d
There was a problem hiding this comment.
Pull request overview
Copilot reviewed 2 out of 2 changed files in this pull request and generated no new comments.
Suppressed comments (1)
justfiles/anvil/checks/doc-test.just:25
cargo metadatais being invoked without--locked, which can allow Cargo to updateCargo.lock(or otherwise resolve outside the locked set) before the subsequentcargo test --doc --lockedruns. Other repository scripts usecargo metadata ... --lockedto keep the workspace state deterministic (e.g.justfiles/extended.just:27). Consider adding--lockedhere as well.
$meta = cargo metadata --no-deps --format-version 1 | ConvertFrom-Json
Keep PR #667 scoped to the snapshot template update. Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com> Copilot-Session: 80ca15a7-29b1-45d6-851f-501e3c6c9f0d
Before:
After: