[Plugin] EnvVarEditor - GUI editor for Sigil's environment variables

[BeckyEbook]

OK. I'll look into it.

[DiapDealer]

Interestingly enough, it seems to be only the setting of SIGIL_PREVIEW_TIMEOUT that prevents the Save button from working correctly. I tried the other variables that took user input and they worked fine for me.

[KevinH]

In case this helps, I changed your use of "try:" to catch a ValueError to instead catch all Exceptions and print out the exceptions error message and I get the following errors when trying to change the Preview Timeout and then save:

Traceback (most recent call last):
  File "/Users/kbhend/Library/Application Support/sigil-ebook/sigil/plugins/EnvVarEditor/plugin.py", line 849, in on_accept
    values = self.get_values()
  File "/Users/kbhend/Library/Application Support/sigil-ebook/sigil/plugins/EnvVarEditor/plugin.py", line 790, in get_values
    if w.value() != default_val:
                    ^^^^^^^^^^^
NameError: name 'default_val' is not defined

Hope this helps.

[BeckyEbook]

Sure. Thanks!
I'll test it and prepare a fix.

[KevinH]

Yes, if I comment that line out, then it all works. But that makes it an exact dupilcate of the next elif. I think only the first int handling elif will ever be used not the next.

[BeckyEbook]

It was indeed unnecessary to have two versions of elif (the old and the new). Thanks.
I've prepared a revised beta version; it's in the second post.

[jugaor]

Originally Posted by BeckyEbook

It was indeed unnecessary to have two versions of elif (the old and the new). Thanks.
I've prepared a revised beta version; it's in the second post.

BeckyEbook, sorry for the delay. This new version works perfectly.

Thank you very much!!

[BeckyEbook]

Thank you for the confirmation.

The plugin hasn't received much feedback, which is why I'm still keeping it in beta. If I don't receive any bug reports for a while, I'll release a stable version.

[philja]

Following my problem with the python function replace, I used this plugin to make use of the SIGIL_FUNCTION_REPLACE_LOGFILE.

My difficulty with this plugin is that its dialog window is not resizable enough especially on the width. The result is that the Set/Unset/Not Set column is truncated in width.

But the most difficulty comes in checking the full path of the logfile. This is no doubt down to the display settings on my desktop machines, Linux and Windows11. The MacBook was a little better but not perfect.

See the attached screenshot (Linux).

I had some initial problem getting the logfile to work and that was down to the specification of the full path. Even clicking in the file name box and using the right arrow, I can't get to the end of the filename. The only way to check was to use CtrlA to select all and then CtrlC and CtrlV in a text editor to see the full path and logfile name.

Can this dialog window have its max width extended somewhat?

[BeckyEbook]

I'll look into it.

EDIT: I've released version 0.1.18 beta.
Although I haven't tested it on Linux, I hope it will work fine.

Thank you for reporting the bug.

[philja]

Originally Posted by BeckyEbook

I'll look into it.

EDIT: I've released version 0.1.18 beta.
Although I haven't tested it on Linux, I hope it will work fine.

Thank you for reporting the bug.

Thanks, Becky. I've tested on all three of my OS and it's greatly improved.

On linux, I can now click in the filename box and left or right arrow to either end.

If you want to 'prettify' a bit, the Set/Unset/Not Set box is a little destitute.

BUT, it is entirely and easily usable now.

See attached screenshot from the LInux OS.

[BeckyEbook]

Thank you for your additional feedback.
I appreciate it when you point out specific issues; I've done my best to fix them.

I've released version 0.1.19 beta.