search

Rainie3535 / sigil

Automatically exported from code.google.com/p/sigil
GNU General Public License v3.0
0 stars 0 forks source link

F&R highlights extra text for some searches #1223

Closed GoogleCodeExporter closed 9 years ago

GoogleCodeExporter commented 9 years ago 
While testing F&R 2 highlighting issues that can be reproduced.

Ubuntu 11.10
Qt 4.7.2
Built from current source+0.5.0

1.
Find highlights extra text in Book View sometimes as seen in the attached 
screenshot.  I put the cursor between one and "one two".  Had to try a few 
times in different places, but it happens repeatedly 
Settings: Book View, F&R: Regex, Current File, Find: one.*two
Open a new file
Type one, return, one two, return
Put cursor end of the first one
Click Replace
Highlight is odd - stretches over multiple lines

Now, if you save the file and load it as in the attached frhighlight1.epub 
file, it highlights ok.  But if you create it and do the search, the highlight 
is wrong.  And this is using the latest code.

If you put a SaveTabContent in FindText then it works ok.  Does ReplaceText 
need one too?

Exists in 0.5.0 and current source.

2.  As in frhighlight2.epub, which you can load
Settings: Book View, F&R: Normal, Current File, Find: one

In Book View put your cursor on the line between two and three, and then do a 
search for one - it highlights the "on" on the first line - even though it 
searching down?

Exists in 0.5.0 and current source.

Original issue reported on code.google.com by meme90...@gmail.com on 1 Feb 2012 at 8:38

Attachments:

GoogleCodeExporter commented 9 years ago 
Ignore the bit about "If you put a SaveTabContent in FindText then it works ok. 
 Does ReplaceText need one too?"

After actually testing this, it results in the text not even being found.   So 
something isn't getting updated from BV.

Original comment by meme90...@gmail.com on 1 Feb 2012 at 8:55

GoogleCodeExporter commented 9 years ago

Original comment by meme90...@gmail.com on 7 Feb 2012 at 8:48

GoogleCodeExporter commented 9 years ago 
BV will be replaced so closing this issue.

Original comment by meme90...@gmail.com on 17 Feb 2012 at 7:27

GoogleCodeExporter commented 9 years ago

Original comment by daveheil...@gmail.com on 19 May 2012 at 3:20