Jump to content

User talk:Writ Keeper/Scripts/teahouseUtility.js

Page contents not supported in other languages.
From Wikipedia, the free encyclopedia

Issues around tipsy generation[edit]

Hey Writ Keeper (also cc DannyS712) I noticed some issues with this around the tipsy messages. Most importantly, as far as I can tell, id is never defined, so the selector would just fail with a message in the console. Second, tipsy needs to run .tipsy({params object}) before it can auto open; I think you can chain them, so .attr("title", 'There was...').tipsy().tipsy('show') should work?

Finally, jquery.tipsy isn't part of the core mw modules. Twinkle uses tipsy, so anyone with Twinkle installed would have them, but I'd like to change that, so if these ever were made functional, it should be directly imported. ~ Amory (utc) 01:53, 1 July 2020 (UTC)[reply]

@Amorymeltzer why am I cc'ed? DannyS712 (talk) 01:55, 1 July 2020 (UTC)[reply]
Was just writing this on GitHub, but: you have a related fork and know things! Apologies for the ping. ~ Amory (utc) 02:01, 1 July 2020 (UTC)[reply]
@Amorymeltzer: Hmm, no idea why I have that fork, but I don't think I've ever used it - tagged for U1 DannyS712 (talk) 04:13, 1 July 2020 (UTC)[reply]
Not important, but I think it was to fix the editToken -> csrfToken in because User:Abelmoschus_Esculentus/Article.js. ~ Amory (utc) 09:21, 1 July 2020 (UTC)[reply]