Refactor +ivy/tasks
Just realized ivy passes the candidate in its original data structure to its handlers.
This commit is contained in:
parent
39d4fa2256
commit
d3cb394ce4
1 changed files with 16 additions and 18 deletions
|
@ -153,19 +153,19 @@ If ARG (universal argument), open selection in other-window."
|
|||
(cl-loop for task in +ivy-task-tags maximize (length (car task))))
|
||||
(max-desc-width
|
||||
(cl-loop for task in tasks maximize (length (cl-cdadr task))))
|
||||
(max-width (max (- (frame-width) (1+ max-type-width) max-desc-width)
|
||||
(max-width (max (+ max-desc-width 3)
|
||||
25)))
|
||||
(cl-loop
|
||||
with fmt = (format "%%-%ds %%-%ds%%s%%s:%%s" max-type-width max-width)
|
||||
with fmt = (format "%%-%ds %%-%ds%%s:%%s" max-type-width max-width)
|
||||
for alist in tasks
|
||||
collect
|
||||
(let-alist alist
|
||||
(format fmt
|
||||
(list (format fmt
|
||||
(propertize .type 'face (cdr (assoc .type +ivy-task-tags)))
|
||||
(substring .desc 0 (min max-desc-width (length .desc)))
|
||||
(propertize " | " 'face 'font-lock-comment-face)
|
||||
(propertize (abbreviate-file-name .file) 'face 'font-lock-keyword-face)
|
||||
(propertize .line 'face 'font-lock-constant-face))))))
|
||||
(propertize .line 'face 'font-lock-constant-face))
|
||||
.type .file .line)))))
|
||||
|
||||
(defun +ivy--tasks (target)
|
||||
(let* (case-fold-search
|
||||
|
@ -203,16 +203,14 @@ If ARG (universal argument), open selection in other-window."
|
|||
|
||||
(defun +ivy--tasks-open-action (x)
|
||||
"Jump to the file and line of the current task."
|
||||
(let ((location (cadr (split-string x " | ")))
|
||||
(type (car (split-string x " "))))
|
||||
(cl-destructuring-bind (file line) (split-string location ":")
|
||||
(cl-destructuring-bind (label type file line) x
|
||||
(with-ivy-window
|
||||
(find-file (expand-file-name file (doom-project-root)))
|
||||
(goto-char (point-min))
|
||||
(forward-line (1- (string-to-number line)))
|
||||
(search-forward type (line-end-position) t)
|
||||
(backward-char (length type))
|
||||
(recenter)))))
|
||||
(when (search-forward type (line-end-position) t)
|
||||
(backward-char (length type)))
|
||||
(recenter))))
|
||||
|
||||
;;;###autoload
|
||||
(defun +ivy/tasks (&optional arg)
|
||||
|
|
Loading…
Add table
Add a link
Reference in a new issue