Merged revisions 1012404 via svnmerge from

svn+ssh://tmcguire@svn.kde.org/home/kde/branches/kdepim/enterprise4/kdepim

................
  r1012404 | winterz | 2009-08-17 18:06:43 +0200 (Mon, 17 Aug 2009) | 10 lines
  
  Merged revisions 1011410 via svnmerge from 
  https://svn.kde.org/home/kde/branches/kdepim/enterprise/kdepim
  
  ........
    r1011410 | tmcguire | 2009-08-14 11:37:29 -0400 (Fri, 14 Aug 2009) | 3 lines
    
    Also scroll when clicking an attachment in the header.
    Thanks to the *Eagle for his help!
  ........
................

svn path=/trunk/KDE/kdepim/; revision=1013411

kmreaderwin.cpp

@@ -2887,6 +2887,10 @@ void KMReaderWin::scrollToAttachment( const partNode *node )
   }
   attachmentDiv.setAttribute( "style", QString( "border:2px solid %1" )
       .arg( cssHelper()->pgpWarnColor().name() ) );
+
+  // Update rendering, otherwise the rendering is not updated when the user clicks on an attachment
+  // that causes scrolling and the open attachment dialog
+  doc.updateRendering();
 }
 
 void KMReaderWin::injectAttachments()

urlhandlermanager.cpp

@@ -556,9 +556,9 @@ namespace {
 
     const bool inHeader = attachmentIsInHeader( url );
     const bool shouldShowDialog = !node->isDisplayedEmbedded() || !inHeader;
-    if ( !shouldShowDialog )
+    if ( inHeader )
       w->scrollToAttachment( node );
-    else
+    if ( shouldShowDialog )
       // PENDING(romain_kdab) : replace with toLocalFile() ?
       w->openAttachment( node->nodeId(), w->tempFileUrlFromPartNode( node ).path() );
     return true;