You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@annotator.apache.org by GitBox <gi...@apache.org> on 2018/01/17 12:12:07 UTC
[GitHub] Treora commented on issue #25: error: context not equal to range's container
Treora commented on issue #25: error: context not equal to range's container
URL: https://github.com/apache/incubator-annotator/issues/25#issuecomment-358286276
As you may be aware, annotator's code is still very immature, and the most relevant part of the error message is the 'not implemented'. So in any case the solution should be to develop this function further, not to change the way you invoke it. I see I put a bit more info in the [source code](https://github.com/apache/incubator-annotator/blob/f7e7daab30949dccbf990105c08bfe887b1b6db9/packages/dom/src/text/quote.mjs#L38-L40):
```
// FIXME We should get range index relative to context. Look at
// dom-anchor-text-position? For now, we implement the easy case where the
// ranges are within the same container.
```
Would be good to sit down some time and develop this to a more usable state.
----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
With regards,
Apache Git Services