fix(site): repair lesson table rendering#231
Conversation
|
No actionable comments were generated in the recent review. 🎉 ℹ️ Recent review info⚙️ Run configurationConfiguration used: Path: .coderabbit.yaml Review profile: CHILL Plan: Pro Run ID: 📒 Files selected for processing (1)
📝 WalkthroughWalkthroughCSS styling improvements and markdown table parser robustness updates fix table rendering issues. Table containers now enable touch-friendly scrolling with auto layout and explicit min-widths; cells disable hyphenation and refine text wrapping. A new markdown parser uses helper functions to correctly split pipe-delimited rows while respecting escaped pipes and inline code, then normalizes rows to the header column count. ChangesTable Rendering Fix
🎯 3 (Moderate) | ⏱️ ~25 minutes Possibly related PRs
🚥 Pre-merge checks | ✅ 5✅ Passed checks (5 passed)
✏️ Tip: You can configure your own custom pre-merge checks in the settings. ✨ Finishing Touches🧪 Generate unit tests (beta)
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
What this PR does
Fixes lesson table rendering when markdown table cells contain escaped pipes like
P(X\|Y)or pipes inside inline code spans.This resolves the broken table layout reported in #193 for the Naive Bayes lesson and applies the fix to the shared lesson renderer so similar tables across the curriculum render
correctly.
Kind of change
Details
\|pipesValidation
phases/02-ml-fundamentals/14-naive-bayespython3 scripts/audit_lessons.pypython3 scripts/check_readme_counts.pygit diff --checkFixes #193