![]() |
#61 |
Sigil Developer
![]() ![]() ![]() ![]() ![]() ![]() ![]() ![]() ![]() ![]() ![]() Posts: 7,987
Karma: 5449552
Join Date: Nov 2009
Device: many
|
Indeed. Bulk loading of epubs like that was never meant to be. The older versions are your best bet. The separate warning message boxes were actually interfering with restoring the proper Sigil MainWindow and DockWidgets sizes at times. By catching them all and delaying them until after the MainWindow has been activated it prevented that interference.
So we won't be going back. I might think about a "dismiss all" button being added but not fixing these warnings can result in errors that could cause problems for Sigil. Last edited by KevinH; 08-22-2024 at 01:27 PM. |
![]() |
![]() |
![]() |
#62 |
Enthusiast
![]() Posts: 29
Karma: 10
Join Date: May 2012
Device: Kindle Oasis, Scribe, Paperwhite // Boox Palma
|
Thanks for the quick response! That's what I thought but figured I'd ask just to be sure. I'll switch back to the previous version since it worked out for my use case.
|
![]() |
![]() |
![]() |
#63 |
A Hairy Wizard
![]() ![]() ![]() ![]() ![]() ![]() ![]() ![]() ![]() ![]() ![]() Posts: 3,154
Karma: 18843349
Join Date: Dec 2012
Location: Charleston, SC today
Device: iPhone 15/11/X/6/iPad 1,2,Air & Air Pro/Surface Pro/Kindle PW & Fire
|
Found a little glitch. Granted, it's caused by my screwup, but it might be an easy fix.
The problem: Select all your xhtml files in book browser. Rename, and type in something that would produce a filename error, such as '.html/' (yes, I have less nimble fingers sometimes and my fingers are moving too fast to catch the error before I hit the enter key...) Sigil correctly admonishes me with an error 'A filename cannot contains [sic] the character "/".' BUT, like an old female horse, it continues to admonish me for EVERY file I had selected.... ![]() The possible solution: can you exit the rename loop on the first error instead?? |
![]() |
![]() |
![]() |
#64 |
Sigil Developer
![]() ![]() ![]() ![]() ![]() ![]() ![]() ![]() ![]() ![]() ![]() Posts: 7,987
Karma: 5449552
Join Date: Nov 2009
Device: many
|
Yes the detection of illegal characters in new file names does not come until quite late in the bulk rename process to better accommodate RegexRename which can generate many different filename pieces derived from the original.
I will duplicate the forbidden character check and add it much earlier in the rename template code and abort out of all renames if any part of the "template" has or uses forbidden characters. Thanks for your bug report! Update: I decided it would be best to check the user supplied rename template for forbidden characters and abort the entire rename if there are any issues by adding this routine in RenameSelect() in BookBrowser.cpp: Code:
if (!template_name.isEmpty()) { // Verify the template name does not have any forbidden chars in the name const QString badchars = "<>:\"/\\|?*"; bool has_bad_chars = false; foreach(QChar ch, template_name) { has_bad_chars = has_bad_chars || badchars.contains(ch); } if (has_bad_chars) { Utility::DisplayStdErrorDialog(tr("Filenames can not contain these characters: \"%1\".").arg(badchars)); return; } } Last edited by KevinH; Today at 10:58 AM. |
![]() |
![]() |
![]() |
#65 |
A Hairy Wizard
![]() ![]() ![]() ![]() ![]() ![]() ![]() ![]() ![]() ![]() ![]() Posts: 3,154
Karma: 18843349
Join Date: Dec 2012
Location: Charleston, SC today
Device: iPhone 15/11/X/6/iPad 1,2,Air & Air Pro/Surface Pro/Kindle PW & Fire
|
Awesome! Thank you!!
|
![]() |
![]() |
![]() |
|
![]() |
||||
Thread | Thread Starter | Forum | Replies | Last Post |
Sigil-1.5.1 Released | DiapDealer | Sigil | 27 | 04-15-2021 10:27 AM |
Sigil-1.2.0 Released | DiapDealer | Sigil | 77 | 07-16-2020 12:55 PM |
Sigil-0.9.7 Released | DiapDealer | Sigil | 90 | 11-10-2016 01:30 PM |
Sigil-0.9.6 Released | DiapDealer | Sigil | 69 | 10-29-2016 10:35 AM |
Sigil-0.8.900 released for testing - Wait for Sigil-0.8.901 | KevinH | Sigil | 106 | 10-04-2015 10:41 AM |