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

Feat: Add a variable to control whether face-today is applied #1

Merged
merged 1 commit into from
Jan 27, 2023

Conversation

vedang
Copy link
Contributor

@vedang vedang commented Jan 21, 2023

This variable gives me control on whether cfw:face-today is applied to the whole day view or not. This is important because I actually prefer the colors that calfw-blocks uses for rendering the calendar than the default used by emacs-calfw, and setting the new cfw:hightlight-today to nil lets me use it so.

This change has also been contributed back to emacs-calfw at: kiwanami/emacs-calfw#154

This variable gives me control on whether `cfw:face-today` is applied
to the whole day view or not. This is important because I actually
prefer the colors that `calfw-blocks` uses for rendering the calendar
than the default used by `emacs-calfw`, and setting the new
`cfw:hightlight-today` to nil lets me use it so.

This change has also been contributed back to `emacs-calfw` at:
kiwanami/emacs-calfw#154
vedang added a commit to vedang/emacs-up that referenced this pull request Jan 23, 2023
Visually looking at the day gives a far more visceral understanding of
what you plan to do today.

Note that I am using personal recipes for both these projects, until
the fixes I've made can be merged upstream. The relevant PRs are:

- kiwanami/emacs-calfw#154
- kiwanami/emacs-calfw#155
- ml729/calfw-blocks#1
- ml729/calfw-blocks#2
@ml729
Copy link
Owner

ml729 commented Jan 27, 2023

Looks good, thanks for the contribution!

@ml729 ml729 merged commit b83569f into ml729:master Jan 27, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants