vojtech-dobes / nette.ajax.js

Flexible AJAX for Nette Framework. Supports snippets, redirects etc.
https://componette.org/vojtech-dobes/nette.ajax.js/
MIT License
149 stars 85 forks source link

Safari browser detection #158

Open aleswita opened 6 years ago

aleswita commented 6 years ago

Correct regex for Safari detection

navigator.userAgent return Mozilla/5.0 (Macintosh; Intel Max OS X 10_12_6) AppleWebKit/605.1.15 (KHTML, like Gecko) Version/11.1.2 Safari/605.1.15

aleswita commented 6 years ago

@vojtech-dobes ping

spernica commented 6 years ago

Prosím, nemůžete použít let ... of, protože není pořádně podporovaný - https://www.w3schools.com/js/js_let.asp

Úplně mi váš skript odstřelil javascript v IE <=11

Konkrétně řádek (hlásí Byl očekáván znak ';') for (var pair of formData.entries()) {

Správně je podle mě:

        if (formData.entries && navigator.userAgent.match(/version\/11(\.[0-9]+)* safari/i)) {
            var entries = formData.entries();
            for (var i = 0; i < entries.length; i++) {
                var pair = entries[i];
                if (pair[1] instanceof File && pair[1].name === '' && pair[1].size === 0) {
                    formData.delete(pair[0]);
                }
            }
        }

Ale nemám otestované safari. Pouze IE a chrome.

Bezdekm commented 5 years ago

@spernica some problem

lukasojd commented 4 years ago

+1