qgis · Dec 1, 2014
diff --git a/‎src/core/composer/qgscomposerhtml.cpp
Lines changed: 5 additions & 1 deletion b/‎src/core/composer/qgscomposerhtml.cpp
Lines changed: 5 additions & 1 deletion
diff --git a/‎tests/src/core/testqgscomposerhtml.cpp
Lines changed: 2 additions & 2 deletions b/‎tests/src/core/testqgscomposerhtml.cpp
Lines changed: 2 additions & 2 deletions
diff --git a/‎tests/src/python/test_qgscomposerhtml.py
Lines changed: 2 additions & 2 deletions b/‎tests/src/python/test_qgscomposerhtml.py
Lines changed: 2 additions & 2 deletions
diff --git a/‎tests/testdata/control_images/expected_composerhtml_smartbreaks1/expected_composerhtml_smartbreaks1.png
-3 Bytes b/‎tests/testdata/control_images/expected_composerhtml_smartbreaks1/expected_composerhtml_smartbreaks1.png
-3 Bytes
diff --git a/‎tests/testdata/control_images/expected_composerhtml_smartbreaks2/expected_composerhtml_smartbreaks2.png
25 Bytes b/‎tests/testdata/control_images/expected_composerhtml_smartbreaks2/expected_composerhtml_smartbreaks2.png
25 Bytes
@@ -376,6 +376,8 @@ double QgsComposerHtml::findNearbyPageBreak( double yPos )
   //of maxSearchDistance
   int changes = 0;
   QRgb currentColor;
+  bool currentPixelTransparent = false;
+  bool previousPixelTransparent = false;
   QRgb pixelColor;
   QList< QPair<int, int> > candidates;
   int minRow = qMax( idealPos - maxSearchDistance, 0 );
@@ -391,12 +393,14 @@ double QgsComposerHtml::findNearbyPageBreak( double yPos )
       //since this is likely a line break, or gap between table cells, etc
       //but very unlikely to be midway through a text line or picture
       pixelColor = mRenderedPage->pixel( col, candidateRow );
-      if ( pixelColor != currentColor )
+      currentPixelTransparent = qAlpha( pixelColor ) == 0;
+      if ( pixelColor != currentColor && !( currentPixelTransparent && previousPixelTransparent ) )
       {
         //color has changed
         currentColor = pixelColor;
         changes++;
       }
+      previousPixelTransparent = currentPixelTransparent;
     }
     candidates.append( qMakePair( candidateRow, changes ) );
   }
 
@@ -200,7 +200,7 @@ void TestQgsComposerHtml::tableMultiFrame()
 void TestQgsComposerHtml::htmlMultiFrameSmartBreak()
 {
   QgsComposerHtml* htmlItem = new QgsComposerHtml( mComposition, false );
-  QgsComposerFrame* htmlFrame = new QgsComposerFrame( mComposition, htmlItem, 10, 10, 100, 50 );
+  QgsComposerFrame* htmlFrame = new QgsComposerFrame( mComposition, htmlItem, 10, 10, 100, 52 );
   htmlItem->addFrame( htmlFrame );
   htmlItem->setResizeMode( QgsComposerMultiFrame::RepeatUntilFinished );
   htmlItem->setUseSmartBreaks( true );
@@ -209,7 +209,7 @@ void TestQgsComposerHtml::htmlMultiFrameSmartBreak()
   htmlItem->setUrl( QUrl( QString( "file:///%1" ).arg( QString( TEST_DATA_DIR ) + QDir::separator() +  "test_html.html" ) ) );
   htmlItem->frame( 0 )->setFrameEnabled( true );
   QgsCompositionChecker checker1( "composerhtml_smartbreaks1", mComposition );
-  bool result = checker1.testComposition( mReport );
+  bool result = checker1.testComposition( mReport, 0, 200 );
 
   //page2
   QgsCompositionChecker checker2( "composerhtml_smartbreaks2", mComposition );
 
@@ -102,7 +102,7 @@ def testHtmlSmartBreaks(self):
         """Test rendering to multiframes with smart breaks."""
         composerHtml = QgsComposerHtml(self.mComposition, False)
         htmlFrame = QgsComposerFrame(self.mComposition, composerHtml,
-                                     10, 10, 100, 50)
+                                     10, 10, 100, 52)
         composerHtml.addFrame(htmlFrame)
         composerHtml.setResizeMode(
             QgsComposerMultiFrame.RepeatUntilFinished)
@@ -113,7 +113,7 @@ def testHtmlSmartBreaks(self):
         print "Checking page 1"
         myPage = 0
         checker1 = QgsCompositionChecker('composerhtml_smartbreaks1', self.mComposition)
-        myTestResult, myMessage = checker1.testComposition( myPage )
+        myTestResult, myMessage = checker1.testComposition( myPage, 200 )
         assert myTestResult, myMessage
 
         print "Checking page 2"
Original file line number	Diff line number	Diff line change
`@@ -376,6 +376,8 @@ double QgsComposerHtml::findNearbyPageBreak( double yPos )`
`376`	`376`	`//of maxSearchDistance`
`377`	`377`	`int changes = 0;`
`378`	`378`	`QRgb currentColor;`
	`379`	`+ bool currentPixelTransparent = false;`
	`380`	`+ bool previousPixelTransparent = false;`
`379`	`381`	`QRgb pixelColor;`
`380`	`382`	`QList< QPair<int, int> > candidates;`
`381`	`383`	`int minRow = qMax( idealPos - maxSearchDistance, 0 );`
`@@ -391,12 +393,14 @@ double QgsComposerHtml::findNearbyPageBreak( double yPos )`
`391`	`393`	`//since this is likely a line break, or gap between table cells, etc`
`392`	`394`	`//but very unlikely to be midway through a text line or picture`
`393`	`395`	`pixelColor = mRenderedPage->pixel( col, candidateRow );`
`394`		`- if ( pixelColor != currentColor )`
	`396`	`+ currentPixelTransparent = qAlpha( pixelColor ) == 0;`
	`397`	`+ if ( pixelColor != currentColor && !( currentPixelTransparent && previousPixelTransparent ) )`
`395`	`398`	`{`
`396`	`399`	`//color has changed`
`397`	`400`	`currentColor = pixelColor;`
`398`	`401`	`changes++;`
`399`	`402`	`}`
	`403`	`+ previousPixelTransparent = currentPixelTransparent;`
`400`	`404`	`}`
`401`	`405`	`candidates.append( qMakePair( candidateRow, changes ) );`
`402`	`406`	`}`