Skip to content

Conversation

@aiktb
Copy link

@aiktb aiktb commented Dec 25, 2025

I tested this on the latest versions of Vue (3.5.24) and vue-tsc (3.1.4). ComponentCustomProperties does not work for registering global directives. Instead, you should use GlobalDirectives.

Copilot AI and others added 5 commits December 24, 2025 15:51
…tion

Co-authored-by: aiktb <104723527+aiktb@users.noreply.github.com>
Co-authored-by: aiktb <104723527+aiktb@users.noreply.github.com>
Fix: Use GlobalDirectives interface for typing global custom directives
Copilot AI review requested due to automatic review settings December 25, 2025 15:16
@netlify
Copy link

netlify bot commented Dec 25, 2025

Deploy Preview for vuejs ready!

Name Link
🔨 Latest commit 6bdf2f5
🔍 Latest deploy log https://app.netlify.com/projects/vuejs/deploys/694d55423d0ba60008ffd7f5
😎 Deploy Preview https://deploy-preview-3325--vuejs.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify project configuration.

Copy link

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

This PR corrects the TypeScript documentation for typing global custom directives in Vue applications. The fix addresses an incorrect reference to ComponentCustomProperties, replacing it with the correct GlobalDirectives interface.

  • Updates the TypeScript interface reference from ComponentCustomProperties to GlobalDirectives for typing global custom directives
  • Ensures consistency between the documentation text and code examples

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

@aiktb aiktb changed the title fix: correct TypeScript interface for typing global custom directives(via GlobalDirectives) docs: correct TypeScript interface for typing global custom directives(via GlobalDirectives) Dec 26, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant