Re: [PATCH] Re: [ISSUE] Re: [PATCH] lisp/org-agenda.el: Add header to agenda clock report table

2022-07-10 Thread Ihor Radchenko
Ihor Radchenko writes: > "Christopher M. Miles" writes: > >> I tested this commit, it should auto append a newline character after >> ~org-agenda-clock-report-header~. Otherwise the header line will in same >> line with table first line. > > Reasonable concern. > The attached patch should fix i

[PATCH] Re: [ISSUE] Re: [PATCH] lisp/org-agenda.el: Add header to agenda clock report table

2022-05-09 Thread Ihor Radchenko
"Christopher M. Miles" writes: > I tested this commit, it should auto append a newline character after > ~org-agenda-clock-report-header~. Otherwise the header line will in same line > with table first line. Reasonable concern. The attached patch should fix it. Best, Ihor >From c918d5097af4ca

[ISSUE] Re: [PATCH] lisp/org-agenda.el: Add header to agenda clock report table

2022-01-28 Thread Christopher M. Miles
I tested this commit, it should auto append a newline character after ~org-agenda-clock-report-header~. Otherwise the header line will in same line with table first line. Samim Pezeshki writes: > Thanks Nicolas for reviewing the changes! > > I applied the changes. > Attached is the updated pat

Re: [PATCH] lisp/org-agenda.el: Add header to agenda clock report table

2022-01-26 Thread Nicolas Goaziou
Hello, Samim Pezeshki writes: > Thanks Nicolas for reviewing the changes! > > I applied the changes. > Attached is the updated patch. Applied. Thank you. Regards, -- Nicolas Goaziou

Re: [PATCH] lisp/org-agenda.el: Add header to agenda clock report table

2022-01-22 Thread Samim Pezeshki
er)) > > Maybe: > > (insert (propertize org-agenda-clock-report-header 'face > 'org-agenda-structure)) > > Could you send an updated patch? > > Regards, > -- > Nicolas Goaziou > From 6691abdcef95181ea9517e2c2c60ca52443c14a6 Mon Sep 17 00:00:00 2001 Fro

Re: [PATCH] lisp/org-agenda.el: Add header to agenda clock report table

2022-01-21 Thread Nicolas Goaziou
Hello, Samim Pezeshki writes: > +(defcustom org-agenda-clock-report-header nil > + "Header for org agenda clock report mode" > + :group 'org-agenda > + :type 'string) Please add :package-version and :safe keywords to the definition. Also, type does not match the default value. You probably

Re: [PATCH] lisp/org-agenda.el: Add header to agenda clock report table

2022-01-21 Thread Samim Pezeshki
Should I do something for this patch to be approved? On Mon, Jan 10, 2022 at 1:58 PM Christopher M. Miles wrote: > > Good idea to add header for clock report table. > > Samim Pezeshki writes: > > > Hi, > > > > This patch adds a header to the clock report table in the agenda, if > > org-agenda-c

Re: [PATCH] lisp/org-agenda.el: Add header to agenda clock report table

2022-01-10 Thread Christopher M. Miles
Good idea to add header for clock report table. Samim Pezeshki writes: > Hi, > > This patch adds a header to the clock report table in the agenda, if > org-agenda-clock-report-header is set. > > Thanks, > Samim > > [4. text/x-patch; > 0001-lisp-org-agenda.el-Add-header-to-agenda-clock-report.

[PATCH] lisp/org-agenda.el: Add header to agenda clock report table

2022-01-10 Thread Samim Pezeshki
: Add header to agenda clock report table * lisp/org-agenda.el (org-agenda-list): Add header to agenda clock report table. TINYCHANGE --- doc/org-manual.org | 4 ++-- etc/ORG-NEWS | 1 + lisp/org-agenda.el | 9 + 3 files changed, 12 insertions(+), 2 deletions(-) diff --git a/doc/org