~eshel

https://eshelyaron.com

~eshel/dev

Last active 9 minutes ago
View more

Recent activity

Re: Eldoc issue 2 days ago

From Eshel Yaron to ~eshel/dev

Hello Philip,

Philip Kaludercic <philip.kaludercic@fau.de> writes:

> Hi, it seems this issue hasn't been mentioned anywhere on the mailing
> list, so I think it is worth bringing up.

Of course, thanks for reporting :)

> Whenever the point is on a symbol, and an eldoc hint should pop up, I
> get this error message instead:
>
> eldoc error: (prolog-exception compound error (compound
> existence_error (atom . procedure) (compound : (atom . sweep)

Re: [PATCH] Do not autoload keymaps a month ago

From Eshel Yaron to ~eshel/dev

Philip Kaludercic <philipk@posteo.net> writes:

> When a keymap is autoloaded (and the definition is copied over to the
> -autoloads.el file), and the function are sharp-quoted, Emacs can
> complain about undefined symbols, especially in combination with
> 'package-quickstart'.
> ---
>  sweeprolog.el | 2 --
>  1 file changed, 2 deletions(-)
>
> diff --git a/sweeprolog.el b/sweeprolog.el
> index 5a5fe7a..567665d 100644
> --- a/sweeprolog.el
> +++ b/sweeprolog.el

Re: Simplify face definitions in sweeprolog 3 months ago

From Eshel Yaron to ~eshel/dev

>> Admittedly, this adds a layer of complication to the code, and I'm not
>> sure the current approach (namely the sweeprolog-defface macro) is the
>> most elegant...
>
> Maybe a more elegant approach (or at least, one that fits better with
> the rest of Emacs facilities) is to make the dark and the light options
> custom-themes?  A "custom theme" is nothing more than a set of
> variable&face settings, and users can enable any number of themes they
> want, so it wouldn't prevent them from using some other theme at the
> same time?

That makes a lot of sense, thanks!  I'll look into extracting the
non-default variants and refactoring them as custom themes.

Re: Simplify face definitions in sweeprolog 3 months ago

From Eshel Yaron to ~eshel/dev

Hi Stefan,

Stefan Monnier <monnier@iro.umontreal.ca> writes:

> Hi,
>
> The patch below simplifies face definitions by consolidating common
> elements from dark&light into the default.

Thank you, I really appreciate you taking an interest in Sweep.

The changes you suggest to the face definitions actually alter the
various faces that we end up with, though.  Basically, we have three
sets of faces: default, light and dark.  The default faces are what most

Re: [PATCH] Minor doc fixes 3 months ago

From Eshel Yaron to ~protesilaos/ef-themes

Protesilaos Stavrou <info@protesilaos.com> writes:

>> From: Eshel Yaron <me@eshelyaron.com>
>> Date: Tue, 20 Jun 2023 15:33:30 +0300
>>
>> * README.org (Preview theme colors): Fix typo.
>> (Sample configuration, Option for headings):
>> * ef-themes.el (ef-themes-headings): Prefer saying '(foo bar baz)'
>> over '(foo . (bar baz))' where appropriate; say "non-nil non-list
>> value" to describe such values.
>> ---
>>  README.org   | 54 ++++++++++++++++++++++++++--------------------------
>>  ef-themes.el | 33 ++++++++++++++++----------------
>>  2 files changed, 44 insertions(+), 43 deletions(-)

[PATCH] Minor doc fixes 3 months ago

From Eshel Yaron to ~protesilaos/ef-themes

* README.org (Preview theme colors): Fix typo.
(Sample configuration, Option for headings):
* ef-themes.el (ef-themes-headings): Prefer saying '(foo bar baz)'
over '(foo . (bar baz))' where appropriate; say "non-nil non-list
value" to describe such values.
---
 README.org   | 54 ++++++++++++++++++++++++++--------------------------
 ef-themes.el | 33 ++++++++++++++++----------------
 2 files changed, 44 insertions(+), 43 deletions(-)

diff --git a/README.org b/README.org
index e0599b6..987cb19 100644
--- a/README.org
+++ b/README.org
[message trimmed]

Re: [PATCH] * sweeprolog.el (sweeprolog--load-module): Use 'module-load'. 4 months ago

From Eshel Yaron to ~eshel/dev

Philip Kaludercic <philipk@posteo.net> writes:

> ---
>  sweeprolog.el | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/sweeprolog.el b/sweeprolog.el
> index cd4bbd8..1900241 100644
> --- a/sweeprolog.el
> +++ b/sweeprolog.el
> @@ -633,7 +633,7 @@ use `autoload/2' for all added directives."
>                              (group-n 1 (one-or-more not-newline))
>                              eos)
>                          line)

Re: [PATCH] Exclude Info indices from the HTML version of the ToC 11 months ago

From Eshel Yaron to ~protesilaos/denote

Protesilaos Stavrou <info@protesilaos.com> writes:

> Installed and pushed.  I also made the update to the website.  Waiting
> for the changes to propagate.

Great! Seems to have worked fine, now the last entry in the table of
contents I see on the website is "19. GNU Free Documentation License".


Best,
Eshel

[PATCH] Exclude Info indices from the HTML version of the ToC 11 months ago

From Eshel Yaron to ~protesilaos/denote

Currently the HTML version of the manual at
https://protesilaos.com/emacs/denote lists in its table of contents
links to the Indices header and its subheaders, but these are not
present in the HTML version of the manual.

Set the UNNUMBERED property of the Indices header to "notoc" to
exclude the entire subtree from the ToC during export to HTML.
---
 README.org | 1 +
 1 file changed, 1 insertion(+)

diff --git a/README.org b/README.org
index 4ef6e75..322b7b6 100644
--- a/README.org
[message trimmed]

[PATCH] denote.el (denote): Use active region in default title prompt 1 year, 1 month ago

From Eshel Yaron to ~protesilaos/denote

Hello Prot, all,

The following is a tiny patch that I want to suggest for denote, which
is intended to facilitate a slightly smoother note taking by passing the
contents of the region from the denote command to denote--title-prompt
as the DEFAULT-TITLE argument, when the region is active.

The idea behind this DWIM-flavored patch is to be able to take a note
about a subject that I appears in a buffer by simply marking it before
invoking the denote command, which feels natural to me.


---
[message trimmed]