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

Cannot view report for rtl languages #54

Closed
mattgarrish opened this issue Oct 5, 2017 · 1 comment
Closed

Cannot view report for rtl languages #54

mattgarrish opened this issue Oct 5, 2017 · 1 comment
Assignees
Labels
Milestone

Comments

@mattgarrish
Copy link

Running some more tests on the samples, I noticed that ace runs without error on RTL files, but when you open the output report, there is nothing in it. (Or I haven't seen this yet in LTR files, anyway.)

The following error is recorded in the console:

Uncaught TypeError: Cannot read property 'cfi' of undefined
    at ace-report-viewer.js:76
    at Array.forEach (<anonymous>)
    at ace-report-viewer.js:66
    at Array.forEach (<anonymous>)
    at AceReport.flattenData (ace-report-viewer.js:63)
    at new AceReport (ace-report-viewer.js:12)
    at HTMLDocument.<anonymous> (report.html:210)
    at j (jquery-3.2.1.min.js:2)
    at k (jquery-3.2.1.min.js:2)

For testing, try:
https://github.com/IDPF/epub3-samples/releases/download/20170606/kusamakura-japanese-vertical-writing.epub
https://github.com/IDPF/epub3-samples/releases/download/20170606/israelsailing.epub

@mattgarrish
Copy link
Author

Just to confirm what the message says, when I look at the assertions there aren't any earl:pointer objects.

@rdeltour rdeltour self-assigned this Oct 5, 2017
@rdeltour rdeltour added the bug label Oct 5, 2017
@rdeltour rdeltour added this to the Beta 1 milestone Oct 5, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

2 participants