Version: Konqueror 4.1.1 (Windows XP) (using KDE 4.1.1) Compiler: n/a n/a OS: MS Windows Installed from: MS Windows http://pc44.one.pl/goorol/bugs/kde_002.html KHTML JS ignores empty rules in styleSheets!
Created attachment 27244 [details] source
Confirmed on konqueror 4.0
SVN commit 867194 by ggarand: do not skip empty CSS rules - they must appear in the cssRules array. digg.com, for instance, needs this to perform some unspeakable CSS hacks. BUG: 170411, 165734 M +1 -1 parser.cpp M +1 -1 parser.y --- trunk/KDE/kdelibs/khtml/css/parser.cpp #867193:867194 @@ -2427,7 +2427,7 @@ kDebug( 6080 ) << "got ruleset" << endl << " selector:"; #endif CSSParser *p = static_cast<CSSParser *>(parser); - if ( (yyvsp[(1) - (2)].selectorList) && (yyvsp[(2) - (2)].ok) && p->numParsedProperties ) { + if ( (yyvsp[(1) - (2)].selectorList) ) { CSSStyleRuleImpl *rule = new CSSStyleRuleImpl( p->styleElement ); CSSStyleDeclarationImpl *decl = p->createStyleDeclaration( rule ); rule->setSelector( (yyvsp[(1) - (2)].selectorList) ); --- trunk/KDE/kdelibs/khtml/css/parser.y #867193:867194 @@ -648,7 +648,7 @@ kDebug( 6080 ) << "got ruleset" << endl << " selector:"; #endif CSSParser *p = static_cast<CSSParser *>(parser); - if ( $1 && $2 && p->numParsedProperties ) { + if ( $1 ) { CSSStyleRuleImpl *rule = new CSSStyleRuleImpl( p->styleElement ); CSSStyleDeclarationImpl *decl = p->createStyleDeclaration( rule ); rule->setSelector( $1 );
SVN commit 873668 by ggarand: automatically merged revision 867194: do not skip empty CSS rules - they must appear in the cssRules array. digg.com, for instance, needs this to perform some unspeakable CSS hacks. BUG: 170411, 165734 M +1 -1 parser.cpp M +1 -1 parser.y WebSVN link: http://websvn.kde.org/?view=rev&revision=873668