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

Add tts:background{Clip,Extent,Origin}. #177

Merged
merged 3 commits into from
Oct 1, 2016

Conversation

skynavga
Copy link
Collaborator

@skynavga skynavga commented Sep 10, 2016

Add tts:background{Clip,Extent,Origin}.
Also add [background {painting,positioning} area] defines (as TBDs).

See issue #175.

…ectangles (formerly areas).

Introduce citations to these definitions.
Change other uses of "positioning area" to "positioning rectangle" as appropriate.
Update schema archives.
</tr>
<tr>
<td><emph>Percentages:</emph></td>
<td>relative to width and height of the applicable <loc href="#terms-background-positioning-rectangle">background positioning rectangle</loc></td>
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This seems a bit confusing - isn't backgroundExtent specifying the background positioning rectangle?

Copy link
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

No. It is specifying the extent (size) of the image with respect to the background positioning rectangle. The default 'auto' uses the intrinsic size, but the author may override that, e.g., by specifying cover or contain or two measures. See CSS3 B&B L3 background-size property for more info.

Copy link
Contributor

@nigelmegitt nigelmegitt Sep 10, 2016

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Oh, then the name backgroundExtent seems to be confusing. Why not call it imageExtent?

Copy link
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Let me give this some thought, since we might need the same attribute be used with tt:image.

@nigelmegitt
Copy link
Contributor

Notes from f2f meeting 2016-09-19:

  1. Need to consider the style attribute names - they all are prefixed "background" but some refer to the background content and some to the background area. (there is the background painting area and the background positioning area, both defined in the terminology section)
  2. Consider whether backgroundExtent can apply to foreground images.
  3. Add some example images.

Proposal is to merge this and raise issues for each of the above.

@skynavga skynavga merged commit 09a014d into gh-pages Oct 1, 2016
@skynavga skynavga deleted the issue-175-background-styles branch October 2, 2016 17:44
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants