emacs-orgmode
[Top][All Lists]
Advanced

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

Re: [PATCH] lisp/org-colview.el: [refactor] [extract function] `org-colu


From: Ihor Radchenko
Subject: Re: [PATCH] lisp/org-colview.el: [refactor] [extract function] `org-columns-move-up' & `org-columns-move-down'
Date: Sun, 04 Feb 2024 18:29:11 +0000

SÅ‚awomir Grochowski <slawomir.grochowski@gmail.com> writes:

> It's a small refactoring - extract function - and the patch is
> self-explanatory.

Thanks!

> +(defun org-columns-move-up ()
> +  (interactive)
> ...
> +(defun org-columns-move-down ()
> +  (interactive)
> ...

May you also add docstrings?
And maybe better names would be org-columns-next/previous-line.

-- 
Ihor Radchenko // yantar92,
Org mode contributor,
Learn more about Org mode at <https://orgmode.org/>.
Support Org development at <https://liberapay.com/org-mode>,
or support my work at <https://liberapay.com/yantar92>



reply via email to

[Prev in Thread] Current Thread [Next in Thread]