[Bug] Gestione apertura link esterni al portale (commenti)

[hashcat]

I link a contenuti esterni presenti nei commenti degli utenti in calce agli articoli (lato portale) non rispettano la convenzione di essere aperti, al click, in una nuova scheda (come sul forum).

#128 (Bug)

[Zane]

Sono diventato matto dietro a questo micro-bug, ma mi sono impuntato che volevo sistemarlo... dopo ORE di maledizioni varie, CE L'HO FATTA!!!!!

È online già adesso Vedi, ad esempio, qui https://turbolab.it/commenti/239

[hashcat]

Ottimo!

P.S.: A cosa era attribuibile il malfunzionamento?

[Zane]

Il fatto che l'effetto "link esterni: nuova tab + iconcina rossa" non funzionasse nei commenti era dovuto al fatto che il js che gestisce l'effetto lavora solo sul corpo degli articoli ($('#corpo')) e da nessuna altra parte, quindi non lavora sui commenti (qui sul forum, la cosa è gestita da un file diverso che lavora, appunto, solo sui link contenuti nel corpo dei post ($('.postlink')).

La micro-difficoltà di far funzionare il primo js anche sui link dei commenti è dovuto al fatto che i commenti vengono caricati in modo asincrono qualche secondo dopo l'articolo stesso, quindi il js non li "vede" mentre fa la sua magia.

Poco male, basta metterlo nel js che carica i commenti stessi, mi sono detto... e qui ho perso le ore di cui parlavo: avevo dimenticato che c'è un effetto di transizione (fadeIn()) che in Jquery non è bloccante, quindi mettere il codice "link esterni: nuova tab + iconcina rossa" semplicemente sotto non funzionava perché scattava prima che il fadeIn() avesse finito, quindi mancava nuovamente il bersaglio... è bastato passare tutto come secondo parametro del fadeIn (la cui descrizione è "A function to call once the animation is complete.", quindi mi sembra proprio a prova di stupido!!!!!!!) per gestire il tutto come voluto

[Zane]

Rapida nota: ho rimosso il file Javascript che gestisce l'apertura dei link esterni in una nuova scheda. la funzionalità è ora gestita tramite un'estensione di phpBB prima di inviare la pagina al browser