fix(web): shorten method labels in sidebar#798
Conversation
PR SummaryLow Risk Overview Refactors heading formatting into helpers and extends tests to cover compacting behavior and collision handling. Reviewed by Cursor Bugbot for commit 2e28a03. Bugbot is set up for automated code reviews on this repo. Configure here. |
|
The latest updates on your projects. Learn more about Vercel for GitHub.
|
Codecov Report❌ Patch coverage is
Additional details and impacted files@@ Coverage Diff @@
## main #798 +/- ##
==========================================
+ Coverage 82.82% 82.85% +0.02%
==========================================
Files 169 169
Lines 14670 14788 +118
Branches 1259 1270 +11
==========================================
+ Hits 12151 12253 +102
- Misses 2513 2529 +16
Partials 6 6 ☔ View full report in Codecov by Sentry. 🚀 New features to boost your workflow:
|
avivkeller
left a comment
There was a problem hiding this comment.
These are intentionally expanded, since the short version credits scenarios were multiple headings have the same navigation entry.
The correct approach to shortening these would be to revert the change that expanded them, and consolidate duplicate ToC entries into one.

The TOCs are barely usable when methods include their possibly many parameters.