Text Input | Support multiLine on validate-later - #77
Conversation
Co-authored-by: Cursor <cursoragent@cursor.com>
|
Navigate logical layers of code changes, visualize relationships, and explore their blast radius. No actionable comments were generated in the recent review. 🎉 ℹ️ Recent review info⚙️ Run configurationConfiguration used: Organization UI Review profile: CHILL Plan: Essentials Run ID: 📒 Files selected for processing (4)
Included review availability: This review used your included allowance. 4 included reviews remain after this review. Your included PR review attempts over the past 7 days set your current allowance at 5 reviews per hour. 📝 WalkthroughWalkthroughThe editor now displays the Multi-line checkbox for Priority: ⬇️ Low Merge Risk: ⚪ Minimal · up to No concrete merge-blocking defect was identified in the multiline validate-later change. 🚥 Pre-merge checks | ✅ 4✅ Passed checks (4 passed)
Comment |
Summary
multiLine=truenow works on[kind: validate-later]text inputs, not only[kind: string]. Authors writing open-ended / review-later prompts can get a textarea the same way scored string answers do.Changes
The markdown option was already parsed. The learner UI only checked
kind === 'string'before creating a<textarea>, so validate-later always rendered a single-line field. That gate now includesvalidate-later. The editor exposes the Multi-line checkbox for that kind too.New example:
data/examples/text-input-multiline-validate-later.md.Testing
text-input-multiline-validate-laterin examples mode; confirm each field is a resizable multi-line textarea[kind: validate-later]withoutmultiLinestays single-linemultiLine=truenode --test test/text-input-validation.test.js