Bug: "Go to link or style" not working - Page 2

DiapDealer · Today, 02:04 PM

When is denest being used? Is it just when parsing/rendering the css, or is the css file actually being rewritten?

I'm wondering if "Go to Link or Style" is going to work properly on nested css if it's only being denested when parsing/rendering. I'm sure I'm missing something, but I would think it would be hard for the cursor to be placed properly if that's the case.

ElMiko · Today, 02:33 PM

I... don't understand any of the preceding. I'm going to venture a guess that it means your turbo encabulator was not effectively deploying its sinusoidal depleneration failsafes. But I'm gratified to learn that I wasn't going crazy or that my css had a secret bug that would require my going back and fixing hundreds upon hundreds of ePubs. My soul is at peace.

I assume that "Sigil master" is a dev tool, and that any fixes will be applied to 2.8.x, so I should stick with 2.7.6 for now, yes?

KevinH · Today, 02:48 PM

Sigil's "Go to link or style" will never work if the user has nested css and does not reformat its css to denest it using Sigil beforehand. But nested css is so rare right now that I am okay with that. The idea was if Go To Link Or Style ever fails, then the code should be denested first (given the last bugfix just now in master) that now works!

The CSSDenest code is invoked by the qCSSParser right before parsing the css. The CSSInfo code uses the qCSSParser to parse a css file to find the line number of each selector (not just classes). The Go To Link or Style runs the CSSInfo code (that uses qCSSParser and CSSDeNest to create a list of all selectors and their starting line numbers and positions in the file) to determine which line of the css file to move to.

At some point in the future, Sigil will autorun the CSSDenest process on first load of any css file and save it. It should change absolutely nothing if the CSS stylesheet uses no nesting (but the bug just fixed was that it added newlines). I was not 100% sure of the denest code and decided using it in qCSSParser was enough until nested css became more common and then moving it to the epub import and add existing stage.

Hope that makes things clearer.

DiapDealer · Today, 02:49 PM

Quote:

Originally Posted by ElMiko

I assume that "Sigil master" is a dev tool, and that any fixes will be applied to 2.8.x, so I should stick with 2.7.6 for now, yes?

Yes. Especially if that feature is important to you. I'm guessing that we'll get a chance to fix a few more bugs before a new release. Thanks for the details to help track this one down!

DiapDealer · Today, 02:49 PM

Quote:

Originally Posted by KevinH

Sigil's "Go to link or style" will never work if the user has nested css and does not reformat its css to denest it using Sigil beforehand. But nested css is so rare right now that I am okay with that. The idea was if Go To Link Or Style ever fails, then the code should be denested first (given the last bugfix just now in master) that now works!

The CSSDenest code is invoked by the qCSSParser right before parsing the css. The CSSInfo code uses the qCSSParser to parse a css file to find the line number of each selector (not just classes). The Go To Link or Style runs the CSSInfo code (that uses qCSSParser and CSSDeNest to create a list of all selectors and their starting line numbers and positions in the file) to determine which line of the css file to move to.

At some point in the future, Sigil will autorun the CSSDenest process on first load of any css file and save it. It should change absolutely nothing if the CSS stylesheet uses no nesting (but the bug just fixed was that it added newlines). I was not 100% sure of the denest code and decided using it in qCSSParser was enough until nested css became more common and then moving it to the epub import and add existing stage.

Hope that makes things clearer.

Got it. That makes sense. Thanks!

KevinH · Today, 02:59 PM

Quote:

Originally Posted by ElMiko

I... don't understand any of the preceding. I'm going to venture a guess that it means your turbo encabulator was not effectively deploying its sinusoidal depleneration failsafes. But I'm gratified to learn that I wasn't going crazy or that my css had a secret bug that would require my going back and fixing hundreds upon hundreds of ePubs. My soul is at peace.

I assume that "Sigil master" is a dev tool, and that any fixes will be applied to 2.8.x, so I should stick with 2.7.6 for now, yes?

Sigil's source code is stored in a github repository where many parallel development streams can be remembered automatically and periodically merged. The main "development stream" is referred to as the "master" branch.

Yes, that branch will be used as the basis for all development of the next Sigil release. That way the fix for your bug is never lost.

And yes, you have two choices: 1) revert to Sigil 2.7.6 or 2) wait for some kind soul that builds from current master to post an updated Sigil 2.8.0 version for you to use. DNSB has posted many such builds to help users in our new features request thread. We can also make a CI build available to you that includes the fix. If additional more dangerous bugs are found and reported in Sigil 2.8.0, then we usually make a fast-follow-on release to get that out there for users, but that is not planned for this bug since being off by a few lines when searching for a class name is not a dangerous issue.

KevinH · Today, 03:05 PM

Or what DiapDealer just said!

ps. And yes, thank you for your detailed bug report with sample epub with instructions!
Those really help.

DNSB · Today, 04:24 PM

For anyone who is interested, I've attached a link to a Sigil build for Windows with the DeNest bug fixed. In my testing, using either F3 or ctrl-click on a style in the XHTML file, opened the CSS file with the cursor at the start of the style.

Link to Sigil-2.8.0-Windows-x64-Setup.exe.

KevinH · Today, 04:37 PM

Quote:

Originally Posted by DNSB

