Conversation
|
Azure Pipelines: 1 pipeline(s) were filtered out due to trigger conditions. |
commit: |
|
Size Change: +239 B (+0.02%) Total Size: 1.47 MB 📦 View Changed
ℹ️ View Unchanged
|
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
Reviewed commit: af52f8ee31
ℹ️ About Codex in GitHub
Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you
- Open a pull request for review
- Mark a draft as ready
- Comment "@codex review".
If Codex has suggestions, it will comment; otherwise it will react with 👍.
Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".
Coverage Report
File Coverage
|
||||||||||||||||||||||||||||||||||||||
|
Note Reviews pausedIt looks like this branch is under active development. To avoid overwhelming you with review comments due to an influx of new commits, CodeRabbit has automatically paused this review. You can configure this behavior by changing the Use the following commands to manage reviews:
Use the checkboxes below for quick actions:
No actionable comments were generated in the recent review. 🎉 ℹ️ Recent review info⚙️ Run configurationConfiguration used: defaults Review profile: CHILL Plan: Pro Run ID: 📒 Files selected for processing (1)
🚧 Files skipped from review as they are similar to previous changes (1)
📝 WalkthroughWalkthroughAdjusted textarea autosize handling: added a Changes
Estimated code review effort🎯 2 (Simple) | ⏱️ ~10 minutes Suggested labels
Suggested reviewers
Poem
🚥 Pre-merge checks | ✅ 4 | ❌ 1❌ Failed checks (1 warning)
✅ Passed checks (4 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 |
|
🧪 Playground Preview: https://element-plus.run/?pr=24078 |
There was a problem hiding this comment.
Actionable comments posted: 1
🤖 Prompt for all review comments with AI agents
Verify each finding against the current code and only fix it if needed.
Inline comments:
In `@packages/components/input/src/input.vue`:
- Around line 603-605: The isResizingTextarea flag can remain true if a mouseup
occurs outside the window; add robust fallback resets by also listening for
pointerup, window blur, and document visibilitychange to set
isResizingTextarea.value = false. Update the useEventListener calls around the
textarea resize logic (the existing useEventListener('mouseup', ...) handler and
related resize handlers) to also register handlers for 'pointerup', 'blur' (on
window), and 'visibilitychange' (on document) that clear isResizingTextarea, and
ensure these listeners use the same cleanup lifecycle as the existing
useEventListener usage; apply the same change to the other similar blocks
referenced (the blocks around the other useEventListener calls).
🪄 Autofix (Beta)
Fix all unresolved CodeRabbit comments on this PR:
- Push a commit to this branch (recommended)
- Create a new PR with the fixes
ℹ️ Review info
⚙️ Run configuration
Configuration used: defaults
Review profile: CHILL
Plan: Pro
Run ID: 79a21dec-d65a-4f31-bd33-dbc2cee53aaa
📒 Files selected for processing (1)
packages/components/input/src/input.vue
|
Azure Pipelines: 1 pipeline(s) were filtered out due to trigger conditions. |

Please make sure these boxes are checked before submitting your PR, thank you!
devbranch.fix #24076
This happens because resizing triggers frequent
mouseenterandmouseleaveevents as the cursor moves, causinghovering.valueto change repeatedly. As a result, the clear icon is constantly created and destroyed, leading to layout reflow.There is no event for detecting textarea resize, so this is a workaround. If there’s a better approach, please let me know.
Summary by CodeRabbit