Sigil-0.9.991 (1.0 Pre-release) - Page 16

KevinH · 11-18-2019, 03:21 PM

I am building the Table on the fly and can not check if the results are duplicates unless I do it in two passes. Even then, it is possible to rename a resource to something that clashes with a resource that is not in the set to be renamed. This Dialog is actually independent of the book being worked on and I would like to keep it that way.

The follow-on code will detect and prevent rename collisions.

KevinH

Quote:

Originally Posted by BeckyEbook

The first tests are very promising.
Strength in simplicity!

Only one thing - after pressing "Cancel" in the prior-revised window could return to the window with the formula entered.

Escape works for me with one backslash.

A suggestion for additional avoidance of duplicates (if someone changes all files): If the name in the "Revised Filename" column is repeated - color it in red.

KevinH · 11-18-2019, 04:41 PM

Okay I just pushed retrying up to 4 times after the second Dialog is rejected, giving the user a chance to alter the existing regex.

BeckyEbook · 11-18-2019, 05:40 PM

Thank you. For me it is a very useful.
It allows me to add prefixes and suffixes and to quickly modify repetitive elements in file names.

I have the impression that the RETable window should be slightly wider (e.g. 795 to fit on 800x600px monitors).
Then the table would have a width of 735, and one section about 362, which will also fit longer names.
Of course, this is not a priority. It is important that the function works.

Ebook to try: Kim

KevinH · 11-19-2019, 10:22 AM

I have made the new RETable widget resizable. Its columns will autosize themselves to fit whatever length is given. You should be able to manually grow the dialog to fit the required column space. You can also drag the column divider to change the allocation of space between the two columns.

I have also grown the default widget size for this table. But just like any other widget in Sigil, it remembers its own geometry so changing that size will only impact first time ever users.

Hope this helps.

KevinH

Quote:

Originally Posted by BeckyEbook

Thank you. For me it is a very useful.
It allows me to add prefixes and suffixes and to quickly modify repetitive elements in file names.

I have the impression that the RETable window should be slightly wider (e.g. 795 to fit on 800x600px monitors).
Then the table would have a width of 735, and one section about 362, which will also fit longer names.
Of course, this is not a priority. It is important that the function works.

Ebook to try: Kim

DNSB · 11-19-2019, 04:46 PM

Are there any plans to make some of the intermediate builds on the way to the 1.0 release publicly accessible? Not a build of the day but updating the version on the Github page weekly or biweekly would be useful (to me, at least).

KevinH · 11-19-2019, 05:37 PM

DiapDealer might be able to help with our automated builds. Please pm him.

snarkophilus · 12-02-2019, 07:56 AM

I've got an epub that Sigil 0.9.991 build 87 spits out the following error when opening:

Code:

The OPF file did not identify the NCX file correctly.
 - Sigil has used the following file as the NCX: Robi_9780553898279_epub_ncx_r1.ncx

If I try to go to a bookmark I get the following error in a popup:

Code:

The file "OEBPS/OEBPS/Robi_9780553898279_epub_c01_r1.htm" does not exist.

There's also a couple of layout glitches when this epub is open. There's a grey gap between the main menu bar and the toolbar icons (see first attached screenshot) and if I opem the validation results window it's only on the right hand half of the main window instead of the whole width (see second attached screenshot).

Other than being confused by bookmarks and the funny layout glitches, Sigil appears to work otherwise properly with this epub.

The structure of the epub zip file looks like this:

Code:

mimetype
META-INF/calibre_bookmarks.txt
META-INF/container.xml
OEBPS/Robi_9780553898279_epub_c01_r1.htm
 ...
OEBPS/Robi_9780553898279_epub_css_r1.css
 ...
OEBPS/images/Robi_9780553898279_epub_001_r1.jpg
 ...
OEBPS/page-template.xpgt
Robi_9780553898279_epub_ncx_r1.ncx
Robi_9780553898279_epub_opf_r1.opf
...

