12-16-2011, 12:06 PM | #61 |
Grand Sorcerer
Posts: 27,550
Karma: 193191846
Join Date: Jan 2010
Device: Nexus 7, Kindle Fire HD
|
Is that how the previous Sigil S&R engine worked on OS X, as well then?
Last edited by DiapDealer; 12-16-2011 at 12:21 PM. |
12-16-2011, 12:50 PM | #62 |
Sigil developer
Posts: 1,275
Karma: 1101600
Join Date: Jan 2011
Location: UK
Device: Kindle PW, K4 NT, K3, Kobo Touch
|
I see what you mean. I was just listing what it appeared to be doing for me, not whether this was 'normal'. Testing MS Word shows it works as DiapDealer states.
I can see why it works as it does (I can't remember how it worked before). The button is labelled Replace Next (tooltip in the current source) - so it does exactly what it says, it replaces the next occurrence. Just like Find Next finds the next occurrence. Being able to see what was just replaced is a good thing - you get to verify it really does look ok without having to scroll back to see if its what was expected. But if the 'standard' way of doing Replace is to a) If the highlighted text is not the text we're looking for then jump to and highlight the next occurrence, or b) If the highlighted text is the text we're looking for then replace the current highlighted text and highlight the next occurrence - then I can see this working too since it gives you a chance to see what you will replace. But no chance to see what it changes to (unless you change the b rule to just leave the cursor at the end of the replaced text - though if this is non-standard it might be confusing too). If the 'standard' way is used, then I wonder if Next/Previous should be used. In fact, does anyone use Find Previous, or especially Replace Previous? I'm sure there are cases for it, but I know I generally just do Find forward with wrap-around (perhaps after being warned that the end is reached, should it start over so you don't keep looping forever), and trying to aim the cursor to select the right arrow can be a bit fiddly. I'm going to have to re-install the older Sigil to see how it does it. (in source I see the F&R close down arrow is now a close button - but its jumped to the other side? and looks different than the book browser close button) (I think my next post might have to say something nice instead of a complaint ) |
12-16-2011, 01:30 PM | #63 | |
Sigil & calibre developer
Posts: 2,488
Karma: 1063785
Join Date: Jan 2009
Location: Florida, USA
Device: Nook STR
|
Quote:
I originally had F&R be a dock widget but I ran into sizing issues I couldn't get sorted out. |
|
12-16-2011, 02:39 PM | #64 | |
Sigil developer
Posts: 1,275
Karma: 1101600
Join Date: Jan 2011
Location: UK
Device: Kindle PW, K4 NT, K3, Kobo Touch
|
Quote:
Checking the previous version of Sigil, I find that it behaves as DiapDealer indicated - highlight and wait until Replace is hit. It would probably be good to stay consistent with the old behaviour. I also see the old F&R had a bewildering set of options so the new version is much better. Keeping a previous/next doesn't seem so bad in context |
|
12-16-2011, 03:37 PM | #65 |
Sigil developer
Posts: 1,275
Karma: 1101600
Join Date: Jan 2011
Location: UK
Device: Kindle PW, K4 NT, K3, Kobo Touch
|
Using Compiled Source doing a Count or Replace generates warning messages in the command line window (Ubuntu 11.10) although the text in the window looks ok, e.g.
Warning: QString::arg: Argument missing: "1224 match(es) were found." , 1224 Warning: QString::arg: Argument missing: "The search term was replaced 81 time(s)." , 81 Oh, I'm actually running on Ubuntu 11.10 and I do have Qt 4.7.4 installed according to the cmake output. But my right menu is still odd, so I still have some searching to do to find out what's going on. |
|
Similar Threads | ||||
Thread | Thread Starter | Forum | Replies | Last Post |
902/902 und elektronische Zeitungen | temujin | PocketBook | 15 | 11-09-2011 06:39 AM |
No Avaliable format ? ? ? | Janette55 | Library Management | 5 | 04-16-2011 04:09 PM |
Beta 2.0.3 firmware 902 problems | Roos | PocketBook | 15 | 12-15-2010 10:11 AM |
Beta Beta Test of Major New Features | Starson17 | Calibre | 45 | 05-17-2010 10:55 AM |