nette.ajax.js icon indicating copy to clipboard operation
nette.ajax.js copied to clipboard

Safari browser detection

Open aleswita opened this issue 6 years ago • 4 comments

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 avatar Jul 12 '18 07:07 aleswita

@vojtech-dobes ping

aleswita avatar Jul 18 '18 03:07 aleswita

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.

spernica avatar Sep 03 '18 15:09 spernica

@spernica some problem

Bezdekm avatar Nov 07 '18 11:11 Bezdekm

+1

lukasojd avatar Oct 31 '19 11:10 lukasojd