and the only reference to the NCX in the OPF file is:

Code:

  ...
        <manifest>
                <item href="Robi_9780553898279_epub_ncx_r1.ncx" id="ncx" media-type="application/x-dtbncx+xml"/>
                <item href="OEBPS/Robi_9780553898279_epub_cvi_r1.htm" id="cvi" media-type="application/xhtml+xml"/>
  ...

On my old laptop I still have Sigil 0.9.16 and that give the same error on opening but otherwise appears works as expected, including bookmarks that are usable. My Kobo Aura 2 seems to display the epub without any problems. Calibre's built in editor and viewer are happy to edit and view the file too, although running a check in the editor gives the following:

Code:

Robi_9780553898279_epub_opf_r1.opf (line: 109)
The tag has no reference to the NCX table of contents file. Without this reference, the table of contents will not work in most readers. The reference should look like .

KevinH · 12-02-2019, 08:38 AM

Given both 0.9.16 and 0.9.991 and calibre all are saying the same thing, your opf is broken and does not identify the ncx appropriately. Note the ncx id should also be included on the spine attribute.

Please pm me a link with a list of all of the files in the epub (run the equivalent of an unzip -t on it) and a copy of both the ncx and the opf file (*before*) they are loaded into Sigil.

And I will verify if there is an issue.

There are many causes for graphical glitches and so I would not worry about the alpha at all.

KevinH

snarkophilus · 12-02-2019, 09:01 AM

Quote:

Originally Posted by KevinH

Please pm me a link ...

And I will verify if there is an issue.

Done, you should have a PM.

Quote:

There are many causes for graphical glitches and so I would not worry about the alpha at all.

Fair enough. The glitches appear to be repeatable for this epub, and this is the only time I've seen any sort of display glitches.

KevinH · 12-02-2019, 10:11 AM

