Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Added org todos and agenda to home screen #4483

Closed
wants to merge 2 commits into from
Closed

Added org todos and agenda to home screen #4483

wants to merge 2 commits into from

Conversation

tonyday567
Copy link
Contributor

replacing #3720

@robbyoconnor
Copy link
Contributor

👍

(cons "file" file)
(cons "pos" pos)
(cons "time" time)))
) rtn)))))
Copy link
Owner

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Please can you break this into sub-functions :-)

@tonyday567
Copy link
Contributor Author

@syl20bnr refactored. I'm not very experienced with elisp - I used a 'loop for collect' but wasn't sure if this is a common idiom. If there's any other improvements or tips, happy to incorporate.

@syl20bnr
Copy link
Owner

syl20bnr commented Feb 1, 2016

Thank you 👍
There some rough edges with it:

  • the point jumps to EOF, it is caused by (require 'org-agenda), I did not dig further
  • There are some errors with org-bullets-mode but it may be specific with my configuration

Cherry-picked into develop branch, you can safely delete your branch.

@syl20bnr syl20bnr closed this Feb 1, 2016
@Andre0991
Copy link
Contributor

@syl20bnr The org-bullets-mode thing is an upstream problem: #4793

I don't know if the fixed version is already in the org repo.

@fintelkai
Copy link
Contributor

I like the idea of this but many of my todos and agenda items have links (to emails in particular) in them and since the spacemacs home buffer is not an org buffer, the links aren't fontified the way they would be in an org-mode buffer. That kind of makes this ugly and not so useful.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants