fix(verilog-editor): enable scrolling in Verilog code editor for large files#850
fix(verilog-editor): enable scrolling in Verilog code editor for large files#850Nihal4777 merged 2 commits intoCircuitVerse:mainfrom
Conversation
✅ Deploy Preview for circuitverse ready!
To edit notification comments on pull requests, go to your Netlify project configuration. |
WalkthroughThis pull request modifies the CSS overflow properties in VerilogEditorPanel.vue to enable scrolling functionality in the Verilog code editor. The changes replace 🚥 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 |
|
@tachyons @Nihal4777 kindly review |
|
@Nihal4777 can you please review |
Nihal4777
left a comment
There was a problem hiding this comment.
Thanks @Me-Priyank for this fix.

Fixes #849
Describe the changes you have made in this PR :
This PR fixes a scrolling issue in the Verilog code editor. Previously, users could only scroll by dragging the scrollbar - mouse wheel, arrow keys, and Page Up/Down did not work.
Root Cause:
The CSS rules in VerilogEditorPanel.vue set
overflow: hidden !importanton multiple elements, blocking all scroll input methods except direct scrollbar manipulation.Fix:
Changed CSS overflow properties to allow proper scrolling:
.code-windowand.code-window .CodeMirror:overflow: visible(content flows naturally).code-window .CodeMirror-scroll:overflow: auto(single scrollbar, responds to all inputs)Screenshots of the UI changes (If any) -
N/A - This is a scrolling behavior fix, no visual changes.
Code Understanding and AI Usage:
Did you use AI assistance (ChatGPT, Claude, Copilot, etc.) to write any part of this code?
If you used AI assistance:
Explain your implementation approach:
Problem: The Verilog code editor was unusable for large files because scrolling only worked by dragging the scrollbar.
Root cause analysis: Found that
overflow: hidden !importantwas set on:.code-window.code-window .CodeMirror.code-window .CodeMirror-scrollThis blocked all scroll events except native scrollbar drag.
Solution approach:
.code-window,.CodeMirror) tooverflow: visible- content flows naturally.CodeMirror-scrollwithoverflow: auto- this is the correct scrolling containerWhy this specific implementation:
Checklist before requesting a review
Note: Please check Allow edits from maintainers if you would like us to assist in the PR.
Summary by CodeRabbit
✏️ Tip: You can customize this high-level summary in your review settings.