Missing "(require 'ace-window)" in zetteldeft.org documentation about zetteldeft

[EFLS]

Yeah that's probably not best practice... I'll have to change that.

I didn't want to oblige people to install ace-window as it is only used for that specific function, but then again requiring inside a function might not be a good solution. Not sure what would be a good approach to avoid requiring ace-window from the start, but in the mean time I should require it similar to other packages.

[EFLS]

I tried figuring out an approach by asking /r/emacs on Reddit. Not sure which answer to follow though...

Check it here:
https://www.reddit.com/r/emacs/comments/gtbfux/question_about_best_practice_for_require_when/

The options seem to be:

• require ace-window at the top level & list it as dependency
• create a defcustom so that people can use a function of choice (but this seems a bit more of a hassle)
• handle this with a soft require, for example showing a warning when ace-window isn't found, but not adding it to the dependencies

The most stable approach seems to be to simply require at the top level (thus loading ace-window even when aren't using the function).

If anyone else has opinions on how to best handle this, please share

[gklimowicz]

I'm fine with including it as a top-level requirement.

[EFLS]

Yeah, I guess that is the cleanest option. Fixed.