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

Remove #textOrientation-sideways-LR. #980

Closed
skynavga opened this issue Aug 26, 2018 · 2 comments
Closed

Remove #textOrientation-sideways-LR. #980

skynavga opened this issue Aug 26, 2018 · 2 comments

Comments

@skynavga
Copy link
Collaborator

The semantics of sidewaysLeft, specifically, a 90 degree counter-clockwise rotation, is not well defined in the context of vertical, top-to-bottom inline progression line layout. For example, rotating latin text 90 degrees counter-clockwise would effectively make it a bottom-to-top inline progression layout, which is not supported by any (currently) defined writing mode in TTML. Reviewing both UTR50 and CSS Writing Modes Level 3 reveals that in all cases, only 90 degree clockwise rotation is sanctioned.

Since we have marked #textOrientation-sideways-LR as at-risk, I suggest we remove this feature, including the semantics of sidewaysLeft which appears to sanction a less than well-defined CCW rotation.

Although this is a substantive change, it is permitted prior to PR without triggering a new CR since it has been marked at risk.

@skynavga skynavga added this to the PR milestone Aug 26, 2018
@skynavga skynavga self-assigned this Aug 26, 2018
@skynavga skynavga changed the title Remove #textOrientation-sideways-LR Remove #textOrientation-sideways-LR. Aug 26, 2018
@nigelmegitt
Copy link
Contributor

only 90 degree clockwise rotation is sanctioned.

@skynavga I'm trying to understand why sideways is no longer writing mode dependent - are you saying here that UTR50 and CSS writing mode level 3 do not set a writing mode dependency on the rotation angle?

@skynavga
Copy link
Collaborator Author

@nigelmegitt correct; as I said yesterday, neither UTR50 nor CSS WM3 define any use of CCW rotation, so it all uses of sideways map to 90CW

skynavga added a commit that referenced this issue Sep 9, 2018
…ideways-lr

Remove #textOrientation-sideways-LR and related semantics (#980).
@skynavga skynavga removed the pr open label Sep 9, 2018
@skynavga skynavga removed their assignment Nov 5, 2018
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