Docs/updates: Improve technical accuracy and clarity of Verilog documentation#533
Docs/updates: Improve technical accuracy and clarity of Verilog documentation#533SouparnaChatterjee wants to merge 21 commits intoCircuitVerse:masterfrom
Conversation
…CircuitVerse#462) * Style: add custom.css and update docusaurus.config.js for responsive iframes * removed !important from iframe css * format css file --------- Co-authored-by: Nihal <65150640+Nihal4777@users.noreply.github.com>
|
No actionable comments were generated in the recent review. 🎉 ℹ️ Recent review infoConfiguration used: Organization UI Review profile: CHILL Plan: Pro 📒 Files selected for processing (1)
💤 Files with no reviewable changes (1)
WalkthroughDeletes Possibly related PRs
🚥 Pre-merge checks | ✅ 4 | ❌ 1❌ Failed checks (1 warning)
✅ Passed checks (4 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 |
There was a problem hiding this comment.
Actionable comments posted: 1
🧹 Nitpick comments (2)
docs/chapter1/1introduction.mdx (1)
14-17: Use semantic<figure>+<figcaption>for consistency with the rest of this PR.These captioned images are currently split into markdown image + italic text. Converting to semantic figure markup will align with
src/css/custom.cssand improve consistency.Also applies to: 30-33, 40-43
docs/chapter8/1circuittoverilog.md (1)
32-35: Consider using<figure>/<figcaption>instead of centered<div><em>...captions.This would standardize media semantics across docs and reuse the new figure styles more consistently.
Also applies to: 44-47
ℹ️ Review info
Configuration used: Organization UI
Review profile: CHILL
Plan: Pro
📒 Files selected for processing (4)
docs/chapter1/1introduction.mddocs/chapter1/1introduction.mdxdocs/chapter8/1circuittoverilog.mdsrc/css/custom.css
Fixes #531
Changes done:
Screenshots:
N/A (Documentation-only changes)
By submitting this PR, I have verified the following
Summary by CodeRabbit
Documentation
Style