Skip to content

refactor: components #4912

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Draft
wants to merge 50 commits into
base: main
Choose a base branch
from
Draft

Conversation

ilyaliao
Copy link
Member

@ilyaliao ilyaliao commented Jul 24, 2025

Based on #4911, I decided to systematically review every @vueuse/components component for props-related issues.

@dosubot dosubot bot added the size:S This PR changes 10-29 lines, ignoring generated files. label Jul 24, 2025
@ilyaliao ilyaliao marked this pull request as draft July 24, 2025 17:02
@ilyaliao
Copy link
Member Author

ilyaliao commented Jul 27, 2025

I found that this kind of definition can handle the types of props and slot better.

CleanShot 2025-07-27 at 18 25 09@2x

See f1907ef

@ilyaliao ilyaliao force-pushed the refactor/components branch from f1907ef to d685a59 Compare July 27, 2025 11:32
ilyaliao added 19 commits July 27, 2025 23:49
@ilyaliao ilyaliao force-pushed the refactor/components branch from d685a59 to 6305838 Compare July 27, 2025 16:00
@ilyaliao ilyaliao removed the size:S This PR changes 10-29 lines, ignoring generated files. label Jul 27, 2025
@ilyaliao ilyaliao closed this Jul 28, 2025
@ilyaliao ilyaliao deleted the refactor/components branch July 28, 2025 15:16
@ilyaliao ilyaliao restored the refactor/components branch July 28, 2025 15:17
@ilyaliao ilyaliao reopened this Jul 28, 2025
ilyaliao added 28 commits July 28, 2025 23:53
@ilyaliao ilyaliao added size:XXL This PR changes 1000+ lines, ignoring generated files. BREAKING CHANGE labels Jul 29, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
BREAKING CHANGE size:XXL This PR changes 1000+ lines, ignoring generated files.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant