[webkit-changes] cvs commit: WebCore/khtml/xml dom_docimpl.cpp

[Adele]

adele       05/10/28 01:01:54

  Modified:    .        ChangeLog
               khtml/xml dom_docimpl.cpp
  Log:
          Reviewed by Hyatt.
  
          -fixed <rdar://problem/4081091> focus() does not work properly on anchors
          <rdar://problem/4317689> new form elements don't get revealed when focus() is called on them
  
          * khtml/xml/dom_docimpl.cpp:
          (DocumentImpl::setFocusNode): call scrollRectToVisible.
  
  Revision  Changes    Path
  1.311     +10 -0     WebCore/ChangeLog
  
  Index: ChangeLog
  ===================================================================
  RCS file: /cvs/root/WebCore/ChangeLog,v
  retrieving revision 1.310
  retrieving revision 1.311
  diff -u -r1.310 -r1.311
  --- ChangeLog	28 Oct 2005 07:03:24 -0000	1.310
  +++ ChangeLog	28 Oct 2005 08:01:47 -0000	1.311
  @@ -1,3 +1,13 @@
  +2005-10-28  Adele Peterson  <adele at apple.com>
  +
  +        Reviewed by Hyatt.
  +
  +        -fixed <rdar://problem/4081091> focus() does not work properly on anchors
  +        <rdar://problem/4317689> new form elements don't get revealed when focus() is called on them
  +
  +        * khtml/xml/dom_docimpl.cpp:
  +        (DocumentImpl::setFocusNode): call scrollRectToVisible.
  +
   2005-10-27  David Hyatt  <hyatt at apple.com>
   
   	Fix for bug 5519, buttons need to honor background/border: none and turn off styles.
  
  
  
  1.265     +6 -1      WebCore/khtml/xml/dom_docimpl.cpp
  
  Index: dom_docimpl.cpp
  ===================================================================
  RCS file: /cvs/root/WebCore/khtml/xml/dom_docimpl.cpp,v
  retrieving revision 1.264
  retrieving revision 1.265
  diff -u -r1.264 -r1.265
  --- dom_docimpl.cpp	27 Oct 2005 23:15:03 -0000	1.264
  +++ dom_docimpl.cpp	28 Oct 2005 08:01:53 -0000	1.265
  @@ -2329,8 +2329,13 @@
               }
               if (focusWidget)
                   focusWidget->setFocus();
  -            else
  +            else {
                   view()->setFocus();
  +                if (m_focusNode->renderer()) {
  +                    updateLayout();
  +                    m_focusNode->renderer()->enclosingLayer()->scrollRectToVisible(m_focusNode->getRect());
  +                }
  +            }
           }
      }