![]() |
#31 |
Grand Sorcerer
![]() ![]() ![]() ![]() ![]() ![]() ![]() ![]() ![]() ![]() ![]() Posts: 28,579
Karma: 204127028
Join Date: Jan 2010
Device: Nexus 7, Kindle Fire HD
|
And in Code View, I would expect "sigil_split_marker" to appear to be just as light/dark as any other html attribute value. They're all set by the same preference.
|
![]() |
![]() |
![]() |
#32 |
Zealot
![]() Posts: 110
Karma: 10
Join Date: Aug 2018
Location: New Jersey
Device: Samsung Tab E nook
|
Thank you,
A little tweaking, and my new borders look great. Brent |
![]() |
![]() |
Advert | |
|
![]() |
#33 |
Running with scissors
![]() ![]() ![]() ![]() ![]() ![]() ![]() ![]() ![]() ![]() ![]() Posts: 1,587
Karma: 14328510
Join Date: Nov 2019
Device: none
|
This new version is blazing fast compared to the previous one. My computer is a beast; 16 cores/32 threads, 64 gig of ram, and the previous version ran like molasses.
|
![]() |
![]() |
![]() |
#34 | ||
Wizard
![]() ![]() ![]() ![]() ![]() ![]() ![]() ![]() ![]() ![]() ![]() Posts: 2,306
Karma: 13057279
Join Date: Jul 2012
Device: Kobo Forma, Nook
|
Quote:
![]() ![]() But there have been a few regressions introduced along the way too. One Spellcheck List issue was already adjusted for next release—1.6.1: - "In spellcheck do FindSelectedWord only if double-clicked to speed paging" * * * The Issue When used on a very large EPUB (~1.6 million words), Tools > Spellcheck > Spellcheck was sluggish, with noticeable delays between clicks/highlights. These were the timings in my VM: - Sigil 1.6.0: ~1.4 seconds - Sigil 1.5.1: ~1.4 seconds - Sigil 0.9.14: <0.5 seconds - Calibre 5.19: Instant Note: In your typical-sized ebook, these steps barely take a perceivable amount of time. VM Note: Windows 10 + 2 cores + on an external hard drive. * * * If you need an enormous EPUB (~1.3 million words) to poke around in, there's the same one I mentioned in my "monolithic single file" report: https://mises.org/library/complete-l...orum-1969-1984 If you want it even bigger, just EPUB Merge it with another large ebook. ![]() The Steps While in Spellcheck List (Tools > Spellcheck > Spellcheck)... Sigil pre-1.6.1 1. Single-click a word in the list. It jumps to the word within Code View. 2. Press Page Down/Up OR single-click any word. (#.## seconds delay to find/jump within Code View.) Page Down multiple times, and the delay hits every single time as Sigil searches through Code View. Sigil 1.6.1+ To mitigate some of this, we settled on: 1. Double-click a word jumps to the word within Code View. 2. Press Page Down/Up OR single-click another word. No jump in Code View. This allows you to easily make all your adjustments directly in Spellcheck List window (sort columns, Change Selected Word To, Add To Dictionary, [...]) without the need to mess with the code. Side Note: This double-click jump + single-click stay is the way Calibre's Spellcheck List already works. * * * Note: KevinH tested this and couldn't reproduce. No/barely any slowness. Also seems like it would be a huge amount of overhaul for fractional gains. Here was his explanation to me via PM: Quote:
... Although at this point, we're niggling over 1 second... Plus, now with Sigil 1.6.1's already-pushed fix, this 1 second will be buried out of the way. ![]() Last edited by Tex2002ans; 06-05-2021 at 02:34 AM. |
||
![]() |
![]() |
![]() |
#35 |
Sigil Developer
![]() ![]() ![]() ![]() ![]() ![]() ![]() ![]() ![]() ![]() ![]() Posts: 8,778
Karma: 6000000
Join Date: Nov 2009
Device: many
|
FYI, That fix will probably appear in a 1.7.0 release after a few new features have been added. As for timings, 0.9.14 only searches for a word, not the word and language, so it will be much faster as it does not require any code parsing at all. Sigil, unlike calibre (unless Kovid changed it recently) only finds that word in the specified language so if a common misspelling exists in multiple languages it will just find the one matching the specific language being searched for. So the timings are actually not comparable since they do different things.
Once 1.7.0 is released in a few months (or if people build their own from master) if others see the same timing issues with your test case (that I can not reproduce under macOS) we can revisit things to see if something platform specific is at issue. Last edited by KevinH; 06-05-2021 at 12:53 PM. |
![]() |
![]() |
Advert | |
|
![]() |
#36 | |
Guru
![]() ![]() ![]() ![]() ![]() ![]() ![]() ![]() ![]() ![]() ![]() Posts: 842
Karma: 3335974
Join Date: Jan 2017
Location: Poland
Device: Various
|
I would like to comment on the problem reported by @Turtle91 here.
Quote:
However, last week I finally bought new equipment (i7-11700, 32GB) and I still have the mentioned problem (Windows 10 Pro). This is exactly step-by-step: * I'm opening a new EPUB file. * I search for a word (the XHTML file with the first finding is opened in a new tab). * In the Preview window I can see the beginning of the file, although the found word is highlighted in Code View in the middle of the file, far from the beginning of the file. * If I search for the next word and it appears in the same XHTML file, the Preview window will show the correct place, but if it requires opening a different XHTML file in a new tab, the problem will appear again. (This is the same situation described by @Turtle91.) Perhaps this is Windows-only related? |
|
![]() |
![]() |
![]() |
#37 |
Sigil Developer
![]() ![]() ![]() ![]() ![]() ![]() ![]() ![]() ![]() ![]() ![]() Posts: 8,778
Karma: 6000000
Join Date: Nov 2009
Device: many
|
That should be fixed in master. Please try opening a CV tab, clicking on a word to sync to Preview, next open any css tab, then switch back to the html tab you had opened. You should see the word you clicked on and Preview should have properly synced.
That bug (also reported by Turtle91) fix seemed to fix Preview syncing to CV either via click or via sync to search. At least it does for macOS. Can you reproduce that part of his bug report in current master as well? Last edited by KevinH; 06-05-2021 at 06:21 PM. |
![]() |
![]() |
![]() |
#38 | |
Guru
![]() ![]() ![]() ![]() ![]() ![]() ![]() ![]() ![]() ![]() ![]() Posts: 842
Karma: 3335974
Join Date: Jan 2017
Location: Poland
Device: Various
|
Quote:
Especially just a moment ago, I made a fresh build from the master with the latest commits. Issue exists. EDIT: After clicking in the Code View window, it works OK. Only searching (no clicking) does not work. Of course, I can live with it if you confirm that this action is intentional. This is strange because - as mentioned above - Find Next in the same XHTML file works fine and I don't need to click anything. Last edited by BeckyEbook; 06-05-2021 at 06:24 PM. |
|
![]() |
![]() |
![]() |
#39 |
Sigil Developer
![]() ![]() ![]() ![]() ![]() ![]() ![]() ![]() ![]() ![]() ![]() Posts: 8,778
Karma: 6000000
Join Date: Nov 2009
Device: many
|
I will try again to reproduce what you are seeing on macOS. But the earlier fix should correct both as it involves opening a new tab resource and having Preview sync to CodeView location in that new tab.
I just want to make sure your master build has that fix for the xhtml->css->xhtml tab sequence sync to Preview fix working. |
![]() |
![]() |
![]() |
#40 |
Sigil Developer
![]() ![]() ![]() ![]() ![]() ![]() ![]() ![]() ![]() ![]() ![]() Posts: 8,778
Karma: 6000000
Join Date: Nov 2009
Device: many
|
It is not intentional. The problem is that loading PV from CV and telling it to go to a specific location was working only 60% of the time due to timing issues with sequences of Qt signals.
I thought it was fixed, but I may be wrong. |
![]() |
![]() |
![]() |
#41 |
Sigil Developer
![]() ![]() ![]() ![]() ![]() ![]() ![]() ![]() ![]() ![]() ![]() Posts: 8,778
Karma: 6000000
Join Date: Nov 2009
Device: many
|
Just to help track this down, did this used to work in 1.5.1, 1.4.3 or 1.3.0?
|
![]() |
![]() |
![]() |
#42 |
Guru
![]() ![]() ![]() ![]() ![]() ![]() ![]() ![]() ![]() ![]() ![]() Posts: 842
Karma: 3335974
Join Date: Jan 2017
Location: Poland
Device: Various
|
Yes. I confirm, that xhtml->css->xhtml fix working.
|
![]() |
![]() |
![]() |
#43 |
Guru
![]() ![]() ![]() ![]() ![]() ![]() ![]() ![]() ![]() ![]() ![]() Posts: 842
Karma: 3335974
Join Date: Jan 2017
Location: Poland
Device: Various
|
|
![]() |
![]() |
![]() |
#44 |
Guru
![]() ![]() ![]() ![]() ![]() ![]() ![]() ![]() ![]() ![]() ![]() Posts: 842
Karma: 3335974
Join Date: Jan 2017
Location: Poland
Device: Various
|
1.5.1 – works.
1.4.3 – works. 1.3.0 – works. |
![]() |
![]() |
![]() |
#45 |
Sigil Developer
![]() ![]() ![]() ![]() ![]() ![]() ![]() ![]() ![]() ![]() ![]() Posts: 8,778
Karma: 6000000
Join Date: Nov 2009
Device: many
|
So, the async Preview Loading we just added seems to be the issue here.
From reading the code, all CV searches that find something should cause PV to sync, but CV is loading before the asynchronous Preview loading is complete and the scroll to the search location is getting ignored since PV is not fully loaded yet. I will have to think about how to handle this since we have moved to asynchronous loading of PV. |
![]() |
![]() |
![]() |
|
![]() |
||||
Thread | Thread Starter | Forum | Replies | Last Post |
Sigil-0.9.5 Released | KevinH | Sigil | 68 | 04-09-2016 06:30 AM |
Sigil-0.9.4 Released | KevinH | Sigil | 40 | 03-26-2016 10:15 AM |
Sigil-0.9.3 Released | KevinH | Sigil | 56 | 03-03-2016 06:46 PM |
Sigil-0.9.1 Released | KevinH | Sigil | 36 | 12-04-2015 03:00 PM |
Sigil-0.8.900 released for testing - Wait for Sigil-0.8.901 | KevinH | Sigil | 106 | 10-04-2015 10:41 AM |