wireui / wireui

TallStack UI components
https://v1.wireui.dev
MIT License
1.37k stars 166 forks source link

WireUI don't work with Chrome < v.80 on Android #51

Closed trippo closed 2 years ago

trippo commented 2 years ago

Wireui don't work with chrome browser on android.

If you try to load the page you have an error: VM28:71 Uncaught SyntaxError: Unexpected token '.' also on documentation page https://livewire-wireui.com/docs/get-started

PH7-Jack commented 2 years ago

What's the android version? What's the chrome version?

trippo commented 2 years ago

Mi test phone has Android 6 Chrome 79

trippo commented 2 years ago

The problem is Optional chaining '?.' on alpine js code on blade components

trippo commented 2 years ago

Example on notification message image

trippo commented 2 years ago

I found the problem https://developer.mozilla.org/en-US/docs/Web/JavaScript/Reference/Operators/Optional_chaining image

trippo commented 2 years ago

What do you think about?

PH7-Jack commented 2 years ago

I'm taking a look, as I see you are correct. Previous versions are not supported, even other browsers from what I've found out. I'm thinking of a way to solve this in all unsupported browsers. Maybe a function like"data_get()" from laravel, but for js

PH7-Jack commented 2 years ago

https://caniuse.com/mdn-javascript_operators_optional_chaining

trippo commented 2 years ago

I think it's a great solution

PH7-Jack commented 2 years ago

Update: Added support for chrome 71+. I'm testing with firefox, safari, and the like I'm refactoring many things, soon I have great news

PH7-Jack commented 2 years ago

Are working in

PH7-Jack commented 2 years ago

In soon I will release a new version with this fix

PH7-Jack commented 2 years ago

Websites to simulate browser

PH7-Jack commented 2 years ago

The new version is now released. Can you try using the latest version? If the problem persists, reopen the issue