Skip to content

Update incorrect GettingStarted CDN instructions#1852

Merged
mattelen merged 1 commit intoshentao:masterfrom
rockisch:master
Feb 24, 2026
Merged

Update incorrect GettingStarted CDN instructions#1852
mattelen merged 1 commit intoshentao:masterfrom
rockisch:master

Conversation

@rockisch
Copy link
Copy Markdown
Contributor

I noticed through this issue that the documentation in the main page of the project is incorrect, since it mentions using window.VueMultiselect.

Afaik, the project moved from exposing itself from window.VueMultiselect to window['vue-multiselect'] when it transitioned to 3.X.

The project moved from exposing itself from  "window.VueMultiselect" to
"window['vue-multiselect']" when it transictioned to 3.0.
@awubc
Copy link
Copy Markdown

awubc commented Mar 4, 2025

This code worked for me.

@jkewley
Copy link
Copy Markdown

jkewley commented Jul 10, 2025

Project maintainers should accept this commit and fix the documentation, This has bitten me twice now.

@mattelen mattelen requested review from Copilot and mattelen and removed request for mattelen February 23, 2026 17:07
@mattelen mattelen self-requested a review February 23, 2026 17:08
Copy link
Copy Markdown
Contributor

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 fixes incorrect CDN usage documentation for vue-multiselect 3.x. The documentation incorrectly stated that the library is exposed as window.VueMultiselect when loaded via CDN, but the UMD build (which is served by unpkg) actually exposes it as window['vue-multiselect']. This caused confusion for users trying to use the library via CDN, as reported in issue #1756.

Changes:

  • Updated CDN usage example to use window['vue-multiselect'].default instead of window.VueMultiselect.default

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

@mattelen mattelen merged commit 2c7335b into shentao:master Feb 24, 2026
4 checks passed
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.

5 participants