Robert Knight

Thanks for the PR. The API that C++ exposes to JS is, generally speaking, a higher level API that does not expose Tesseract internals, except for the API to specifically...

I'm open to offering some segmentation controls but I don't intend to offer every single option that Tesseract supports internally as the main API. One reason for that is that...

The `setVariable` method does not yet exist for the `OCRClient` class which you are using, only the lower-level `OCREngine` class which exists in the web worker. A totally-unsupported, will-break-in-future way...

Couple of notes from a discussion today: - We discussed reducing the minimum length limit from 3 to 1 - Apparently the longest course name in the lms DB is...

With the changes in this PR, users will get a confusing error if the API cookie expires or gets removed but the main site cookie still exists. I did this...

@seanh - Do you still plan to go ahead and land this as-is following our discussion at the end of last week or do you plan to make any changes?

Thanks for the update. For the benefit of anyone else who runs into this, a dumb workaround that we settled on is to configure our minifier to limit the length...

I have not tried running this project outside of the browser or Node.js. You're welcome to have a go at figuring this out. Please let me know how you get...

Using flexbox like this aligns the center of the icon with the center of the inline box in which the font is drawn. That won't give consistent results across fonts...

> BTW, I noticed you added the underline to the text via . Although it probably makes sense from an accessibility point of view, should we defer that and do...