Opened 15 years ago
Closed 12 years ago
#5901 closed Bug (invalid)
Firefox 3.6.4 Breaking background-image:url('foo/bar.png'); with "
Reported by: | wolfehr | Owned by: | |
---|---|---|---|
Priority: | Normal | Milestone: | |
Component: | Core : Output Data | Version: | 3.3.1 |
Keywords: | Firefox | Cc: |
Description
Tested on Firefox 3.6.4 - CKEditor Version 3.3.1
Code = <div id="right_top" style="width: 308px; height: 13px; background-image: url('/images/right_top.png');">Test</div>
This is converted after switching source view off and on again to the following:
Amended = <div id="right_top" style="width: 308px; height: 13px; background-image: url("/images/right_top.png");">
Test</div>
If this is then submitted, the html is completely garbled by the editor and becomes the folloing:
Final output = <div id="right_top" images="" right_top.png="" style="width: 308px; height: 13px;">
Test</div>
Inline styling has been fine up until now and this has only happened since this new version of firefox.
Change History (4)
comment:1 Changed 15 years ago by
Keywords: | Firefox Confirmed added; " firefox 3.6.4 quote problem firefox garbling background-image removed |
---|---|
Priority: | High → Normal |
comment:2 Changed 15 years ago by
Component: | General → Core : Output Data |
---|---|
Keywords: | Pending added; Confirmed removed |
There are some things to be considered:
- It looks like Firefox transforms the single quote char (') into a quote char (") when parsing the HTML. So, that's what the editor gets from it.
- It *should* be ok to have " there. It *should* not break things, other than being the right way to have quote chars inside elements' attributes.
- The data you see in the source view is *exactly* the same you get from the editor output. In fact we have the same procedure being used for both cases. So, it's impossible to have a different output.
Regarding the last point, I think there is something else handling the data. I'm not able to reproduce the problem in any way.
Can you investigate it a bit further, giving us more details on how to reproduce the problem? A sample page could help a lot.
comment:3 Changed 15 years ago by
I believe this issue is to do with implementation where mysql_real_escape_string is being used on the content. This is what is going through when submitting the above:
Data Submitted Through Editor - <div id="right_top" style="width: 308px; height: 13px; background-image: url("/images/right_top.png");">Test</div>
After processing WITH mysql_real_escape_string - <div id="\"right_top\"" style="" 308px;="" height:="" 13px;="" background-image:="" url("="" images="" right_top.png");\="">Test</div> </div>
After processing WITHOUT mysql_real_escape_string - <div id="right_top" style="width: 308px; height: 13px; background-image: url("/images/right_top.png");">Test</div>
comment:4 Changed 12 years ago by
Resolution: | → invalid |
---|---|
Status: | pending → closed |
Sorry but this is not CKEditor issue.
I can confirm it on FF 3.6.3 also. This is not CKEditor regression.