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:
us...@infra.apache.org


With regards,
Apache Git Services

Reply via email to