Conversation
Muniya-64bit
commented
Feb 13, 2026
- Replaces code mirror using Monaco editor
- UI update obo code mobile view now elements fit into screen
- Monaco added as a comman package
- Setup in packages as a comman component
- removed mini map
AnasSAV
left a comment
There was a problem hiding this comment.
- Give Reasoning/fixes for the changes requested
| [onChange] | ||
| ); | ||
|
|
||
| // Suppress Monaco Editor cancellation errors |
There was a problem hiding this comment.
- Why are we globally suppressing errors?
There was a problem hiding this comment.
Error: Canceled
at Object.canceled (monaco.js:...)
at processRequest (monaco.js:...)
...dozens of these per second while editing
monaco normally throws these kind of harmless errors.
I should have supress them in globally it affect to whole system fixed it now only supress silent errors by monaco
| object-fit: contain; | ||
| } | ||
|
|
||
| /* Hide Monaco Editor iPad Keyboard Widget */ |
There was a problem hiding this comment.
- Consider conditional rendering based on device detection
There was a problem hiding this comment.
- This is a small widget comes with monaco to access keyboard but in mobile displays its kind block. So ia removed it .
Critical Changes in overallIncorrect Power Operator - generator.ts:183Uses ^ instead of ** for Python power operation const code = value_left + " ^ " + value_right; // WRONG - should be ** Division by Zero Not Handled - generator.ts:183Division block generates code without safety checks SQL-Injection Style Issue in String Block - generator.ts:183No escaping of quotes in user input const code = '"' + text_input + '"'; // Breaks if text_input contains " Function Definition in Obo Blocks
As the function is defined it doesnt show in UI |
Feature Testing Rubric – Monaco Editor MigrationFeature Overview
Scoring Guide
1. Requirements & UI Coverage
2. Functional UI TestingOBO Blocks
OBO Code
3. Editor Features Parity CheckCompare with previous CodeMirror functionality:
4. Input Validation & Edge Cases
Devices & Viewports
6. Performance TestingRuntime Performance
7. Critical Issues from Code ReviewMust be tested/verified:
8. Migration Checklist
Tester Sign-off
|

