publ-a11y icon indicating copy to clipboard operation
publ-a11y copied to clipboard

Update pre recorded audio

Open clapierre opened this issue 1 year ago • 1 comments

Note this is slightly different than the ONIX version, and will require some careful review / discussion.

clapierre avatar May 17 '24 18:05 clapierre

I have a few questions:

First, in the vvriable setup it says: /package/metadata/meta[@property="schema:accessModeSufficient" and text()="auditory"].

I don't know if the and text () is correct. Somebody should check. This appears in several places here.

Second, regarding accessibilityFeature="ttsMarkup" , I understand that this has the potential to improve TTS pronunciations , but does it belong in the pre-recorded audio section? It may be that the synchronized audio, if generated by TTS would probably be better, but should we say anything in this section? I don't think we know if the audio is a human or TTS.

GeorgeKerscher avatar May 18 '24 16:05 GeorgeKerscher

I don't know if the and text () is correct. Somebody should check. This appears in several places here.

That is correct. In the XPATH language, it is the way to identify the value contained in an XML tag, so it is the way to find the XML element with a certain value.

gregoriopellegrino avatar May 19 '24 09:05 gregoriopellegrino

@GeorgeKerscher, I was mistaken as I thought the ONIX code's 20 and 21 was referring to Code List 196 which does align with ttsMarkup as we have in our Crosswalk document. I agree that the doesn't belong for pre-recorded audio, so I will remove this.

clapierre avatar May 20 '24 16:05 clapierre

Ok, I made that change @GeorgeKerscher and removed ttsMarkup from the EPUB pre-recorded audio technique.

clapierre avatar May 20 '24 16:05 clapierre

With the changes made, can we merge this? @GeorgeKerscher @gregoriopellegrino

clapierre avatar May 21 '24 14:05 clapierre