feat(Textarea): expose autoResize method#6120
Conversation
Be able to call autoResize without forcefully triggering an Input (which could have other functional implications e.g. making api requests on input events).
📝 WalkthroughWalkthroughThe Estimated code review effort🎯 2 (Simple) | ⏱️ ~10 minutes 🚥 Pre-merge checks | ✅ 3✅ Passed checks (3 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 |
commit: |
autoResize method
benjamincanac
left a comment
There was a problem hiding this comment.
Looks good to me, however we need to update the ### Expose section in textarea.md.
There was a problem hiding this comment.
Caution
Some comments are outside the diff and can’t be posted inline due to platform limitations.
⚠️ Outside diff range comments (1)
docs/content/docs/2.components/textarea.md (1)
257-264:⚠️ Potential issue | 🟡 MinorAdd a
Soonbadge to this newly introduced Expose API update.This PR documents a new feature (
autoResizeexposure), so the updated docs heading should include the release-gap badge (e.g., on### Expose) to avoid implying immediate npm availability.As per coding guidelines: “
docs/**/*.md: Add:badge{label="Soon" class="align-text-top"}to docs headings when introducing new features or fixes, as the docs deploy on merge but features ship on next npm release”.🤖 Prompt for AI Agents
Verify each finding against the current code and only fix it if needed. In `@docs/content/docs/2.components/textarea.md` around lines 257 - 264, The "### Expose" heading for the new Expose API (showing textareaRef and autoResize) is missing the required Soon badge; update the heading text "### Expose" to include :badge{label="Soon" class="align-text-top"} so the docs reflect the release gap for the new autoResize exposure (references: heading "### Expose", exported names `textareaRef` and `autoResize` in the table).
🤖 Prompt for all review comments with AI agents
Verify each finding against the current code and only fix it if needed.
Outside diff comments:
In `@docs/content/docs/2.components/textarea.md`:
- Around line 257-264: The "### Expose" heading for the new Expose API (showing
textareaRef and autoResize) is missing the required Soon badge; update the
heading text "### Expose" to include :badge{label="Soon" class="align-text-top"}
so the docs reflect the release gap for the new autoResize exposure (references:
heading "### Expose", exported names `textareaRef` and `autoResize` in the
table).
ℹ️ Review info
⚙️ Run configuration
Configuration used: Organization UI
Review profile: CHILL
Plan: Pro
Run ID: bbc49413-572d-49f2-8ed5-b8ff0c4c1e4f
📒 Files selected for processing (1)
docs/content/docs/2.components/textarea.md
Be able to call autoResize without forcefully triggering an Input (which could have other functional implications e.g. making api requests on input events).
❓ Type of change
📚 Description
Say you have a Textarea containing text and the window is resized; the autoResize will not be triggered (there are other cases too).
This change will allow you to do something like this:
Without manually triggering, say, an input event, which could have other implications when using a manual
@update:modelValue"...".📝 Checklist