0.6 Beta take 3 (0.5.906)

[crutledge]

Delete Unused Stylesheet Styles is a great addition that cleans-up the code.

If possible, it would be nice if this could be turned on/off in Preferences.

I would leave it ON.

[meme]

Its not a checkbox/option - its a function/action that gets run, like Validate. The same with Delete Unused Images. So you get to choose to use it or not

[mmat1]

Regex seams not to work properly. It seams that the "Dotall" option is always turned on (I'm sure that it is not checked")

[dynabook]

Could someone explain the dotall option? Does it have to do with greedy matching? Or what?
--MH

[kiwidude]

@mmat1 - oops - that would be because the settings are the wrong way around in the code. Will be fixed for the next build. I always leave both turned on so never noticed. Thanks for reporting this.

[kiwidude]

Originally Posted by dynabook

Could someone explain the dotall option? Does it have to do with greedy matching? Or what?
--MH

DotAll is (supposed to be) the (?s) PCRE flag.

Basically it comes down to if in an expression you do .* it matches just a single line or not. Say you want to get rid of whole script blocks, you can do <script>.*</script> with DotAll turned on, but that won't work with it turned off because the closing </script> is on a different line in the file.

[crutledge]

Originally Posted by meme

Its not a checkbox/option - its a function/action that gets run, like Validate. The same with Delete Unused Images. So you get to choose to use it or not

What I would expect from a Preferences Option.

[kiwidude]

@crutledge - it can only run at a point in time - just like validating with FlightCrew for instance. It is the same sort of feature. Otherwise you wouldn't be able to declare a new style and then switch to an html file to use it, it would be gone already . The only way at which you would know it was "safe" to be run is when the user says so - which is exactly what you have now by triggering it from the tools menu.

[meme]

Originally Posted by crutledge

What I would expect from a Preferences Option.

I don't think I quite understand how this would work, or help.

Delete Unused Stylesheet Classes (renamed for the next beta to be clear its only classes - although technically they are selectors) checks your HTML and stylesheets then deletes the stylesheet entries that weren't matched by a class used in the HTML.

Delete Unused Image Files does something similar - removes any image file that was not included in your HTML with an img tag.

These are actions you run on demand. If they were on or off in a preferences setting, then as soon as you added an image or typed a class it would be deleted before you could use it.

[theducks]

Originally Posted by crutledge

What I would expect from a Preferences Option.

These are not Toggled options.
They are click to run. No clickey, no runney which is a good thing. I might have a default stylesheet, that I have not finished editing the book sections, some of which may use some of those styles (later)

Sigil crew: GREAT feature

[Jabby]

Originally Posted by kiwidude

We have put it on our list to think about, as I said above it is too late for 0.6 now but if we can agree on something it will be done one day.

Sounds fair to me - thanks.

Regards - John

[mmat1]

Reformat (to single line per style) eats my characters

span.fakesmallcaps {font-size: .75em}

becomes:

span.fakesmallcaps { font-size: .75e; }

This happens probably only with lines which are already a single line style definition plus it does not happen always.

In my example I had 4 single line definitions at the end of the file (showing one of them above) and the very last 3 were faulty afterwards.

[kiwidude]

@mmat1 - thanks, that is an interesting one, it does seem the wind needs to blow in a particular direction for it to fail. It will get sorted for the next build.

[mmat1]

Originally Posted by kiwidude

@mmat1 - thanks, that is an interesting one, it does seem the wind needs to blow in a particular direction for it to fail. It will get sorted for the next build.

If you need the full css, its here

[kiwidude]

@mmat1 - thanks but I figured out the cause and the fix. It was a simple "one-off" error after the code was changed to handle three different purposes. Did the two new ones well, the original purpose not so much any more...