Bug 167268

Summary:	konqueror, stylesheets, accessibility, colors
Product:	[Applications] konqueror	Reporter:	Emanuel Helms <kurama08>
Component:	general	Assignee:	Konqueror Bugs <konqueror-bugs-null>
Status:	RESOLVED FIXED
Severity:	normal	CC:	jlp, mail, maksim, ruano1983
Priority:	NOR
Version First Reported In:	unspecified
Target Milestone:	---
Platform:	Ubuntu
OS:	Linux
Latest Commit:		Version Fixed/Implemented In:
Sentry Crash Report:

Description Emanuel Helms 2008-07-23 00:53:30 UTC

Version:           4.00.98 (using KDE 4.0.98)
Installed from:    Ubuntu Packages
OS:                Linux

Hi,

when I activate the accessibility stylesheets in Konqueror and then change the colors to have a black bg and a white font it does not work. Alsa I can't change the link-color. This is a problem when I have a black bg. A darl blue link on black bg..

Its working in KDE3 but here I can't change the link color too.


Hope the problem will be fixed! :)

Comment 1 George Kiagiadakis 2008-07-23 01:00:50 UTC

I can confirm this, I set the accessibility stylesheet to have black font on white bg and tried to visit blackle.com, which, however, still appears as white font on black bg.

Comment 2 Maksim Orlovich 2008-10-01 15:42:30 UTC

*** Bug 171964 has been marked as a duplicate of this bug. ***

Comment 3 Maksim Orlovich 2008-10-01 15:42:48 UTC

Confirm breakage, testing fix now..

Comment 4 Maksim Orlovich 2008-10-01 15:48:42 UTC

To fix loading:
--- khtml_part.cpp	(revision 865951)
+++ khtml_part.cpp	(working copy)
@@ -128,6 +128,8 @@
 
 #include "ecma/debugger/debugwindow.h"
 
+#include <iostream>
+
 namespace khtml {
     class PartStyleSheetLoader : public CachedObjectClient
     {
@@ -144,7 +146,8 @@
         {
             if ( m_cachedSheet ) m_cachedSheet->deref(this);
         }
-        virtual void setStyleSheet(const DOM::DOMString&, const DOM::DOMString &sheet, const DOM::DOMString &)
+	
+	virtual void setStyleSheet(const DOM::DOMString&, const DOM::DOMString &sheet, const DOM::DOMString &, const DOM::DOMString &)
         {
           if ( m_part )
             m_part->setUserStyleSheet( sheet.string() );


.... But there seems to be an another bug in re-setting, also the kcm seems a bit buggy.

Comment 5 Maksim Orlovich 2008-10-27 01:13:05 UTC

SVN commit 876287 by orlovich:

Fix signature mis-match here, unbreaks loading of accessibility/user stylesheet, and also crash on exit 
when one is set.
BUG:167268
BUG:164796


 M  +1 -1      khtml_part.cpp  


WebSVN link: http://websvn.kde.org/?view=rev&revision=876287

Comment 6 Maksim Orlovich 2008-10-27 01:15:20 UTC

SVN commit 876288 by orlovich:

Merged revision 876287:
Fix signature mis-match here, unbreaks loading of accessibility/user stylesheet, and also crash on exit 
when one is set.
BUG:167268
BUG:164796

 M  +1 -1      khtml_part.cpp  


WebSVN link: http://websvn.kde.org/?view=rev&revision=876288