domtronn / spaceline-all-the-icons.el

A Spaceline Mode Line theme using All The Icons for Emacs
MIT License
235 stars 25 forks source link

all-the-icons-org-clock-current-task broke modeline for active buffer #60

Closed drrlvn closed 7 years ago

drrlvn commented 7 years ago

Commit f49ae9c7c6ed7001b7feffc6e03d51481ec67103 broke the modeline on the active buffer. Commenting it out solves the issue.

domtronn commented 7 years ago

That commit worked when I tested, and I don't use org-clock at all - But I've added a predicate to check that the variable is bound in the :when predicate 👍

alexmurray commented 7 years ago

Oops, sorry about that. FYI, org-clock-current-task is a variable so that should be boundp not fboundp

domtronn commented 7 years ago

You're right, thanks @alexmurray - I'll push that fix as well 😅 I can use bound-and-true-p which is a little more concise

drrlvn commented 7 years ago

You can't use a symbol with bound-and-true-p, need to simply pass the variable: (bound-and-true-p org-clock-current-task).