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

List:       kde-commits
Subject:    branches/KDE/4.1/kdelibs/khtml/css
From:       Harri Porten <porten () kde ! org>
Date:       2008-08-24 20:24:43
Message-ID: 1219609483.030629.1469.nullmailer () svn ! kde ! org
[Download RAW message or body]

SVN commit 851879 by porten:

Merged revision 851878:
Fixed rgba() string representation. Instead of #rrggbb
use rgb() as Firefox does.

 M  +6 -4      css_valueimpl.cpp  


--- branches/KDE/4.1/kdelibs/khtml/css/css_valueimpl.cpp #851878:851879
@@ -1149,12 +1149,14 @@
 		if (m_value.rgbcolor == khtml::transparentColor)
 		    text = "transparent";
 		else
-		    text = "rgba(" + QString::number(qRed  (m_value.rgbcolor)) + ","
-				   + QString::number(qBlue (m_value.rgbcolor)) + ","
-				   + QString::number(qGreen(m_value.rgbcolor)) + ","
+		    text = "rgba(" + QString::number(qRed  (m_value.rgbcolor)) + ", "
+				   + QString::number(qGreen(m_value.rgbcolor)) + ", "
+				   + QString::number(qBlue (m_value.rgbcolor)) + ", "
 				   + QString::number(qAlpha(m_value.rgbcolor)/255.0) + ")";
 	    } else {
-		text = QColor(m_value.rgbcolor).name();
+                text = "rgb(" + QString::number(qRed  (m_value.rgbcolor)) + ", "
+                              + QString::number(qGreen(m_value.rgbcolor)) + ", "
+                              + QString::number(qBlue (m_value.rgbcolor)) + ")";
 	    }
 	    break;
         case CSSPrimitiveValue::CSS_PAIR:
[prev in list] [next in list] [prev in thread] [next in thread]