Browse Source

fixed safari autofill bug (#1969)

* fixed safari autofill bug

* removed test html mockup
Dede Muzaffer 3 years ago
parent
commit
b41b35832a
1 changed files with 3 additions and 1 deletions
  1. 3 1
      packages/alpinejs/src/directives/x-model.js

+ 3 - 1
packages/alpinejs/src/directives/x-model.js

@@ -60,8 +60,10 @@ function generateAssignmentFunction(el, modifiers, expression) {
     return (event, currentValue) => {
         return mutateDom(() => {
             // Check for event.detail due to an issue where IE11 handles other events as a CustomEvent.
+            // Safari autofill triggers event as CustomEvent and assigns value to target 
+            // so we return event.target.value instead of event.detail
             if (event instanceof CustomEvent && event.detail !== undefined) {
-                return event.detail
+                return event.detail || event.target.value
             } else if (el.type === 'checkbox') {
                 // If the data we are binding to is an array, toggle its value inside the array.
                 if (Array.isArray(currentValue)) {