23744 2009-02-04 14:45:50 -0800 stylesheet not applied because css file not parsed correctly if it contains comments and charset 2009-10-16 12:07:53 -0700 1 1 1 Unclassified WebKit CSS 525.x (Safari 3.1) All All RESOLVED DUPLICATE 18265 http://csdraveurs.qc.ca/ HasReduction P2 Normal --- 1 jasneet webkit-unassigned abarth ap jasneet jeff oldest_to_newest 108335 0 jasneet 2009-02-04 14:45:50 -0800 I Steps: Go to http://csdraveurs.qc.ca/ II Issue: left navigation bar links have extra scroll bars III Conclusion: The issue is caused by the css file not being applied correctly because of the /*comment*/ and @charset in its contents. The css file looks like: /* Définition de règle */ @charset "iso-8859-1"; .leftframe {color: 0067c6; font-family: "Times New Roman", Times, serif; font-size: 10px; font-weight: bold; line-height: normal; text-align: center} IV Other Browsers: IE7: ok FF3: ok V Nightly tested: 40471 Bug in Chromium : http://code.google.com/p/chromium/issues/detail?id=5771 108336 1 27330 jasneet 2009-02-04 14:48:14 -0800 Created attachment 27330 testcase 109797 2 ap 2009-02-15 01:44:19 -0800 This is broken CSS - "@charset" should be the first bytes in file, except for BOM. But obviously, we should be more forgiving for compatibility with other browsers. One thing we should test for is whether this charset is honored, or just ignored. 110384 3 taruchhaya 2009-02-20 05:07:04 -0800 I have done some analysis on this bug.My observations are: 1) "leftframe.css" :: Issue is caused by the css file not being applied correctly because of the /*comment*/ and @charset in its contents. The css file looks like: /* Dfinition de rgle */ @charset "iso-8859-1"; .leftframe {color: 0067c6; font-family: "Times New Roman", Times, serif; font-size: 10px; font-weight: bold; line-height: normal; text-lign: center} Reason::The @charset At-Rule is allowed to occur only once in an external style sheet and it must be the very first statement in the style sheet. QUERIES: 1) "/*comment*/" is a valid CSS statement, which is escaped/ignored while executing the code. So shouldn't the same approach be followed in this case and @charset should be considered/honored. While debugging the Grammar.y, I found that "/*comment*/" is not the issue. The issue is in "/r" & "/n" after the comment, which is forcing the parser to consider the rest of the document as an invalid rule. 2) I also found another issue in this bug,i.e. in "Link Tag". Reason:: In Safari, Link Tag is not able to open/recognize the externally linked CSS file. As, HTML document contains the following snippet for link tag: <link href="leftframe.css" rel="stylesheet" type="text/css"> <link href="../leftframe.css" rel="stylesheet" type="text/css"> Here, In second link statement: "../leftframe.css" is invalid because it is trying to access the .css file from the root level, which is not available, webkit should check this condition and then resolve it by appending to the base URI just like how IE does. Please provide me some suggestions on the above issues. 110386 4 ap 2009-02-20 05:19:08 -0800 (In reply to comment #3) > 2) I also found another issue in this bug,i.e. in "Link Tag". Please file a new bug for this new issue. 110528 5 taruchhaya 2009-02-22 20:51:10 -0800 Raised the new Bug ID: 24084:: "Link Tag" issue 116849 6 ap 2009-04-08 01:59:09 -0700 See also: bug 18265 (@charset in inline CSS). 116853 7 ap 2009-04-08 02:28:22 -0700 *** Bug 23947 has been marked as a duplicate of this bug. *** 155163 8 abarth 2009-10-16 12:07:53 -0700 This is the same a Bug 18265 because the comment get stripped out by the tokenizer. *** This bug has been marked as a duplicate of bug 18265 *** 27330 2009-02-04 14:48:14 -0800 2009-02-04 14:48:14 -0800 testcase testcase.zip application/zip 895 jasneet UEsDBBQAAAAIAKeSKDrMEVJ5AAEAAG0BAAAJAHEAdGVzdC5odG1sU0RcAKQAAAAACABGylEQY2Rg aRBhYGAwYIAAHyBmZAUzWUWBxAyhcKZHhgE6q9SqOBx7mHDKMTIxMDAxRDAwg6VVGP4zyuM1h4FB BKKGEaJGCExJQMSYIGIKQEIByAYAVVQNAAeZtGZJbLVmSSC0ZklNkE9PwkAQxe8mfodx7nYk4AVa Ei1NJAEhtMZ4XNsp27i0ze5I4dvbLZC4l7fz5/3yMuHDYhNnX9sE3rL1CrYfr6tlDPhI9DmOiRbZ 4jKYBE8jouQdAbVIOyXqui7oxkFj95TtSMvBTMiJrXIJCilwfn8X+uagrAqvpqp/QFsuIzRcSmnV gYPcOQTLJkInZ8NOMwuCnFuOUPgk5Be8m26Y76Y499r/iuoIuVHO/QMiDJwImyPb0jTdFNSvNDPo qkL0FEbPo/Y0A83VXosv+8rzAQacMtW+jjDnWthe+v6F6hoc56skhXizXi/T9GW5S9KQ1MVOvX8I etNr0JCuh/gDUEsDBBQAAAAIAAGTKDrcKUWMoQAAAMkAAAANAHEAbGVmdGZyYW1lLmNzc1NEXACk AAAAAAgARspREGNkYGkQYWBgMGCAAB8gZmQFM1lFgcQMoXCmR4YBOqvUqjgce5hwyjEyMTAwMUQw MIOlVRj+M8rjNYeBQQSihhGiRghMSUDEmCBiCkBCAcgGAFVUDQAHQbVmSYC1ZkkXtGZJLcq7CsIw FAbgXfAdfjKWxtbBquni4OwgvkBsT9oDuUgSqBd8X30LQbp+fFWB49ew58zBoyfEz2AJRbVcHLpR x0QZglOQu91mL9eiXS5Wlkw2UTvCqws2RIW6brZd08IEn6XRju1DAeLCjhJONOEcnPaixF9KJIps 5p74SQrr+nafYSIexqxwDbZvYdmTHGfyITptW2S6Z6ktD16hI58pvn9QSwECFwsUAAAACACnkig6 zBFSeQABAABtAQAACQARAAAAAAABACAAtoEAAAAAdGVzdC5odG1sU0QEAKQAAABVVAUAB5m0ZklQ SwECFwsUAAAACAABkyg63ClFjKEAAADJAAAADQARAAAAAAABACAAtoGYAQAAbGVmdGZyYW1lLmNz c1NEBACkAAAAVVQFAAdBtWZJUEsFBgAAAAACAAIAlAAAANUCAAAAAA==