akira-kurogane / furigana-injector

Automatically exported from code.google.com/p/furigana-injector
3 stars 1 forks source link

"attachPopupTriggerToAllRT is not defined" / No translation pop-ups #59

Closed GoogleCodeExporter closed 9 years ago

GoogleCodeExporter commented 9 years ago
What steps will reproduce the problem?
1. Insert furigana as normal.
2. Try to view translations by mousing over the furigana-inserted words.

What is the expected output? What do you see instead?

Sometimes no pop-ups will appear. The javascript error log of window will show 
"Uncaught ReferenceError: attachPopupTriggerToAllRT is not defined" in this 
case.

This seems to be an issue where the background script is executing code to 
attach handlers all <rt> elements using the attachPopupTriggerToAllRT() 
function, before the script that contains it has become available. The script 
is inserted first by order from the background but I guess it's done 
asynchronously.

Original issue reported on code.google.com by akira%ya...@gtempaccount.com on 7 Jan 2011 at 6:19

GoogleCodeExporter commented 9 years ago

Original comment by akira%ya...@gtempaccount.com on 7 Jan 2011 at 6:20

GoogleCodeExporter commented 9 years ago
Note to self: Using a callback on executeScript seems like a good idea, 
combined with only running the standalone "attachPopupTriggerToAllRT()" as an 
ELSE action to inserting the script file.

Original comment by akira%ya...@gtempaccount.com on 7 Jan 2011 at 8:28

GoogleCodeExporter commented 9 years ago

Original comment by akira%ya...@gtempaccount.com on 7 Jan 2011 at 12:57

GoogleCodeExporter commented 9 years ago
Version 1.1.4 released to fix this.

Original comment by akira%ya...@gtempaccount.com on 7 Jan 2011 at 1:07