<?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>10511</bug_id>
          
          <creation_ts>2010-08-30 17:10:44 +0000</creation_ts>
          <short_desc>[scroll] fragment identifier algo should use scroll element into view</short_desc>
          <delta_ts>2010-10-13 20:19:51 +0000</delta_ts>
          <reporter_accessible>1</reporter_accessible>
          <cclist_accessible>1</cclist_accessible>
          <classification_id>1</classification_id>
          <classification>Unclassified</classification>
          <product>HTML WG</product>
          <component>pre-LC1 HTML5 spec (editor: Ian Hickson)</component>
          <version>unspecified</version>
          <rep_platform>PC</rep_platform>
          <op_sys>All</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>10533</dependson>
          
          <everconfirmed>1</everconfirmed>
          <reporter name="Anne">annevk</reporter>
          <assigned_to name="Ian &apos;Hixie&apos; Hickson">ian</assigned_to>
          <cc>bzbarsky</cc>
    
    <cc>ian</cc>
    
    <cc>mike</cc>
    
    <cc>public-html-admin</cc>
    
    <cc>public-html-wg-issue-tracking</cc>
          
          <qa_contact name="HTML WG Bugzilla archive list">public-html-bugzilla</qa_contact>

      

      

      

          <comment_sort_order>oldest_to_newest</comment_sort_order>  
          <long_desc isprivate="0" >
    <commentid>38218</commentid>
    <comment_count>0</comment_count>
    <who name="Anne">annevk</who>
    <bug_when>2010-08-30 17:10:44 +0000</bug_when>
    <thetext>http://dev.w3.org/csswg/cssom-view/#scroll-an-element-into-view should be used by the fragment identifier stuff, I think.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>38219</commentid>
    <comment_count>1</comment_count>
    <who name="Boris Zbarsky">bzbarsky</who>
    <bug_when>2010-08-30 17:29:17 +0000</bug_when>
    <thetext>Note that there are some security/privacy issues here as regards scrolling up the parent iframe chain; Mozilla&apos;s been considering changing its behavior in various ways for this stuff.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>38278</commentid>
    <comment_count>2</comment_count>
    <who name="Anne">annevk</who>
    <bug_when>2010-08-31 05:43:19 +0000</bug_when>
    <thetext>The algorithm is blocking on cross-origin at the moment.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>39378</commentid>
    <comment_count>3</comment_count>
    <who name="Ian &apos;Hixie&apos; Hickson">ian</who>
    <bug_when>2010-09-25 18:16:34 +0000</bug_when>
    <thetext>Why shouldn&apos;t this be left up to the browser? The CSSOM algorithm seems to be media-specific, too.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>39461</commentid>
    <comment_count>4</comment_count>
    <who name="Anne">annevk</who>
    <bug_when>2010-09-26 09:33:28 +0000</bug_when>
    <thetext>I think visual user agents should do the same thing. And should be equally protected from the privacy issue.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>39894</commentid>
    <comment_count>5</comment_count>
    <who name="Ian &apos;Hixie&apos; Hickson">ian</who>
    <bug_when>2010-09-29 07:46:07 +0000</bug_when>
    <thetext>By media-specific, I mean that the algorithm in CSSOM makes no sense e.g. for audio browsers, unlike the one in the HTML spec which, while vague, does at least make sense for those media.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>39910</commentid>
    <comment_count>6</comment_count>
    <who name="Anne">annevk</who>
    <bug_when>2010-09-29 08:14:35 +0000</bug_when>
    <thetext>Right, I meant that HTML5 could keep the generic stuff but defer for visual media. But maybe CSSOM View should take it all given that scrollIntoView() uses the same algorithm.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>40279</commentid>
    <comment_count>7</comment_count>
    <who name="Ian &apos;Hixie&apos; Hickson">ian</who>
    <bug_when>2010-09-30 19:44:17 +0000</bug_when>
    <thetext>Let me know what you decide to do and I&apos;ll fix this bug and 10533 accordingly.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>41261</commentid>
    <comment_count>8</comment_count>
    <who name="Ian &apos;Hixie&apos; Hickson">ian</who>
    <bug_when>2010-10-13 20:19:51 +0000</bug_when>
    <thetext>Checked in as WHATWG revision r5620.
Check-in comment: Move scrollIntoView() to CSSOM View and update the other
scroll algorithm to use the CSSOM View definition.
http://html5.org/tools/web-apps-tracker?from=5619&amp;to=5620

EDITOR&apos;S RESPONSE: This is an Editor&apos;s Response to your comment. If you are satisfied with this response, please change the state of this bug to CLOSED. If you have additional information and would like the editor to reconsider, please reopen this bug. If you would like to escalate the issue to the full HTML Working Group, please add the TrackerRequest keyword to this bug, and suggest title and text for the tracker issue; or you may create a tracker issue yourself, if you are able to do so. For more details, see this document:
   http://dev.w3.org/html5/decision-policy/decision-policy.html

Status: Accepted
Change Description: see diff given above
Rationale: CSSOM View will be updated accordingly</thetext>
  </long_desc>
      
      

    </bug>

</bugzilla>