#3176 closed Bug (fixed)
Match the whole word in Find it does not match
Reported by: | Aneta | Owned by: | Garry Yao |
---|---|---|---|
Priority: | Normal | Milestone: | CKEditor 3.0 |
Component: | General | Version: | SVN (CKEditor) - OLD |
Keywords: | Confirmed IBM | Cc: |
Description
- Hit the Find button and type a word from the text.
- Tick only - match the whole word and click on Find button.
Result: A notice pops out - The specified text was not found.
Attachments (2)
Change History (10)
comment:1 Changed 16 years ago by
Keywords: | Pending added |
---|
comment:2 Changed 16 years ago by
Keywords: | Confirmed added; Pending removed |
---|
Ok, it looks like this problem doesn't happen always. I found a way to reproduce it consistently though:
- Load the editor and remove all the text on it.
- Manually type three paragraphs, each one containing one of these words, in this order: "test", "try" and "test".
- Move the caret to the start of the text.
- Open the Find dialog.
- In the "Find what" field, type "try" and hit the "Find" button. The word will be correctly found.
- Hit "Find" again. A js error is thrown.
- Close the dialog.
- Repeat steps 3 to 5. It will not work anymore. Actually, the Find dialog will not work anymore until reloading the editor.
Confirmed with FF3.
Changed 16 years ago by
Attachment: | 3176.patch added |
---|
comment:3 Changed 16 years ago by
Keywords: | Review? added |
---|---|
Owner: | set to Garry Yao |
Status: | new → assigned |
The patch improve the logic of characterRange and domRange parts, it also fix a bug in range.js.
Changed 16 years ago by
Attachment: | 3176_2.patch added |
---|
comment:4 Changed 16 years ago by
Updated Garry's patch with respect to the current trunk for easier reviewing.
comment:5 Changed 16 years ago by
Keywords: | Review- added; Review? removed |
---|
Review- because the patch doesn't seem to fix the problem described in comment:2. I'm still getting the not found problem when match whole word is ticked.
comment:7 Changed 16 years ago by
Resolution: | → fixed |
---|---|
Status: | assigned → closed |
comment:8 Changed 16 years ago by
Keywords: | IBM added |
---|
Are you using a specific browser?