<?xml version="1.0" encoding="UTF-8" standalone="yes" ?>
<!DOCTYPE bugzilla SYSTEM "https://www.w3.org/Bugs/Public/page.cgi?id=bugzilla.dtd">

<bugzilla version="5.0.4"
          urlbase="https://www.w3.org/Bugs/Public/"
          
          maintainer="sysbot+bugzilla@w3.org"
>

    <bug>
          <bug_id>22789</bug_id>
          
          <creation_ts>2013-07-24 06:25:37 +0000</creation_ts>
          <short_desc>WebVTT: Data model for line position should not include overlap avoidance</short_desc>
          <delta_ts>2014-01-28 11:17:44 +0000</delta_ts>
          <reporter_accessible>1</reporter_accessible>
          <cclist_accessible>1</cclist_accessible>
          <classification_id>1</classification_id>
          <classification>Unclassified</classification>
          <product>TextTracks CG</product>
          <component>WebVTT</component>
          <version>unspecified</version>
          <rep_platform>PC</rep_platform>
          <op_sys>All</op_sys>
          <bug_status>RESOLVED</bug_status>
          <resolution>WORKSFORME</resolution>
          
          
          <bug_file_loc></bug_file_loc>
          <status_whiteboard>v1</status_whiteboard>
          <keywords></keywords>
          <priority>P2</priority>
          <bug_severity>normal</bug_severity>
          <target_milestone>---</target_milestone>
          
          
          <everconfirmed>1</everconfirmed>
          <reporter name="Silvia Pfeiffer">silviapfeiffer1</reporter>
          <assigned_to name="Silvia Pfeiffer">silviapfeiffer1</assigned_to>
          <cc>mike</cc>
    
    <cc>philipj</cc>
    
    <cc>public-texttracks</cc>
          
          <qa_contact name="This bug has no owner yet - up for the taking">dave.null</qa_contact>

      

      

      

          <comment_sort_order>oldest_to_newest</comment_sort_order>  
          <long_desc isprivate="0" >
    <commentid>91207</commentid>
    <comment_count>0</comment_count>
    <who name="Silvia Pfeiffer">silviapfeiffer1</who>
    <bug_when>2013-07-24 06:25:37 +0000</bug_when>
    <thetext>The data model for line positioning is defined in:
http://dev.w3.org/html5/webvtt/#dfn-text-track-cue-line-position

If line is auto (i.e. not set), then the line position is determined based on the other text tracks that are showing.

I have several issues with this approach:

Firstly, it assumes that each cue is only one line long and thus adds &quot;1&quot; to the line position of every track. This is a faulty assumption.

Secondly, it would be better to have the same overlap avoidance that applies to multiple cues within a track applied to all cues that are active at a certain point in time.

I&apos;m therefore suggesting to remove steps 4 - 10 of that algorithm.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>91217</commentid>
    <comment_count>1</comment_count>
    <who name="Silvia Pfeiffer">silviapfeiffer1</who>
    <bug_when>2013-07-24 09:56:55 +0000</bug_when>
    <thetext>Also, the notion of &quot;text track cue computed line position&quot; is only used in the rendering section so it should be moved there completely.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>99276</commentid>
    <comment_count>2</comment_count>
    <who name="Philip Jägenstedt">philipj</who>
    <bug_when>2014-01-28 08:28:44 +0000</bug_when>
    <thetext>(In reply to Silvia Pfeiffer from comment #0)
&gt; The data model for line positioning is defined in:
&gt; http://dev.w3.org/html5/webvtt/#dfn-text-track-cue-line-position
&gt; 
&gt; If line is auto (i.e. not set), then the line position is determined based
&gt; on the other text tracks that are showing.
&gt; 
&gt; I have several issues with this approach:
&gt; 
&gt; Firstly, it assumes that each cue is only one line long and thus adds &quot;1&quot; to
&gt; the line position of every track. This is a faulty assumption.

Assuming the cues are rendered in the correct order then even if they are multi-line they should move due to overlap avoidance, so they should end up in the correct order. However, if that&apos;s true, I&apos;m not sure why they can&apos;t all be placed at -1 to let the overlap algorithm do all the work.

&gt; Secondly, it would be better to have the same overlap avoidance that applies
&gt; to multiple cues within a track applied to all cues that are active at a
&gt; certain point in time.

I think this is already the case, the rendering section already considers all the tracks together.

&gt; I&apos;m therefore suggesting to remove steps 4 - 10 of that algorithm.

To help illuminate this issue, can you show an example where the rendered result would be different and how removing these steps is better? (If there aren&apos;t any cases where this matters then of course the steps can be removed.)</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>99288</commentid>
    <comment_count>3</comment_count>
    <who name="Silvia Pfeiffer">silviapfeiffer1</who>
    <bug_when>2014-01-28 11:17:44 +0000</bug_when>
    <thetext>This is a 18 months old and the line numbers probably don&apos;t apply any longer.
I&apos;m going to close this, since we are now discussion on list how to integrate the rendering algorithms. Whatever we come up with there should be applied and not this.</thetext>
  </long_desc>
      
      

    </bug>

</bugzilla>