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

<bugzilla version="5.0.4.1"
          urlbase="https://bugs.webkit.org/"
          
          maintainer="admin@webkit.org"
>

    <bug>
          <bug_id>20361</bug_id>
          
          <creation_ts>2008-08-12 10:46:42 -0700</creation_ts>
          <short_desc>Should WebKit implement on* event handler properties for transition events?</short_desc>
          <delta_ts>2008-09-01 17:15:02 -0700</delta_ts>
          <reporter_accessible>1</reporter_accessible>
          <cclist_accessible>1</cclist_accessible>
          <classification_id>1</classification_id>
          <classification>Unclassified</classification>
          <product>WebKit</product>
          <component>DOM</component>
          <version>528+ (Nightly build)</version>
          <rep_platform>Mac</rep_platform>
          <op_sys>OS X 10.5</op_sys>
          <bug_status>RESOLVED</bug_status>
          <resolution>FIXED</resolution>
          
          
          <bug_file_loc></bug_file_loc>
          <status_whiteboard></status_whiteboard>
          <keywords></keywords>
          <priority>P2</priority>
          <bug_severity>Normal</bug_severity>
          <target_milestone>---</target_milestone>
          <dependson>20571</dependson>
          
          <everconfirmed>1</everconfirmed>
          <reporter name="Dean Jackson">dino</reporter>
          <assigned_to name="Nobody">webkit-unassigned</assigned_to>
          <cc>dino</cc>
    
    <cc>hyatt</cc>
    
    <cc>rik</cc>
    
    <cc>simon.fraser</cc>
          

      

      

      

          <comment_sort_order>oldest_to_newest</comment_sort_order>  
          <long_desc isprivate="0" >
    <commentid>88320</commentid>
    <comment_count>0</comment_count>
    <who name="Dean Jackson">dino</who>
    <bug_when>2008-08-12 10:46:42 -0700</bug_when>
    <thetext>Currently the CSS Animation and Transition events are implemented using addEventListener and HTML attributes. Should we also add the .onwebkittransitionevent property on Element?</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>88321</commentid>
    <comment_count>1</comment_count>
    <who name="Timothy Hatcher">timothy</who>
    <bug_when>2008-08-12 10:50:36 -0700</bug_when>
    <thetext>One (cranky and sleepy) side of me says we should only support addEventListener for these events. Since I see the on* event attributes/properties as legacy. If we don&apos;t plan to add the properties we should remove the HTML attributes.

Or we should just support all three and let people use whatever they like, even if HTML attributes and string values assigned to event properties are less performant.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>88326</commentid>
    <comment_count>2</comment_count>
    <who name="Anthony Ricaud">rik</who>
    <bug_when>2008-08-12 11:19:19 -0700</bug_when>
    <thetext>I think this is important because every event is accessible through properties. And this way of doing events is sometime more useful. For example, it&apos;s easier to delete events handler.

It also seems strange to have HTML attributes without DOM properties.
</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>88370</commentid>
    <comment_count>3</comment_count>
    <who name="Sam Weinig">sam</who>
    <bug_when>2008-08-12 20:32:13 -0700</bug_when>
    <thetext>(In reply to comment #1)
&gt; One (cranky and sleepy) side of me says we should only support addEventListener
&gt; for these events. Since I see the on* event attributes/properties as legacy. If
&gt; we don&apos;t plan to add the properties we should remove the HTML attributes.

What makes you feel that the attribute/property style event listeners is legacy.  Many new specs, including XMLHttpRequest Level 2, include new ones.(In reply to comment #2)

&gt; I think this is important because every event is accessible through properties.
&gt; And this way of doing events is sometime more useful. For example, it&apos;s easier
&gt; to delete events handler.

Just to be clear though, it is not true that every event is accessible through properties, the &quot;message&quot; event used by postMessage doesn&apos;t for example.

That said, I do think attribute/property style event listeners should be added for consistency in the API.
</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>89770</commentid>
    <comment_count>4</comment_count>
    <who name="Dean Jackson">dino</who>
    <bug_when>2008-08-29 19:33:32 -0700</bug_when>
    <thetext>Turns out that this was mostly (but not quite) implemented.
See https://bugs.webkit.org/show_bug.cgi?id=20571

</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>89880</commentid>
    <comment_count>5</comment_count>
    <who name="Dean Jackson">dino</who>
    <bug_when>2008-09-01 17:15:02 -0700</bug_when>
    <thetext>Done now in blocking bug.

</thetext>
  </long_desc>
      
      

    </bug>

</bugzilla>