Summary: | article comments in digg displayed wrong | ||
---|---|---|---|
Product: | [Applications] konqueror | Reporter: | dresnu <j053f.k> |
Component: | general | Assignee: | Konqueror Developers <konq-bugs> |
Status: | RESOLVED FIXED | ||
Severity: | normal | CC: | germain |
Priority: | NOR | ||
Version: | unspecified | ||
Target Milestone: | --- | ||
Platform: | Ubuntu | ||
OS: | Linux | ||
Latest Commit: | Version Fixed In: |
Description
dresnu
2008-07-04 17:18:34 UTC
Original poster here. The problem seems to be javascript. When it is disabled I can load digg pages without any problem, but then enabling javascript and the debugger gives a whole lot of errors and eventually crashes the page. In particular the line: dapMgr.enableACB("top_ad_msft", false); in a script type seems to be the first error in the page. 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 ); |