Yes your OPF file does not meet epub2 spec:
(http://idpf.org/epub/20/spec/OPF_2.0...htm#Section2.4)

The manifest appears to be fine as it lists the exact NCX file with the proper href and proper media type.

The problem is according to the epub2 spec, the spine must have a toc attribute that must point to the manifest id of the NCX as the NCX is required in epub2.

The manifest id on your ncx file is:

id="ncx"

So open the epub in Calibre or Sigil and open the content.opf tab. If it has not already been fixed for you ...

Then change spine tag as from:

Code:

<spine>

to

Code:

<spine toc="ncx">

And then save it. You should no longer get that warning or error message.

KevinH

KevinH · 12-02-2019, 02:16 PM

@snarkophilus,
BTW BeckyEBook was able to recreate your graphics glitch and posted a test case for me. If it can be recreated it is not just a glitch. This is not something I am seeing on macOS but I did try delaying the load warning message display until after the initial restoreState is done. So hopefully this "glitch" is now fixed in master.

KevinH

snarkophilus · 12-02-2019, 04:32 PM

Quote:

Originally Posted by KevinH

@snarkophilus,
BTW BeckyEBook was able to recreate your graphics glitch and posted a test case for me. [...] So hopefully this "glitch" is now fixed in master.

KevinH

Yep, confirmed on my dud epub that the layout is normal now. Thanks!

snarkophilus · 12-02-2019, 04:56 PM

Quote:

Originally Posted by KevinH

Yes your OPF file does not meet epub2 spec:
(http://idpf.org/epub/20/spec/OPF_2.0...htm#Section2.4)

No suprise there

Quote:

So open the epub in Calibre or Sigil and open the content.opf tab. If it has not already been fixed for you ...

Then change spine tag as from:

Code:

<spine>

to

Code:

<spine toc="ncx">

And then save it. You should no longer get that warning or error message.

"Hmmm." The OPF definitely doesn't have toc="ncx" before I start:

Code:

NUC 5667> unzip -p *.epub | grep -a '<spine'
        <spine>
NUC 5668>

but when I open the epub in Sigil, I get the same popup about but opening the OPF shows that (I assume) Sigil has changed the spine entry automatically to

Code:

  <spine toc="ncx">

If I try a bookmark I get the same error as before:

Code:

The file "OEBPS/OEBPS/Robi_9780553898279_epub_c01_r1.htm" does not exist.

Saving and re-opening the saved epub then indeed doesn't show an error, but the Sigil still thinks the bookmark references still all start with "OEBPS/OEBPS/..."

I'll send you a link to a scrambled version of the epub that shows the problem.

KevinH · 12-02-2019, 07:14 PM

What bookmark are you talking about? Is this the calibre bookmarks file inside the META-INF folder?

If so, Sigil should not touch it. I have no idea how it links or points to a file or tag.

snarkophilus · 12-02-2019, 11:13 PM

Quote:

Originally Posted by KevinH

What bookmark are you talking about?

Sorry, I meant TOC entry not bookmark.

Clicking on any entry in the TOC pane gets the error popup about a filename starting with "OEBPS/OEBPS/..." does not exist.

12-02-2019, 07:56 AM	#232
snarkophilus Wannabe Connoisseur Posts: 426 Karma: 2516674 Join Date: Apr 2011 Location: Geelong, Australia Device: Kobo Libra 2, Kobo Aura 2, Sony PRS-T1, Sony PRS-350, Palm TX	I've got an epub that Sigil 0.9.991 build 87 spits out the following error when opening: Code: The OPF file did not identify the NCX file correctly. - Sigil has used the following file as the NCX: Robi_9780553898279_epub_ncx_r1.ncx If I try to go to a bookmark I get the following error in a popup: Code: The file "OEBPS/OEBPS/Robi_9780553898279_epub_c01_r1.htm" does not exist. There's also a couple of layout glitches when this epub is open. There's a grey gap between the main menu bar and the toolbar icons (see first attached screenshot) and if I opem the validation results window it's only on the right hand half of the main window instead of the whole width (see second attached screenshot). Other than being confused by bookmarks and the funny layout glitches, Sigil appears to work otherwise properly with this epub. The structure of the epub zip file looks like this: Code: mimetype META-INF/calibre_bookmarks.txt META-INF/container.xml OEBPS/Robi_9780553898279_epub_c01_r1.htm ... OEBPS/Robi_9780553898279_epub_css_r1.css ... OEBPS/images/Robi_9780553898279_epub_001_r1.jpg ... OEBPS/page-template.xpgt Robi_9780553898279_epub_ncx_r1.ncx Robi_9780553898279_epub_opf_r1.opf ... and the only reference to the NCX in the OPF file is: Code: ... <manifest> <item href="Robi_9780553898279_epub_ncx_r1.ncx" id="ncx" media-type="application/x-dtbncx+xml"/> <item href="OEBPS/Robi_9780553898279_epub_cvi_r1.htm" id="cvi" media-type="application/xhtml+xml"/> ... On my old laptop I still have Sigil 0.9.16 and that give the same error on opening but otherwise appears works as expected, including bookmarks that are usable. My Kobo Aura 2 seems to display the epub without any problems. Calibre's built in editor and viewer are happy to edit and view the file too, although running a check in the editor gives the following: Code: Robi_9780553898279_epub_opf_r1.opf (line: 109) The tag has no reference to the NCX table of contents file. Without this reference, the table of contents will not work in most readers. The reference should look like . Attached Thumbnails

12-02-2019, 10:11 AM	#235
KevinH Sigil Developer Posts: 8,849 Karma: 6120478 Join Date: Nov 2009 Device: many	Yes your OPF file does not meet epub2 spec: (http://idpf.org/epub/20/spec/OPF_2.0...htm#Section2.4) The manifest appears to be fine as it lists the exact NCX file with the proper href and proper media type. The problem is according to the epub2 spec, the spine must have a toc attribute that must point to the manifest id of the NCX as the NCX is required in epub2. The manifest id on your ncx file is: id="ncx" So open the epub in Calibre or Sigil and open the content.opf tab. If it has not already been fixed for you ... Then change spine tag as from: Code: <spine> to Code: <spine toc="ncx"> And then save it. You should no longer get that warning or error message. KevinH Last edited by KevinH; 12-02-2019 at 10:14 AM.

Similar Threads
Thread	Thread Starter	Forum	Replies	Last Post
5.8.11 Pre=release	knc1	Kindle Developer's Corner	21	04-17-2018 08:42 PM
Overdrive - Search for Pre-Release	nynaevelan	General Discussions	11	11-24-2013 02:27 PM
Where are the pre-release purchase buttons?	Sydney's Mom	General Discussions	8	09-06-2012 10:57 PM
KF Android 4.0 Pre-Release Version	robertc88	Kindle Fire	22	01-22-2012 07:24 PM
PDF Viewer 0.3.0 pre-release	pruss	Android Devices	62	11-22-2011 11:18 AM

11-18-2019, 04:41 PM	#227
KevinH Sigil Developer Posts: 8,849 Karma: 6120478 Join Date: Nov 2009 Device: many	Okay I just pushed retrying up to 4 times after the second Dialog is rejected, giving the user a chance to alter the existing regex.

11-18-2019, 05:40 PM	#228
BeckyEbook Guru Posts: 850 Karma: 3341026 Join Date: Jan 2017 Location: Poland Device: Various	Thank you. For me it is a very useful. It allows me to add prefixes and suffixes and to quickly modify repetitive elements in file names. I have the impression that the RETable window should be slightly wider (e.g. 795 to fit on 800x600px monitors). Then the table would have a width of 735, and one section about 362, which will also fit longer names. Of course, this is not a priority. It is important that the function works. Ebook to try: Kim

11-19-2019, 04:46 PM	#230
DNSB Bibliophagist Posts: 46,648 Karma: 169712392 Join Date: Jul 2010 Location: Vancouver Device: Kobo Sage, Libra Colour, Lenovo M8 FHD, Paperwhite 4, Tolino epos	Are there any plans to make some of the intermediate builds on the way to the 1.0 release publicly accessible? Not a build of the day but updating the version on the Github page weekly or biweekly would be useful (to me, at least).

11-19-2019, 05:37 PM	#231
KevinH Sigil Developer Posts: 8,849 Karma: 6120478 Join Date: Nov 2009 Device: many	DiapDealer might be able to help with our automated builds. Please pm him.

12-02-2019, 08:38 AM	#233
KevinH Sigil Developer Posts: 8,849 Karma: 6120478 Join Date: Nov 2009 Device: many	Given both 0.9.16 and 0.9.991 and calibre all are saying the same thing, your opf is broken and does not identify the ncx appropriately. Note the ncx id should also be included on the spine attribute. Please pm me a link with a list of all of the files in the epub (run the equivalent of an unzip -t on it) and a copy of both the ncx and the opf file (before) they are loaded into Sigil. And I will verify if there is an issue. There are many causes for graphical glitches and so I would not worry about the alpha at all. KevinH

12-02-2019, 02:16 PM	#236
KevinH Sigil Developer Posts: 8,849 Karma: 6120478 Join Date: Nov 2009 Device: many	@snarkophilus, BTW BeckyEBook was able to recreate your graphics glitch and posted a test case for me. If it can be recreated it is not just a glitch. This is not something I am seeing on macOS but I did try delaying the load warning message display until after the initial restoreState is done. So hopefully this "glitch" is now fixed in master. KevinH

12-02-2019, 07:14 PM	#239
KevinH Sigil Developer Posts: 8,849 Karma: 6120478 Join Date: Nov 2009 Device: many	What bookmark are you talking about? Is this the calibre bookmarks file inside the META-INF folder? If so, Sigil should not touch it. I have no idea how it links or points to a file or tag.

Advert

Advert