[faces] set all/only hl-todo
keywords which are present
#7
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
I load
hl-todo-mode
before my humanoid theme and add a new todo(keyword . face) pair to
hl-todo-keyword-faces
. I was frustrated fora while as to why this new pair didn't show up in actual operation.
The answer is that
humanoid-themes
overwriteshl-todo-keyword-faces
with a hardcoded value.All this is meant to achieve is to fix the faces in the list to the
relevant colours. This PR offers a more robust solution: for every
word in the actual list
hl-todo-keyword-faces
when the theme is set,if that word appears in a canonical alist in the theme code, set the
colour/face to the one in the canonical alist. Otherwise, leave it
alone. This retains edits (additions and deletions) the user makes,
but still fixes the colours.
Original commit message for sole commit: