Skip to content

Add permalink to post date#4

Open
popkin wants to merge 1 commit intogsabater:masterfrom
popkin:master
Open

Add permalink to post date#4
popkin wants to merge 1 commit intogsabater:masterfrom
popkin:master

Conversation

@popkin
Copy link
Copy Markdown

@popkin popkin commented Feb 6, 2016

Soy Djtonight de MV. No estoy muy puesto en github, pero he intentado hacer algo a ver si te viene bien.
Simplemente he hecho la función:

  • Un enlace permanente al post en la fecha
    Que aparece en las sugerencias de la comunidad.

Si lo ves bien puedo ir ayudando poco a poco!

Un saludo

@BondaCB
Copy link
Copy Markdown
Collaborator

BondaCB commented Feb 6, 2016

Hola Toni,
en primer lugar, gracias por el curro, y me encantaría que me ayudaras a hacer funciones.

He estado revisando lo que me has mandado, y está bien como primera aproximación, pero no funcionaría completamente.

Te comento, en primer lugar no basta con poner la id del post en el enlace. Eso puede llevar varios problemas, y lo suyo sería poner el enlace completo del hilo / pagina # id del post.

En segundo lugar, lo que has añadido solamente funcionaría en la primera página del post. Al cargar nuevos posts por ajax (infinite scroll), estos no tienen el enlace.

Te recomiendo que hagas una nueva función, que haga un for(i in _posts){ } y ahí busques la fecha y pongas el enlace. De esta forma la función podrás llamarla después de poner un ajax, como por ejemplo las funciones de reversequote.

la variable _posts tiene en memoria todos los posts del hilo, para que no se tenga que buscar el DOM otra vez.

Saludos!

@popkin
Copy link
Copy Markdown
Author

popkin commented Feb 6, 2016

Genial.
He ido un poco a lo loco sin mirar al 100% como organizas el código, así que la explicación me viene de perlas.
Haré los cambios!

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

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants