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

Usage pattern for specifying an image's alternate text. #490

Open
skynavga opened this issue Sep 27, 2019 · 8 comments
Open

Usage pattern for specifying an image's alternate text. #490

skynavga opened this issue Sep 27, 2019 · 8 comments

Comments

@skynavga
Copy link
Contributor

IMSC1.1 [1] states that

A div element that contains a child image element SHOULD contain a metadata element containing an altText named metadata item that is a Text Alternative of the image resource referenced by the image element.

This is inconsistent with the intended usage shown in TTML2 [2], which would have

<div>
  <image ...>
    <ttm:item name="altText">Alternate Text</ttm:item>
  </image>
</div>

instead of the current recommendation which would have

<div>
  <metadata>
    <ttm:item name="altText">Alternate Text</ttm:item>
  </metadata>
  <image .../>
</div>

The problem with this latter usage is that the alternate text is not semantically associated (by document structure) with the image, but instead, with the div.

Apparently I did not catch this earlier when IMSC1.1 was being published. I would suggest this be addressed in IMSC1.2 by deprecating the current recommendation in favor of a new recommendation based on the first usage example shown above.

[1] https://www.w3.org/TR/ttml-imsc1.1/#image-ttml-image
[2] https://www.w3.org/TR/2018/REC-ttml2-20181108/#xml-lang-example-1

@palemieux
Copy link
Contributor

The problem with this latter usage is that the alternate text is not semantically associated (by document structure) with the image, but instead, with the div.

I am not convinced it is a fatal issue since, in Image Profile, each div contains zero or one image element. This also parallels the use of ittm:altText in conjunction with smpte:backgroundImage.

If we decide the current approach was an error, then IMHO it should be changed in IMSC 1.1.

@skynavga
Copy link
Contributor Author

I didn't say it was a fatal issue, but I do believe it was an error, and not consistent with the semantics of metadata association as described shown in examples in TTML. I don't have a strong opinion as to where it gets addressed, but it should be addressed somewhere.

@skynavga
Copy link
Contributor Author

As for paralleling the use of smpte:backgroundImage, that was an attribute on div, so it made sense there, but image is a child of div, so it is not quite the same.

@nigelmegitt
Copy link
Contributor

nigelmegitt commented Oct 15, 2019

Sorry for the not very helpful comment, but I have a half-recollection that we discussed this point before arriving at the current solution. I don't have time to do the archaeological dig to find that right now though.

@nigelmegitt
Copy link
Contributor

@skynavga we'd like to be able to resolve this before or during Thursday's call if possible.

@skynavga
Copy link
Contributor Author

@nigelmegitt I don't see a proposed resolution.

@palemieux
Copy link
Contributor

@skynavga The proposal is to do nothing.

@css-meeting-bot
Copy link
Member

The Timed Text Working Group just discussed Usage pattern for specifying an image's alternate text. imsc#490, and agreed to the following:

  • RESOLUTION: We will not address this in IMSC 1.2 but leave on the backlog for potentially fixing in some future version.
The full IRC log of that discussion <nigel> Topic: Usage pattern for specifying an image's alternate text. imsc#490
<nigel> github: https://github.com//issues/490
<nigel> Nigel: This has been in IMSC since 1.1 hasn't it?
<nigel> Pierre: Correct, and it follows the pattern for when smpte:backgroundImage is used, which is why we ended up here.
<nigel> .. Another data point is we should really refrain from making changes to image profile unless absolutely necessary.
<nigel> .. Maybe in the future there will be some new requirement for the image profile that will give us an opportunity to
<nigel> .. correct these things. At this point I don't think it would be worth the trouble and it would be particularly disruptive.
<nigel> .. I'm happy to close it or deschedule it and defer it to a backlog so we don't forget about it.
<nigel> Nigel: That was going to be my suggestion.
<nigel> Pierre: I propose moving to the backlog.
<nigel> Nigel: Any other views on this? Any problems saying we may deal with it one day but not right now?
<nigel> Pierre: An advantage of doing this is that if someone runs into this and starts raising an issue they should be able to
<nigel> .. find it in the open issues list fairly quickly, and comment on it.
<nigel> Nigel: I'm hearing no objections - obviously Glenn is absent but he has the opportunity to comment during the review
<nigel> .. period as per our Decision Policy.
<nigel> RESOLUTION: We will not address this in IMSC 1.2 but leave on the backlog for potentially fixing in some future version.

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

No branches or pull requests

4 participants