[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [PATCHES 1, 2, 3/3] Minor customisation features for ox-icalendar an
|
From: |
Mikhail Skorzhinskii |
|
Subject: |
Re: [PATCHES 1, 2, 3/3] Minor customisation features for ox-icalendar and agenda |
|
Date: |
Mon, 21 Sep 2020 10:23:39 +0200 |
On Wednesday, September 16, 2020 4:15:20 AM CEST Ihor Radchenko wrote:
> > 1. Show document title instead of file name in org-agenda;
> >
> > This should be useful for any users of org-roam package or anyone who
> > autogenerate file names and uses document titles instead for navigation.
>
> Interesting idea. This may also be useful as an option to
> `org-refile-use-outline-path'. Currently, its value may be 'file to
> prepend file names to the outline paths in refile targets. Using
> document title (i.e. when the variable is set to 'title) in the refile
> paths may also be handy.
This is indeed better way.
Fixed.
Thanks,
Mikhail
> Best,
> Ihor
>
> Mikhail Skorzhinskii <mskorzhinskiy@eml.cc> writes:
> > Hello forum,
> >
> > I'd like to introduce three new features and request to merge the with
> > current master:
> >
> > 1. Show document title instead of file name in org-agenda;
> >
> > This should be useful for any users of org-roam package or anyone who
> > autogenerate file names and uses document titles instead for navigation.
> >
> > 2. Customise summary lines of exported events in ox-icalendar
> >
> > I personally export deadlines and scheduled items to the separate files so
> > I don't need these cookies and they consume precious space of summary
> > line. So I prefer them be disabled. This should be possible to do now
> > with only customisation setting.
> >
> > 3. Force creation of an alarm when exporting event in ox-icalendar
> >
> > Sometimes I prefer to create icalendar alarms exactly at the event start,
> > but it's not possible since alarm set to zero means that alarm will be
> > disabled. Forcing alarm creation will now create alarm no matter the
> > setting.
> >
> > I've updated news files and attempted to replicate the style of
> > submissions, but I admit that I didn't read much beyond CONTRIBUTE file.
> > Please point me to the right direction in case of any issues I can fix.
> >
> > Kind regards,
> > Mikhail Skorzhinskii
> >
> > From 111e6886564abbf3becb2a94e66f235f502b79d9 Mon Sep 17 00:00:00 2001
> > From: Mikhail Skorzhinskii <mskorzhinskiy@eml.cc>
> > Date: Sat, 12 Sep 2020 18:52:39 +0200
> > Subject: [PATCH 3/3] ox-icalendar.el: introduce setting to force alarms
> >
> > This is a new setting for icalendar exports to allow users set alarms
> > exactly at the event start. With this setting set to non-nil and alarm
> > set to zero (by global setting or APPT_WARNTIME property) it will create
> > an alarm at the event start.
> >
> > Note, that zero alarm set as APPT_WARNTIME property will override
> > default warning time.
> > ---
> >
> > etc/ORG-NEWS | 10 ++++++++++
> > lisp/ox-icalendar.el | 24 +++++++++++++++++++++---
> > 2 files changed, 31 insertions(+), 3 deletions(-)
> >
> > diff --git a/etc/ORG-NEWS b/etc/ORG-NEWS
> > index b912e807d..eb950d934 100644
> > --- a/etc/ORG-NEWS
> > +++ b/etc/ORG-NEWS
> > @@ -23,6 +23,16 @@ path.
> >
> > Through these new options for icalendar export, one can customise the
> > looks of summary lines in exported events.
> >
> > +*** New option ~org-icalendar-force-alarm~
> > +
> > +This is a new setting for icalendar exports to allow users set alarms
> > +exactly at the event start. With this setting set to non-nil and alarm
> > +set to zero (by global setting or APPT_WARNTIME property) it will
> > +create an alarm at the event start.
> > +
> > +Note, that zero alarm set as =APPT_WARNTIME= property will override
> > +default warning time.
> > +
> >
> > * Version 9.4
> > ** Incompatible changes
> > *** Possibly broken internal file links: please check and fix
> >
> > diff --git a/lisp/ox-icalendar.el b/lisp/ox-icalendar.el
> > index f110a4b2b..baed925e8 100644
> > --- a/lisp/ox-icalendar.el
> > +++ b/lisp/ox-icalendar.el
> > @@ -64,6 +64,15 @@ for timed events. If non-zero, alarms are created.
> >
> > :version "24.1"
> > :type 'integer)
> >
> > +(defcustom org-icalendar-force-alarm nil
> > + "Non-nil means alarm will be created even if is set to zero.
> > +
> > +This overrides default behaviour where zero means no alarm. With
> > +this set to non-nil and alarm set to zero, alarm will be created
> > +and will fire at the event start."
> > + :group 'org-export-icalendar
> > + :type 'bool)
> > +
> >
> > (defcustom org-icalendar-combined-name "OrgMode"
> >
> > "Calendar name for the combined iCalendar representing all agenda
> > files."
> >
> > :group 'org-export-icalendar
> >
> > @@ -797,8 +806,11 @@ Return VALARM component as a string, or nil if it
> > isn't allowed.">
> > (let ((alarm-time
> >
> > (let ((warntime
> >
> > (org-element-property :APPT_WARNTIME entry)))
> >
> > - (if warntime (string-to-number warntime) 0))))
> > - (and (or (> alarm-time 0) (> org-icalendar-alarm-time 0))
> > + (if warntime (string-to-number warntime) nil))))
> > + (and (or (and alarm-time
> > + (> alarm-time 0))
> > + (> org-icalendar-alarm-time 0)
> > + org-icalendar-force-alarm)
> >
> > (org-element-property :hour-start timestamp)
> > (format "BEGIN:VALARM
> >
> > ACTION:DISPLAY
> >
> > @@ -806,7 +818,13 @@ DESCRIPTION:%s
> >
> > TRIGGER:-P0DT0H%dM0S
> > END:VALARM\n"
> >
> > summary
> >
> > - (if (zerop alarm-time) org-icalendar-alarm-time
alarm-time)))))
> > + (if org-icalendar-force-alarm
> > + (if alarm-time
> > + alarm-time
> > + org-icalendar-alarm-time)
> > + (if (zerop alarm-time)
> > + org-icalendar-alarm-time
> > + alarm-time))))))
> >
> > ;;;; Template
> >
> > From 1c30be14e2e7a6774d499388ec207b1950963746 Mon Sep 17 00:00:00 2001
> > From: Mikhail Skorzhinskii <mskorzhinskiy@eml.cc>
> > Date: Sat, 12 Sep 2020 18:27:23 +0200
> > Subject: [PATCH 2/3] ox-icalendar.el: allow to customise deadline and
> >
> > scheduled items summary
> >
> > In some of export schemes these cookies at the start of each event can
> > be a distraction rather then storage of useful information.
> > ---
> >
> > etc/ORG-NEWS | 5 +++++
> > lisp/ox-icalendar.el | 10 ++++++++--
> > 2 files changed, 13 insertions(+), 2 deletions(-)
> >
> > diff --git a/etc/ORG-NEWS b/etc/ORG-NEWS
> > index 32f64d84e..b912e807d 100644
> > --- a/etc/ORG-NEWS
> > +++ b/etc/ORG-NEWS
> > @@ -18,6 +18,11 @@ When set to 'title, will show document title in outline
> > path in echo>
> > area instead of file name. When set to nil will show only outline
> > path.
> >
> > +*** New options ~org-icalendar-scheduled-summary-prepend~ and
> > ~org-icalendar-deadline-summary-prepend~ +
> > +Through these new options for icalendar export, one can customise the
> > +looks of summary lines in exported events.
> > +
> >
> > * Version 9.4
> > ** Incompatible changes
> > *** Possibly broken internal file links: please check and fix
> >
> > diff --git a/lisp/ox-icalendar.el b/lisp/ox-icalendar.el
> > index 0f890534a..f110a4b2b 100644
> > --- a/lisp/ox-icalendar.el
> > +++ b/lisp/ox-icalendar.el
> > @@ -82,6 +82,12 @@ keyword."
> >
> > :group 'org-export-icalendar
> > :type '(repeat (string :tag "Tag")))
> >
> > +(defcustom org-icalendar-scheduled-summary-prepend "S: "
> > + "String used for prepending summary in exported scheduled headlines.")
> > +
> > +(defcustom org-icalendar-deadline-summary-prepend "DL: "
> > + "String used for prepending summary in exported deadlines.")
> > +
> >
> > (defcustom org-icalendar-use-deadline '(event-if-not-todo todo-due)
> >
> > "Contexts where iCalendar export should use a deadline time stamp.
> >
> > @@ -606,7 +612,7 @@ inlinetask within the section."
> >
> > (_ (memq 'event-if-not-todo use-deadline)))
> >
> > (org-icalendar--vevent
> >
> > entry deadline (concat "DL-" uid)
> >
> > - (concat "DL: " summary) loc desc cat tz class)))
> > + (concat org-icalendar-deadline-summary-prepend
summary) loc desc cat
> > tz class)))>
> > (let ((scheduled (org-element-property :scheduled entry))
> >
> > (use-scheduled (plist-get info :icalendar-use-
scheduled)))
> >
> > (and scheduled
> >
> > @@ -617,7 +623,7 @@ inlinetask within the section."
> >
> > (_ (memq 'event-if-not-todo use-scheduled)))
> >
> > (org-icalendar--vevent
> >
> > entry scheduled (concat "SC-" uid)
> >
> > - (concat "S: " summary) loc desc cat tz class)))
> > + (concat org-icalendar-scheduled-summary-prepend
summary) loc desc cat
> > tz class)))>
> > ;; When collecting plain timestamps from a headline and its
> > ;; title, skip inlinetasks since collection will happen once
> > ;; ENTRY is one of them.
> >
> > From 6468b0402bf55ee21eb199395e41bc706fa17354 Mon Sep 17 00:00:00 2001
> > From: Mikhail Skorzhinskii <mskorzhinskiy@eml.cc>
> > Date: Sat, 12 Sep 2020 18:10:05 +0200
> > Subject: [PATCH 1/3] org-agenda.el: allow to customize outline path in
> > echo
> >
> > area
> >
> > Some of the org-mode data bases auto-generate file names, so they don't
> > carry a lot of information and can be noisy. Allowing user to show
> > document titles instead could be useful. Showing titles is disabled by
> > default.
> > ---
> >
> > etc/ORG-NEWS | 7 +++++++
> > lisp/org-agenda.el | 10 +++++++++-
> > lisp/org.el | 25 ++++++++++++++++++++++---
> > 3 files changed, 38 insertions(+), 4 deletions(-)
> >
> > diff --git a/etc/ORG-NEWS b/etc/ORG-NEWS
> > index 0ed626fb7..32f64d84e 100644
> > --- a/etc/ORG-NEWS
> > +++ b/etc/ORG-NEWS
> > @@ -11,6 +11,13 @@ See the end of the file for license conditions.
> >
> > Please send Org bug reports to mailto:emacs-orgmode@gnu.org.
> >
> > * Version 9.5 (not yet released)
> >
> > +** New options and settings
> > +*** New option ~org-agenda-show-outline-path-prepend~
> > +
> > +When set to 'title, will show document title in outline path in echo
> > +area instead of file name. When set to nil will show only outline
> > +path.
> > +
> >
> > * Version 9.4
> > ** Incompatible changes
> > *** Possibly broken internal file links: please check and fix
> >
> > diff --git a/lisp/org-agenda.el b/lisp/org-agenda.el
> > index 82fe6091c..25c9051e2 100644
> > --- a/lisp/org-agenda.el
> > +++ b/lisp/org-agenda.el
> > @@ -1044,6 +1044,14 @@ current item's tree, in an indirect buffer."
> >
> > :group 'org-agenda-startup
> > :type 'boolean)
> >
> > +(defcustom org-agenda-show-outline-path-prepend t
> > + "Non-nil means show outline path with document title (if present) in
> > echo area instead of file name." + :group 'org-agenda-startup
> > + :type '(choice
> > + (const :tag "Show only outline path without file name or document
> > title." nil) + (const :tag "Prepend outline path with file name."
t)
> > + (const :tag "Prepend outline path with title name. Fallback to
file
> > name is no title is present." title))) +
> >
> > (defcustom org-agenda-start-with-entry-text-mode nil
> >
> > "The initial value of entry-text-mode in a newly created agenda
> > window."
> >
> > :group 'org-agenda-startup
> >
> > @@ -8777,7 +8785,7 @@ When called with a prefix argument, include all
> > archive files as well.">
> > (org-agenda-tree-to-indirect-buffer nil)
> >
> > (org-agenda-show)))
> >
> > (and org-agenda-show-outline-path
> >
> > - (org-with-point-at m (org-display-outline-path t))))))
> > + (org-with-point-at m (org-display-outline-path
> > org-agenda-show-outline-path-prepend))))))>
> > (defun org-agenda-show-tags ()
> >
> > "Show the tags applicable to the current item."
> >
> > diff --git a/lisp/org.el b/lisp/org.el
> > index 020cfc873..d13d78dbf 100644
> > --- a/lisp/org.el
> > +++ b/lisp/org.el
> > @@ -8024,10 +8024,24 @@ the default is \"/\"."
> >
> > (setf (substring fpath (- width 2)) "..")))
> >
> > fpath))
> >
> > -(defun org-display-outline-path (&optional file current separator
> > just-return-string) +(defun org-get-title-from-file (file)
> > + "Collect tilte from the provided `org-mode' FILE."
> > + (let (title)
> > + (when file
> > + (with-current-buffer
> > + (get-file-buffer file)
> > + (pcase (org-collect-keywords '("TITLE"))
> > + (`(("TITLE" . ,val))
> > + (setq title (car val)))))
> > + title)))
> > +
> > +
> > +(defun org-display-outline-path (&optional file-or-title current
> > separator just-return-string)>
> > "Display the current outline path in the echo area.
> >
> > -If FILE is non-nil, prepend the output with the file name.
> > +If FILE-OR-TITLE is 'title, prepend outline with file title. If
> > +it is non-nil or title is not present in document, prepend
> > +outline path with the file name.
> >
> > If CURRENT is non-nil, append the current heading to the output.
> > SEPARATOR is passed through to `org-format-outline-path'. It separates
> > the different parts of the path and defaults to \"/\".
> >
> > @@ -8035,6 +8049,8 @@ If JUST-RETURN-STRING is non-nil, return a string,
> > don't display a message.">
> > (interactive "P")
> > (let* (case-fold-search
> >
> > (bfn (buffer-file-name (buffer-base-buffer)))
> >
> > + (title (when (and file-or-title (string= file-or-title 'title))
> > + (org-get-title-from-file bfn)))
> >
> > (path (and (derived-mode-p 'org-mode) (org-get-outline-path)))
> > res)
> >
> > (when current (setq path (append path
> >
> > @@ -8046,7 +8062,10 @@ If JUST-RETURN-STRING is non-nil, return a string,
> > don't display a message.">
> > (org-format-outline-path
> >
> > path
> > (1- (frame-width))
> >
> > - (and file bfn (concat (file-name-nondirectory bfn) separator))
> > + (and file-or-title bfn (concat (if (and (string= file-or-title
> > 'title) title) +
title
> > + (file-name-
nondirectory bfn))
> > + separator))
> >
> > separator))
> >
> > (add-face-text-property 0 (length res)
> >
> > `((t :height ,(face-attribute 'default
:height)))
0001-org-agenda.el-allow-to-customize-outline-path-in-ech.patch
Description: Text Data