For anyone who is interested, I've attached a link to a Sigil build for Windows with the DeNest bug fixed. In my testing, using either F3 or ctrl-click on a style in the XHTML file, opened the CSS file with the cursor at the start of the style.

Link to Sigil-2.8.0-Windows-x64-Setup.exe.

Thank you!

DNSB · Today, 04:57 PM

You're welcome.

ElMiko · Today, 05:01 PM

Quote:

Originally Posted by DNSB

For anyone who is interested, I've attached a link to a Sigil build for Windows with the DeNest bug fixed. In my testing, using either F3 or ctrl-click on a style in the XHTML file, opened the CSS file with the cursor at the start of the style.

Link to Sigil-2.8.0-Windows-x64-Setup.exe.

When's your next show? You're a rockstar.

DNSB · Today, 06:53 PM

Quote:

Originally Posted by ElMiko

When's your next show? You're a rockstar.

Since I already compile the latest updates for my use and testing, it takes very little time to upload the executable and create the link. I think of it as paying forward the help I received from many MobileRead members when I first joined.

It also makes it easier to find bugs such as the one you found with more eyes looking for issues.

Today, 03:05 PM	#22
KevinH Sigil Developer Posts: 9,699 Karma: 6774572 Join Date: Nov 2009 Device: many	Or what DiapDealer just said! ps. And yes, thank you for your detailed bug report with sample epub with instructions! Those really help. Last edited by KevinH; Today at 03:48 PM.

Similar Threads
Thread	Thread Starter	Forum	Replies	Last Post
Title Case not working on "the" "a" "in" etc	mike the leg	Library Management	9	11-06-2023 09:27 AM
Bug: Automater substitution of "and" with "&" in author field	mikebw	Library Management	2	04-15-2019 03:39 AM
"Doghouse Diaries" and "PHD Comics" not working	frakman1	Recipes	0	08-22-2014 11:32 AM
Creator taking control of the "first page" link in the "content" menu	cyberbaffled	Kindle Formats	2	07-11-2010 04:21 PM
BUG: GUI View button not working if Author tag contains "ë"	ShellShock	Calibre	1	11-15-2008 01:21 PM

Today, 02:04 PM	#16
DiapDealer Grand Sorcerer Posts: 29,489 Karma: 212177546 Join Date: Jan 2010 Device: Nexus 7, Kindle Fire HD	When is denest being used? Is it just when parsing/rendering the css, or is the css file actually being rewritten? I'm wondering if "Go to Link or Style" is going to work properly on nested css if it's only being denested when parsing/rendering. I'm sure I'm missing something, but I would think it would be hard for the cursor to be placed properly if that's the case.

Today, 02:33 PM	#17
ElMiko Fanatic Posts: 574 Karma: 65460 Join Date: Jun 2011 Device: Kindle Voyage, Boox Go 7	I... don't understand any of the preceding. I'm going to venture a guess that it means your turbo encabulator was not effectively deploying its sinusoidal depleneration failsafes. But I'm gratified to learn that I wasn't going crazy or that my css had a secret bug that would require my going back and fixing hundreds upon hundreds of ePubs. My soul is at peace. I assume that "Sigil master" is a dev tool, and that any fixes will be applied to 2.8.x, so I should stick with 2.7.6 for now, yes?

Today, 02:48 PM	#18
KevinH Sigil Developer Posts: 9,699 Karma: 6774572 Join Date: Nov 2009 Device: many	Sigil's "Go to link or style" will never work if the user has nested css and does not reformat its css to denest it using Sigil beforehand. But nested css is so rare right now that I am okay with that. The idea was if Go To Link Or Style ever fails, then the code should be denested first (given the last bugfix just now in master) that now works! The CSSDenest code is invoked by the qCSSParser right before parsing the css. The CSSInfo code uses the qCSSParser to parse a css file to find the line number of each selector (not just classes). The Go To Link or Style runs the CSSInfo code (that uses qCSSParser and CSSDeNest to create a list of all selectors and their starting line numbers and positions in the file) to determine which line of the css file to move to. At some point in the future, Sigil will autorun the CSSDenest process on first load of any css file and save it. It should change absolutely nothing if the CSS stylesheet uses no nesting (but the bug just fixed was that it added newlines). I was not 100% sure of the denest code and decided using it in qCSSParser was enough until nested css became more common and then moving it to the epub import and add existing stage. Hope that makes things clearer.

Today, 04:24 PM	#23
DNSB Bibliophagist Posts: 52,280 Karma: 180577194 Join Date: Jul 2010 Location: Vancouver Device: Kobo Sage, Libra Colour, Lenovo M8 FHD, Paperwhite 4, Tolino epos	For anyone who is interested, I've attached a link to a Sigil build for Windows with the DeNest bug fixed. In my testing, using either F3 or ctrl-click on a style in the XHTML file, opened the CSS file with the cursor at the start of the style. Link to Sigil-2.8.0-Windows-x64-Setup.exe.

Today, 04:57 PM	#25
DNSB Bibliophagist Posts: 52,280 Karma: 180577194 Join Date: Jul 2010 Location: Vancouver Device: Kobo Sage, Libra Colour, Lenovo M8 FHD, Paperwhite 4, Tolino epos	You're welcome.

Advert

Advert