-
Notifications
You must be signed in to change notification settings - Fork 31
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
Apparent double nesting of rubies impacting ruby/bouten spacing and baseline shifts (when rubyReserve is used) #184
Comments
To polyfill Line 782 in 060e583
|
Thank you, @palemieux! Would you happen to have a target date for the 1.1.2 release? Much Appreciated, |
@jbaden It would be good to catch all blocking issues in that release. Anything else from you? |
@jbaden See #187. I have deployed it at http://sandflow.com/imsc1_1/index.html (be sure to hit Shift-F5). Looking forward to your feedback. |
Thank you, @palemieux. It looks like this issue has now been resolved. That said, I did want to call your attention to what I believe to be a browser-related issue. There appears to be relative movement of the baseline text in the 1-line event "with" and "without" rubies. See events timed at (88 & 89). I noticed that this very slight shift only occurs in Firefox, but not in Chrome, which makes me think this is more of a browser issue and not an issue with imscJS. Thanks! |
@jbaden Thanks for the review. The issue at #184 (comment) was in fact caused by style properties not being copied correctly to the hidden |
Hi @palemieux, After further testing and review, I believe the rubyReserve issue you identified may also impact boutens (tts:textEmphasis). "To polyfill tts:rubyReserve, applyRubyReserve() inserts a zero-width ruby annotation before the first HTML span in each line. This breaks if the first span is itself within a ruby annotation. Probably can be fixed." In the following XML excerpts & corresponding images generated from IMSC11 Renderer, I've noticed that the baseline shifts when boutens are applied if they are exercised at the very beginning of a line (Example 2). In Example 3, I've added punctuation before the tts:textEmphasis span and the baseline is preserved (as expected with rubyReserve): Example 1 2-line Horizontal Event without Boutens; with rubyReserve: Example 2 2-line Horizontal Event with Boutens + rubyReserve (no preceding character/punctuation before span tts:textEmphasis) Example 3 2-line Horizontal Event with Boutens + rubyReserve (with a preceding character/punctuation before span tts:textEmphasis)
Test file: |
@jbaden Yes, it looks like the issue is with the double application of boutens. See simpler example below. Can you open a different issue?
EDIT: Clarified it is unrelated to |
Hello,
There appears to be a bug in IMSC.js resulting in double/nested rubies when rubyReserve is used. This issue is illustrated in the attached test file rubyReserve005_NFLX_Sample.xml.zip
In order to render correctly, ruby events appear to require non-space characters at the beginning of a line both before and outside of the ruby container. That is to say, a ruby event cannot begin with tts:ruby="container" else this double/nested ruby rendering issue will occur.
without non-space character
with non-space character
The text was updated successfully, but these errors were encountered: