[prev in list] [next in list] [prev in thread] [next in thread] 

List:       kde-commits
Subject:    KDE/kdegraphics/kviewshell/plugins
From:       Stefan Kebekus <kebekus () kde ! org>
Date:       2006-11-05 20:51:35
Message-ID: 1162759895.596064.32262.nullmailer () svn ! kde ! org
[Download RAW message or body]

SVN commit 602383 by kebekus:

fixes minor problems with hyperlinks in DVI and re-loading in PDF files


 M  +7 -2      dvi/dviRenderer.cpp  
 M  +1 -1      pdf/pdfRenderer.cpp  
 M  +1 -1      pdf/pdfRenderer.h  


--- trunk/KDE/kdegraphics/kviewshell/plugins/dvi/dviRenderer.cpp \
#602382:602383 @@ -202,8 +202,13 @@
 
     bool merged = false;
 
-    // Merge all hyperlinks that point to the same target, and have the \
                same baseline.
-    while (hi.linkText == hj.linkText && hi.baseline == hj.baseline)
+    // Merge all hyperlinks that point to the same target, have the
+    // same baseline, and are no more than 3mm (but at least 2 pixels)
+    // apart
+    int minDistance = QMAX(2, (int)(0.3/2.54*id.resolution+0.5)); // \
Compute what 3mm is in pixels +    while ((hi.linkText == hj.linkText) &&
+           (hi.baseline == hj.baseline) && 
+           ( hi.box.unite(hj.box).width() <= \
hi.box.width()+hj.box.width()+minDistance ))  {
       merged = true;
       hi.box = hi.box.unite(hj.box);
--- trunk/KDE/kdegraphics/kviewshell/plugins/pdf/pdfRenderer.cpp \
#602382:602383 @@ -500,7 +500,7 @@
 }
 
 
-bool PdfRenderer::isValidFile(const QString fileName)
+bool PdfRenderer::isValidFile(const QString& fileName) const
 {
   Poppler::Document* doc = Poppler::Document::load(fileName.toAscii());
   if (doc)
--- trunk/KDE/kdegraphics/kviewshell/plugins/pdf/pdfRenderer.h \
#602382:602383 @@ -118,7 +118,7 @@
   virtual void clear();
 
   /** Check if the give file is a valid PDF Document. */
-  virtual bool isValidFile(const QString fileName);
+  virtual bool isValidFile(const QString& fileName) const;
 
   /** Rendering a page
 


[prev in list] [next in list] [prev in thread] [next in thread]