Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Fallo css en algunas páginas de discourse #257

Open
nukeador opened this issue Feb 18, 2015 · 3 comments
Open

Fallo css en algunas páginas de discourse #257

nukeador opened this issue Feb 18, 2015 · 3 comments

Comments

@nukeador
Copy link
Member

Como por ejemplo:

https://foro.mozilla-hispano.org/404

También afecta a la página que aparece al registrarse, por lo que es importante.

@ghost
Copy link

ghost commented Oct 30, 2015

Por ver si se puede avanzar algo con este tema, he estado mirando la página 404 y veo algunas diferencias con páginas que sí funcionan. Detallo lo que he visto, aunque creo que solo los .js y clases deberían estar afectando.

  • Detrás de :
    En la página que no funciona:
    Y en la que sí funciona se añaden clases a html:

  • Faltan referencias a scripts justo delante de:
    Concretamente falta:

    <script src="/assets/preload_store-d16a3675434b5a0043157cfc2b850471.js"></script> <script src="/assets/locales/es-60b0211457b727ad7400d9283502094e.js"></script> <script src="/assets/vendor-d726f166eae32585d948cc4b83590d33.js"></script> <script src="/assets/application-7a255dcf0883f0db1a0a2ac873b49d26.js"></script>
  • Justo delante de falta:

@nukeador, si pudieras aplicarlo en alguna página (p.ej. la de 404) podría seguir mirándolo.

@nukeador
Copy link
Member Author

@jlgetxo tu mensaje ha quedado cortado. En todo caso la página de 4040 la genera discourse y no se si es algo que podamos modificar sin tocar el core. Ping @stripTM

@ghost
Copy link

ghost commented Oct 31, 2015

Ya veo, mi mensaje quedó ilegible, tendría que haber mirado la vista previa :(
En el caso de la página 404 parece ser una decisión de diseño de Discourse para que tenga el menor peso posible:
https://meta.discourse.org/t/keyboard-shortcuts-do-not-work-on-404-page/18534/6

codinghorror Jeff Atwood co-founder
I guess I view the 404 page as a special page that doesn't load a lot of the other code we need to render topics, replies, etc. And going back is trivial. Try it yourself with the above link and standard browser keyboard shortcuts that work on every page on the Internet.

Adding a bunch of dependencies to the 404 page can cause extra load, as lots of web spiders hit the 404 pretty hard in many cases for a bunch of different reasons.

He probado a registrar un nuevo usuario jlgetxotest (se puede borrar @nukeador) y el problema es exactamente el mismo. Faltan referencias a clases y archivos de script, con lo que la página no tiene todas las funcionalidades js/css. He preguntado sobre esto en el foro de Discourse, avisaré cuando sepa algo más.

Edito: a ver si se puede hacer esta prueba rápida para la página que aparece tras registrarse. (no tengo dónde probarlo)
archivo: discourse-master\app\controllers\users_controller.rb
línea: 475 (dentro de def account_created)
Cambiar: render layout: 'no_ember'
por: render layout: 'application'

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants