Conversation
|
@roymckenzie, we could try to set another option as described here #2 🤔 |
|
I saw your comments on the pull request you reviewed (thank you!), but am still not clear on why this error is being thrown. Is the DOM not fully loaded? Is setting an arbitrary delay going to cover edge cases? |
|
The issue is we manipulate the DOM and when we use This possible solution is to set an arbitrary delay and with that wait for a time when all the js manipulation are done. But I think perhaps we should change the |
|
Ah! I see. That makes sense. I dug around a bit. I tried to see if there was any way to set style rules based on the type of character. No dice. I am open to seeing a different solution other than innerHTML to inject a span. Barring a new solution to injecting the span, I think a time delay is an acceptable solution. I do not think it is necessary to allow the extension user to set the time. I think hardcoding a time delay is sufficient (wrapping the This will likely cover most edge cases until a cleaner solution can be found. |
PoC
Before

After
