Skip to content

fix: increase all gate embed sizes for better readability#510

Open
arnav25102004 wants to merge 1 commit intoCircuitVerse:masterfrom
arnav25102004:fix/increase-gate-embed-size
Open

fix: increase all gate embed sizes for better readability#510
arnav25102004 wants to merge 1 commit intoCircuitVerse:masterfrom
arnav25102004:fix/increase-gate-embed-size

Conversation

@arnav25102004
Copy link
Copy Markdown

@arnav25102004 arnav25102004 commented Feb 1, 2026

Fixes #509

Changes done:

  • Increased all 9 gate embed iframes from 300x200px to 600x400px in docs/chapter4/4gates.md

All embedded circuit iframes on the Gates page were set to 300x200px which made the gates and connections difficult to see clearly. Increased all 9 iframes to 600x400px so the circuits are clearly visible and readable.

The issue mentioned only the AND gate, but every single embed on this page had the same problem so fixed them all together.

Screenshots:

BEFORE -
image

AFTER -
image

Preview Link(s):

✅️ By submitting this PR, I have verified the following

  • [+] Checked to see if a similar PR has already been opened 🤔️
  • [ +] Reviewed the contributing guidelines 🔍️
  • Sample preview link added (add a link from the checks tab after checks complete)
  • [+ ] Tried Squashing the commits into one

Summary by CodeRabbit

  • Documentation
    • Updated embedded content dimensions in chapter 4 documentation for improved visibility and readability.

✏️ Tip: You can customize this high-level summary in your review settings.

@coderabbitai
Copy link
Copy Markdown
Contributor

coderabbitai bot commented Feb 1, 2026

Walkthrough

The documentation file docs/chapter4/4gates.md has been updated to increase the dimensions of embedded iframe elements. Multiple iframe blocks have their width increased from 300px to 600px and their height increased from 200px to 400px. These changes apply to the circuit diagram embeds in the gates documentation section. The total modification consists of 18 lines added and 18 lines removed, with no changes to the core content or logic of the documentation.

🚥 Pre-merge checks | ✅ 5
✅ Passed checks (5 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed The title accurately describes the main change: increasing embedded gate sizes for better readability, which matches the core modification in the changeset.
Linked Issues check ✅ Passed The PR addresses the readability issue from #509 by increasing iframe dimensions from 300x200px to 600x400px on the gates documentation page, improving visibility as requested.
Out of Scope Changes check ✅ Passed The PR appropriately expands the scope beyond the AND gate mentioned in #509 to improve all nine gate embeds on the page, which is a reasonable extension of the core objective.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.

✨ Finishing touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Post copyable unit tests in a comment

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.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

Copy link
Copy Markdown
Contributor

@coderabbitai coderabbitai bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Actionable comments posted: 0

Caution

Some comments are outside the diff and can’t be posted inline due to platform limitations.

⚠️ Outside diff range comments (1)
docs/chapter4/4gates.md (1)

28-38: ⚠️ Potential issue | 🟡 Minor

Avoid duplicate id="projectPreview" across multiple iframes.

IDs must be unique per page; duplicates can break anchors, scripting, and accessibility tooling. Prefer unique IDs or drop the id and use a shared class instead.

✅ Suggested fix (unique ids or class)
-  id="projectPreview"
+  class="project-preview"
-  id="projectPreview"
+  id="projectPreview-and-gate"

Also applies to: 43-54, 71-82, 99-110, 125-136, 153-162, 179-190, 207-218, 235-246

🧹 Nitpick comments (1)
docs/chapter4/4gates.md (1)

28-31: Consider responsive sizing to avoid mobile overflow.

Fixed 600×400px can overflow small screens. A responsive width with a max-width and aspect ratio keeps readability without breaking layout.

💡 Suggested approach (CSS class)
-  width="600px"
-  height="400px"
+  class="project-preview"
.project-preview {
  width: 100%;
  max-width: 600px;
  aspect-ratio: 3 / 2;
  height: auto;
}

Also applies to: 43-46, 71-74, 99-102, 125-128, 153-156, 179-182, 207-210, 235-238

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

AND gate circuit diagram appears too small to read

1 participant