emacs-orgmode
[Top][All Lists]
Advanced

[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 13:29:18 +0200

Resending the patch. There were missing parenthesis. And I didn't test build 
after modification. Fixed and retested.

Mikhail

On Monday, September 21, 2020 10:23:39 AM CEST Mikhail Skorzhinskii wrote:
> 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)))

Attachment: 0001-org-agenda.el-allow-to-customize-outline-path-in-ech.patch
Description: Text Data


reply via email to

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