pombreda / cdpedia

Automatically exported from code.google.com/p/cdpedia
0 stars 0 forks source link

falta el archivo IEFixes.js ? #145

Closed GoogleCodeExporter closed 9 years ago

GoogleCodeExporter commented 9 years ago
Usando IE8 como navegador parece que falta un archivo de compatibilidad con IE, 
aunque en IE8 la pagina se vé bien.

Reproducir:
1. lanzar cdpedia.py
2. usar como navegador IE8, apuntandolo a cualquier pagina, por ejemplo:
http://localhost:8000/wiki/Turismo_arqueológico

Mirando el log / consola se ve
127.0.0.1 - - [25/Sep/2012 20:27:01] "GET /wiki/Turismo_Arqueol%C3%B3gico HTTP/1
.1" 200 -
127.0.0.1 - - [25/Sep/2012 20:27:01] "GET /static/skins/common/IEFixes.js HTTP/1
.1" 404 -

Si bien existe el dir /static/skins/ , no existe el dir /static/skins/common , 
tampoco veo al file suelto por ahi.

Original issue reported on code.google.com by ccanepacc@gmail.com on 25 Sep 2012 at 11:40

GoogleCodeExporter commented 9 years ago
No tengo idea de cómo o cuándo se recolectaron los recursos en static.

Una posibilidad de obtenerlo es del repo git de wikipedia [0]
El problema es saber qué version usar.

Aparentemente ese repo marca las releases como branches, REL_1 .. REL_20
Pero esos branches despues reciben backports.

Comparando el wikibits.js de cdpedia, el release mas cercano corresponde a 
REL_16 , tiene tres o cuatro lineas diferentes nada mas. las REL anterior y 
siguiente tienen muchas mas diferencias.

Ademas puede ser que haya que ajustar algun path: en el repo IEFixes.js está 
en el mismo dir que wikibits.js (core/skins/common) mientras que en cdpedia 
wikibits vive en static/skins/js

Hay dos lugares donde se menciona a IEFixes:
  D:\tmp\cd_mini_3\cdpedia\assets\static\skins\js\wikibits.js (1 hits)
    Line 1048:  importScriptURI( stylepath + '/common/IEFixes.js' );
  D:\tmp\cd_mini_3\cdpedia\src\web\templates\cdpedia_base.html (1 hits)
    Line 20:     <!--[if IE]><script type="text/javascript" src="/static/skins/js/IEFixes.js"></script>

[0] http://www.mediawiki.org/wiki/Git 

Original comment by ccanepacc@gmail.com on 10 Oct 2012 at 4:49

GoogleCodeExporter commented 9 years ago
IEFixes.js  parece lidiar principalmente con compatibilidad para IE5, IE6, 
quizas IE7.

Versiones mas modernas del repo agregan un file .js adicional para usar con IE8.

La version mas cercana para wikibits.js es la de 2010 09 29 , 
b9ca5cda0b6c75b0c7802c7dfe2934131e00dfdc.

Tomando el IEFixes de ese momento y corrigiendo los paths para que sea 
encontrado 
se observa que el render de la pagina empeora en IE8 (lo mas notorio es que el 
logo se escapa de la sidebar e invade el area del contenido).

IE8 es el ultimo navegador provisto por MS para windows xp; no es el navegador 
original que viene con Windows XP pero por cuestiones de seguridad es probable 
que la mayoria de los xp estén usando IE8.

El render de cdpedia que se obtiene sin incluir IEFixes.js es bueno.

Recomendaria entonces no incluirlo en el proximo release de cdpedia.

Si les parece, cierren el issue como wontfix.

Original comment by ccanepacc@gmail.com on 26 Oct 2012 at 1:31

GoogleCodeExporter commented 9 years ago
Genial, Claudio, gracias por el análisis!

Original comment by facundob...@gmail.com on 26 Oct 2012 at 12:32