Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

move annotatorjs to the bottom of the body so that the "article" tag is loaded before #1721

Merged
merged 1 commit into from
Feb 24, 2016

Conversation

tcitworld
Copy link
Member

Q A
Bug fix? yes
New feature? no
BC breaks? yes
Deprecations? yes
Tests pass? yes
Documentation no
Translation no
Fixed tickets #1718
License MIT

@j0k3r
Copy link
Member

j0k3r commented Feb 24, 2016

Does it fix #1718 or just related to it ?

@tcitworld
Copy link
Member Author

It fixes it (annotator now applies to the article tag which is the start of our content).

j0k3r added a commit that referenced this pull request Feb 24, 2016
…themes

move annotatorjs to the bottom of the body so that the "article" tag is loaded before
@j0k3r j0k3r merged commit 0688a83 into v2-annotator-comments Feb 24, 2016
@j0k3r j0k3r deleted the v2-fix-comments-for-different-themes branch February 24, 2016 13:51
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants