-
Notifications
You must be signed in to change notification settings - Fork 200
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
Annotate/Highlight dialogue box detached from selected text #619
Comments
@dhimmel, thanks for the heads-up. When I set |
Thanks @dsiddy. I'm interested in the potential fix you've discovered. Our CSS currently includes. /***Page margins when printing***/
@page {
margin-top: 19mm;
margin-bottom: 16mm;
margin-left: 0mm;
margin-right: 0mm;
} However, the P.S. feel free to submit a PR to Manubot, if you feel so inclined. |
Sorry for the belated reply, @dhimmel. I just accepted your invitation and submitted a pull request (manubot/rootstock#95) that should resolve the issue. |
By positioning the `body` element, we tell Hypothesis to use it (rather than the document) as the container element for the adder. (See hypothesis/client#487 for more information.) Closes: #90, hypothesis/client#619
Closed in manubot/rootstock@d209731. Thanks @dsiddy for the fix! |
By positioning the `body` element, we tell Hypothesis to use it (rather than the document) as the container element for the adder. (See hypothesis/client#487 for more information.) Closes: #90, hypothesis/client#619
By positioning the `body` element, we tell Hypothesis to use it (rather than the document) as the container element for the adder. (See hypothesis/client#487 for more information.) Closes: #90, hypothesis/client#619
We're interested in using the hypothesis javascript client for our Manubot HTML pages (manubot/rootstock#90). We want to apply Hypothesis to pages like https://greenelab.github.io/manubot-rootstock/.
However, when we add Hypothesis like https://via.hypothes.is/https://greenelab.github.io/manubot-rootstock/, there's an issue with the dialogue box placement:
The annotation options appear far away from the actual point of highlighting. Not sure whether this is an issue with our CSS or a Hypothesis client issue. Hat tip to @RaoOfPhysics for bringing this issue to our attention.
The text was updated successfully, but these errors were encountered: