Consider inserting modal at end of body

[yomed]

Hi, thanks for the great project!

I noticed that currently, the modal gets inserted into the DOM at the top of the body. For some use cases, this could be undesirable. For example, when Google picks up content for page descriptions, it often goes to the first text on the page. I imagine that in most cases, content in the modal text would not be considered important enough for that to be desired.

The only change needed would be on this line:

tingle/dist/tingle.js

Line 53 in 4e4237a

document.body.insertBefore(this.modal, document.body.firstChild)

I noticed that this change would also help solve another issue #109 (cc @emmanuelballery)

[robinparisi]

Hey @yomed,

Done here: caaa6ff

To be honest, since Google crawlers are using a Chrome headless, they can detect hidden content and are able to extract main content. I'm not sure tingle content can be found in meta description but if you have an example, I'm interested!

[yomed]

@robinparisi Thanks for the fix! In seems like in some cases Google ignores the meta description, and instead goes looking for text from the main content. So I was running into that with my project, where it picked up the modal text.

[robinparisi]

Ok, that's interesting, thanks for the feedback! 😃