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

Incorrect description of tts:position example. #712

Closed
palemieux opened this issue Apr 5, 2018 · 4 comments
Closed

Incorrect description of tts:position example. #712

palemieux opened this issue Apr 5, 2018 · 4 comments

Comments

@palemieux
Copy link
Contributor

palemieux commented Apr 5, 2018

The bottom of the region seems to be positioned 10% * (root container region height) from the bottom of the root container region.

This is inconsistent with the clause percentage value component for a vertical position offset is relative to the height of the positioning rectangle minus the height of the associated region.

@palemieux
Copy link
Contributor Author

See test and renders at w3c/imsc-tests#49

@skynavga skynavga added this to the CR2 milestone May 10, 2018
@skynavga
Copy link
Collaborator

skynavga commented May 13, 2018

The CSS spec I derived from (https://www.w3.org/TR/css-backgrounds-3/#the-background-position) is rather vague. I will check further with CSS implementations.

@skynavga
Copy link
Collaborator

OK, I verified that both offset and edge position calculations are the same. See https://codepen.io/skynavga/pen/BxVrRj. I will update the description of the example. I think we don't need to actually change the image since one can't really (visually) discern what offset appears in the image, and the image is intended to only be representative.

@skynavga skynavga changed the title Incorrect "Example Rendition – Position" Incorrect description of tts:position example. May 13, 2018
@palemieux
Copy link
Contributor Author

0

Here's a render in case it helps.

Also, the region style should have something like <style tts:fontSize="40%"/> otherwise the text is too big.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants