rc452860 / ie7-js

Automatically exported from code.google.com/p/ie7-js
0 stars 0 forks source link

:before :after problems in attribute content with (X)HTML entities as ISO in CSS content #254

Open GoogleCodeExporter opened 9 years ago

GoogleCodeExporter commented 9 years ago
What steps will reproduce the problem?
1. Create a style rule like this in the CSS file:
    #span_test:before {
        content:"\00a0\00a0|\00a0\00a0";
    }
Of course create an element with that id in HTML file.
2. Open it in any modern browser. I tested in Safari, Opera, Firefox and Chrome.
3. Open it in IE7 or IE6.
4. Compare.

What is the expected output? What do you see instead?
The browser must replace the string "\00a0" into a blank space, the equivalent 
to entity " ".
Expected result
"  |  "
What's instead:
"\00a0\00a0|\00a0\00a0"

What version of the product are you using?
2.1(beta4)

Please provide any additional information below.
The version 2.0(beta3) didn't have this bug.

Original issue reported on code.google.com by cab...@gmail.com on 8 Jun 2010 at 9:27

GoogleCodeExporter commented 9 years ago
would be useful to fix this cause it will be necessary for the quote property 
too (LTE IE7 not supported)

Original comment by fun...@gmail.com on 12 Jun 2010 at 2:20

GoogleCodeExporter commented 9 years ago
I've attached a patch that brings the escaped unicode character support from 
2.0(beta3) to 2.1(beta4). I applied it to IE9.js, but it looks like it will 
cleanly apply to IE8.js too. Note, 2.0 HAD code to support this feature, but 
the code was removed for 2.1. I don't know if this patch breaks anything else, 
but my site didn't show anything weird, so I'm using it.

Original comment by mjlie...@gmail.com on 17 Aug 2010 at 4:31

Attachments:

GoogleCodeExporter commented 9 years ago
Dear mjlieske,

Could you please advise me how to use this patch. I'm not very skilled with 
JavaScript.

Many thanks and regards,

Grant Bailey

Original comment by grantmal...@gmail.com on 12 Oct 2010 at 5:40

GoogleCodeExporter commented 9 years ago
Hello,

I managed to work out how to patch the file. It is attached for others to use. 
Thanks to mjlieske and of course, to Dean Edwards.

Regards,

Grant Bailey

Original comment by grantmal...@gmail.com on 14 Oct 2010 at 8:15

Attachments:

GoogleCodeExporter commented 9 years ago
I can confirm the patched version from comment #4 works. Thank you!

Original comment by Mihaela....@gmail.com on 9 Feb 2012 at 3:36