mxabierto / debate

(sin mantenimiento) Debate público sobre la Política Nacional de Datos Abiertos en México. Powered by DemocracyOS.
http://politica.datos.gob.mx
1 stars 0 forks source link

No renderean las secciones en Safari de iPhone #65

Closed defvol closed 10 years ago

defvol commented 10 years ago

@urkonn lo revisará en saucelabs

cristiandouce commented 10 years ago

En la última versión de DemocracyOS/app actualizamos secciones de la aplicación que no se construían en Firefox.

Podrían probar si la versión actual de DemocracyOS/app funciona en Safari de iPhone?

defvol commented 10 years ago

Falla en Safari de iOS 7.0

Encontramos esto:

QuotaExceededError: DOM Exception 22: An attempt was made to add something to storage that exceeded the quota.

defvol commented 10 years ago

Saucelabs utitliza el simulador, y en ese caso siempre funciona bien (iOS 6 y 7).

Donde falla es con devices reales de iOS 7.0 (en safari)

defvol commented 10 years ago

Es pedo de safari.

No de los sitios, p.ej. Khan Academy tiene el mismo issue: https://code.google.com/p/khanacademy/issues/detail?id=7936

defvol commented 10 years ago

"It may be necessary to purge or expire local storage items to stay under the limits. I assume I can clear out my local storage cache and get it running again, but reporting this in case it may be a problem for others."

defvol commented 10 years ago

2014-03-12 05 13 26 pm

defvol commented 10 years ago

app.js:24093

require.register("store/local.js", function(exports, require, module) {
    ...
    LocalStore.prototype.set = function(key, value) {
        return storage.setItem(key, serialize(value)), this;
    }
defvol commented 10 years ago

hey @cristiandouce ¿para qué están usando html localStorage?

defvol commented 10 years ago

¿es necesario ese componente? ¿hacemos un localStorage.clear();? parecen datos algo triviales:

localStorage
Storage {_at.hist.0211: "{"m":600,"k":2,"l":{"0":23,"1":109},"b":{"0":83886…12":0,"13":0,"14":0,"15":0,"16":0,"17":0,"18":0}}", _at.hist.0212: "{"m":600,"k":2,"l":{"0":23,"1":109},"b":{"0":83886…12":0,"13":0,"14":0,"15":0,"16":0,"17":0,"18":0}}", atpro-config: "false", debug: "undefined", laws-filter: "{"sort":"by-law"}"…}
_at.hist.0211: "{"m":600,"k":2,"l":{"0":23,"1":109},"b":{"0":8388608,"1":0,"2":0,"3":8192,"4":0,"5":0,"6":0,"7":0,"8":0,"9":0,"10":0,"11":0,"12":0,"13":0,"14":0,"15":0,"16":0,"17":0,"18":0}}"
_at.hist.0212: "{"m":600,"k":2,"l":{"0":23,"1":109},"b":{"0":8388608,"1":0,"2":0,"3":8192,"4":0,"5":0,"6":0,"7":0,"8":0,"9":0,"10":0,"11":0,"12":0,"13":0,"14":0,"15":0,"16":0,"17":0,"18":0}}"
atpro-config: "false"
debug: "undefined"
laws-filter: "{"sort":"by-law"}"
length: 6
notfirsttime: "1"
__proto__: Storage
defvol commented 10 years ago

al parecer el localStorage es por sesión del browser no por página. Debuggeando Safari iPhone nos dimos cuenta que la sesión de DOS no tiene localStorage.

Si abrimos una sesión de "private browsing" donde el localStorage = null, la página sí se ve.

defvol commented 10 years ago

tenemos un fix que ya está funcionando en http://apertura-staging.herokuapp.com/