Skip to content

Commit 18187d2

Browse files
authored
Fix alpine v3 crash (#262)
* optional chaining on window.Alpine.version * use optional chaining to read alpine version
1 parent 88d44ad commit 18187d2

File tree

2 files changed

+10
-3
lines changed

2 files changed

+10
-3
lines changed

packages/shell-chrome/src/backend.js

Lines changed: 8 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -45,9 +45,14 @@ function init() {
4545

4646
this._stopMutationObserver = false
4747
this._lastComponentCrawl = Date.now()
48+
}
49+
50+
get alpineVersion() {
51+
return window?.Alpine?.version || ''
52+
}
4853

49-
this.alpineVersion = window.Alpine.version
50-
this.isV3 = isRequiredVersion('3.0.0', this.alpineVersion)
54+
get isV3() {
55+
return isRequiredVersion('3.0.0', this.alpineVersion)
5156
}
5257

5358
runWithMutationPaused(cb) {
@@ -105,6 +110,7 @@ function init() {
105110
return
106111
}
107112
if (isRequiredVersion('2.8.1', this.alpineVersion)) {
113+
console.log('aaa')
108114
window.addEventListener('error', (errorEvent) => {
109115
if (errorEvent.error && errorEvent.error.el && errorEvent.error.expression) {
110116
const { el, expression } = errorEvent.error

packages/simulator/navigation-target.html

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -6,7 +6,8 @@
66
<!-- <script src="https://cdn.jsdelivr.net/gh/alpinejs/[email protected]/dist/alpine.min.js" defer></script> -->
77
<!-- Rollup server is configured to serve contents of node_modules/alpinejs/dist, see rollup.config.js -->
88
<!-- Uncomment the following script to use that version, eg. when working offline -->
9-
<script src="/alpine.js" defer></script>
9+
<!-- <script src="/alpine.js" defer></script> -->
10+
<script defer src="https://unpkg.com/[email protected]/dist/cdn.min.js"></script>
1011
</head>
1112

1213
<body>

0 commit comments

Comments
 (0)