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

Agenda highlight headlines #281

Merged
merged 2 commits into from
Dec 29, 2020
Merged

Conversation

ricardoricho
Copy link
Contributor

Introduces dashboard-agenda-highlight-headlines as a Boolean to
highlight agenda headlines using org default faces.

Headlines are highlighted by setting the corresponding
face text-property to each part of the headline. Issue #264 shows
that headlines are highlighted only after visit the org buffer and
when org buffer is killed headlines looses their faces.

I couldn't found the function that sets the faces, so now
faces are assigned to the TODO and HEADING parts only if they don't
already had a face.

@jcs090218
Copy link
Member

Thanks for fixing this bug for us.

I wonder maybe we can remove variable dashboard-agenda-highligth-headlines and just highlight the headlines as the default behaviour? So user can have less variables to customize.

Introduces `dashboard-agenda-highlight-headlines` as a Boolean to
highlight agenda headlines using org default faces.

Headlines are highlighted by setting the corresponding
`face` text-property to each part of the headline. Issue emacs-dashboard#264 shows
that headlines are highlighted only after visit the org buffer and
when org buffer is killed headlines looses their faces.

I couldn't found the function that sets the faces, so now
faces are assigned to the TODO and HEADING parts only if they don't
already had a face.
@jcs090218 jcs090218 assigned jcs090218 and unassigned jcs090218 Dec 29, 2020
@jcs090218 jcs090218 merged commit c4283b7 into emacs-dashboard:master Dec 29, 2020
@jcs090218
Copy link
Member

LGTM, thanks for making this package better! ;)

@ricardoricho ricardoricho deleted the highlight branch December 31, 2020 03:20
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.

None yet

2 participants