Opened 17 years ago
Closed 17 years ago
#1319 closed Bug (fixed)
fckw3crange.html : test_toString_W3C_1 fails
Reported by: | Hallvord R. M. Steen (Opera Software) | Owned by: | Frederico Caldeira Knabben |
---|---|---|---|
Priority: | Normal | Milestone: | FCKeditor 2.5 Beta |
Component: | General | Version: | SVN (FCKeditor) - Retired |
Keywords: | Cc: |
Description
Run automated tests from FCKEditor SVN.
Note error:
"trunk/_test/automated/tests/fckw3crange.html:test_toString_W3C_1 failed" Expected <his is some> (String) but was <his is some text.> (String)
Opera and Firefox both "fail" this test, it is probably broken.
Change History (3)
comment:1 Changed 17 years ago by
Owner: | set to Frederico Caldeira Knabben |
---|
comment:2 Changed 17 years ago by
comment:3 Changed 17 years ago by
Milestone: | → FCKeditor 2.5 |
---|---|
Resolution: | → fixed |
Status: | new → closed |
I've dropped FCKDomRange.GetText and FCKW3CRange.toString with [978] as we are not using them at this moment.
This is most likely caused by [806]. I changed the function to make it non-intrusive (i.e. not changing the document structure) when I thought I needed it in the indent commands. However, after writing the indent command logic, I figured out a way to do it without using FCKW3CRange.toString() or the related FCKDomRange.GetText().
So the new toString() code isn't needed by anything right now. Therefore, if the non-intrusiveness isn't considered important, this might be fixed by reverting to the code before [806].