![]() |
#91 | |
Sigil Developer
![]() ![]() ![]() ![]() ![]() ![]() ![]() ![]() ![]() ![]() ![]() Posts: 7,998
Karma: 5449552
Join Date: Nov 2009
Device: many
|
Quote:
Did Sigil ever work that way in the AddCover? Why would it default to having something already selected. It is the act of clicking that actually makes your choice apparent. I am not 100% on this. |
|
![]() |
![]() |
![]() |
#92 |
Sigil Developer
![]() ![]() ![]() ![]() ![]() ![]() ![]() ![]() ![]() ![]() ![]() Posts: 7,998
Karma: 5449552
Join Date: Nov 2009
Device: many
|
And ditto for the MetaEditor? I will look into why focus is stolen when clicking on the remove button.
Update: I tried Sigil-2.0.2, and Sigil-2.1.0 and in "AddCover" neither one highlights a pick when first opened. So there is no change from Sigil 2.2.0 or Sigil 2.2.1. Did the same for the MetaData Editor and nothing was highlighted there either when first opened. So Sigil-2.0.2, and Sigil-2.1.0, work the same as Sigil 2.2.X in that regard. As for the Remove button not letting focus go back to the MetaEditor Window, I have found and fixed that. If you run into any others. Please let me know. Last edited by KevinH; 07-03-2024 at 06:29 PM. |
![]() |
![]() |
![]() |
#93 | |
Enthusiast
![]() Posts: 31
Karma: 46
Join Date: Mar 2017
Device: None
|
Quote:
You are probably correct, this is nothing new. But, I would like to use the arrow keys to cycle through the list. Consider this: 1. Start Sigil and add in some images that are potential covers, say they are flower.jpg, marmot,jpg and zebra.jpg. 2. Open Add Cover. screenshot1 shows the picture at top of the list (flower) but no highlighting. I want flower as the cover, I know what a flower looks like, and so I click OK and all is well. 3. Well, I don't want flower so I will use down arrow to see the next image. screenshot2 shows nothing has changed except flower.jpg is highlighted on the list. But I don't want flower, why didn't the down arrow key advance the list the next image in the list? 4. I press down arrow again and the image now changes to the next item in the list: marmot. Again, no highlighting, but I know what a marmot looks like. I don't want that image either so again with down arrow. There is no effect and so the down arrow key is now useless: I can't cycle through the list with down arrow key nor the up arrow key. Here's my opinion: 1. I want to be able to cycle through the list with the up/down arrow keys but I can't, I have to use multiple mouse clicks to cycle through the list. 2. There should always be highlighting that corresponds to the visible image. Highlighting is positive confirmation that the visible image and the filename are paired. If you know names and not pictures then using the mouse (or shortcut) is the only way to go. |
|
![]() |
![]() |
![]() |
#94 |
Sigil Developer
![]() ![]() ![]() ![]() ![]() ![]() ![]() ![]() ![]() ![]() ![]() Posts: 7,998
Karma: 5449552
Join Date: Nov 2009
Device: many
|
Technically, all QTreeViews (which is what AddCover uses), should automatically use keyboard arrow keys for navigation. Somehow the timing of loading the WebEngine to show the detailed image is messing it up. I will look why that is happening.
|
![]() |
![]() |
![]() |
#95 |
Sigil Developer
![]() ![]() ![]() ![]() ![]() ![]() ![]() ![]() ![]() ![]() ![]() Posts: 7,998
Karma: 5449552
Join Date: Nov 2009
Device: many
|
@mrprobert,
Okay, It seems that when the QWebEngineView (the big image in the left side) finishes being updated it steals focus to itself and does not return it. This is a bug in Qt. I have had to extensively re-engineer the SelectFiles.h/cpp/ui files using my own focusWidget tracking and QTimers to prevent interrupting the previous load of the QWebEngineView with a new one. All of this now seems to work. I can use Arrow keys to now happily walk down the list of cover images. FWIW - Instead of using WebEngineView to show the detailed image of the cover you can instead use the splitter to hide the WebEngineView and use the + and - keys to set the size of inline thumbnail images to the immediate right of the file name. Your change request is actually a "bug report" as far as I am concerned and its fix will appear in the next release of Sigil. Thank you. Last edited by KevinH; 07-05-2024 at 07:59 AM. |
![]() |
![]() |
![]() |
#96 | |
Sigil Developer
![]() ![]() ![]() ![]() ![]() ![]() ![]() ![]() ![]() ![]() ![]() Posts: 7,998
Karma: 5449552
Join Date: Nov 2009
Device: many
|
Heads Up - still not enough missing metadata-refines for media
Quote:
For those interested, I found a few epub3s that use media overlays and this opf_id_updater v2 still does not properly update the refines used for media duration information in the opf: Support for things like the following: <meta property="media:duration" refines="#chapter_001_overlay">0:14:20.500</meta> where the refines value (after removing the leading #) refers to the manifest id of an smil file are not properly updated. So there will be a v3 version of opf_id_updater coming up probably this weekend with this deficiency fixed. |
|
![]() |
![]() |
![]() |
#97 |
Sigil Developer
![]() ![]() ![]() ![]() ![]() ![]() ![]() ![]() ![]() ![]() ![]() Posts: 7,998
Karma: 5449552
Join Date: Nov 2009
Device: many
|
New Version: opf_id_updater_v3
For those interested.
I have now updated the opf_id_updater to v3. It now will look up meta property refines and update any that point into manifest ids which are common when media overlays are used (smil) files. I included a new test case package.opf which is the opf from the official epub3 media-overlay example file Moby-Dick. I used it to convert the package.opf to npackage.opf As far as I know, this v3 version should be the *final* version. So barring unforeseen issues, I will either include this as a new Sigil plugin or possibly as a internal Sigil python3lib internal tool. I have not decided if this is needed/useful to a broad enough audience to include internally in Sigil for everyone. To run the program after unzipping opf_id_updater_v3: cd into that folder and run: python3 ./fix_opf_ids.py ./package.opf > newpackage.opf Then you can compare your newpackage.opf to the included npackage.opf. Feedback and comments welcome. Especially whether you think this tool would be useful to enough people to actually include inside Sigil under the Tools menu or if it would be better as a standalone Sigil plugin interested users can choose to download and install. Hope this helps. |
![]() |
![]() |
![]() |
#98 | |
Evangelist
![]() ![]() ![]() ![]() ![]() ![]() ![]() ![]() ![]() ![]() ![]() Posts: 435
Karma: 7446396
Join Date: Aug 2013
Location: Hamden, CT
Device: Kindle Paperwhite (11th gen), Scribe
|
Quote:
Code:
A.Long.Filename.jpeg A Long Filename.jpeg |
|
![]() |
![]() |
![]() |
#99 |
Sigil Developer
![]() ![]() ![]() ![]() ![]() ![]() ![]() ![]() ![]() ![]() ![]() Posts: 7,998
Karma: 5449552
Join Date: Nov 2009
Device: many
|
The only period being replaced is at the beginning of the file extension. Also each potential new id is always checked for uniqueness and made unique before it is used.
Note the filenames themselves do not have to be unique at all as long as their epub relative paths from root are different. So uniqueness does not rely on just the filename. Do you have a test case opf that illustrates an issue? If so please post it for me to recreate the bug. Or is this only in the abstract case? If the latter, no worries. Last edited by KevinH; 07-06-2024 at 11:18 AM. |
![]() |
![]() |
![]() |
#100 |
Sigil Developer
![]() ![]() ![]() ![]() ![]() ![]() ![]() ![]() ![]() ![]() ![]() Posts: 7,998
Karma: 5449552
Join Date: Nov 2009
Device: many
|
For those following along, I have slightly modified opf_id_update_v3.zip (see earlier post) and incorporated it into Sigil via its embedded python interface to create a new Epub Tool menu routine:
Rebase OPF Manifest IDs using Current Filenames It will take the current OPF and pass it through that python opf_update_ids_v3 code to rebase all of the current manifest IDs based on its current filename. It will properly update the OPF: - metadata cover-image - any meta refines that point to something in the manifest - the spine toc attribute - the spine itemrefs - the manifest media-overlay and fallback properties - the bindings handler if any exist. This new tool will be part of our next release. |
![]() |
![]() |
![]() |
#101 | |
Enthusiast
![]() Posts: 31
Karma: 46
Join Date: Mar 2017
Device: None
|
Quote:
Whenever I encounter a quirk while using Sigil, my first thought is cockpit error, otherwise I blame Qt. Thank you for the all the work here, it's much appreciated. -- Mark |
|
![]() |
![]() |
![]() |
|
![]() |
||||
Thread | Thread Starter | Forum | Replies | Last Post |
Sigil-1.3.0 Released | DiapDealer | Sigil | 8 | 09-15-2020 08:03 AM |
Sigil-0.9.10 Released | DiapDealer | Sigil | 149 | 11-19-2018 10:20 PM |
Sigil-0.8.900 released for testing - Wait for Sigil-0.8.901 | KevinH | Sigil | 106 | 10-04-2015 10:41 AM |
Sigil 0.8.2 Released | user_none | Sigil | 12 | 12-22-2014 06:02 PM |
Sigil 0.7.0 Released | user_none | Sigil | 75 | 03-03-2013 12:41 PM |