-
Notifications
You must be signed in to change notification settings - Fork 129
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
Make icon display predicate customizeable #442
Make icon display predicate customizeable #442
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Some nits.
LGTM, thanks for taking care of this!
:type '(choice (function :tag "Predicate function") | ||
(boolean :tag "Predicate value"))) | ||
|
||
(defun dashboard-display-icons-p () |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Can you document this function? Thanks!
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Sure, just added a simple docstring.
dashboard-widgets.el
Outdated
to a function the result of the function will be interpreted as the | ||
predicate value." | ||
:type '(choice (function :tag "Predicate function") | ||
(boolean :tag "Predicate value"))) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
We're getting warning:
dashboard-widgets.el:198:1:Warning: defcustom for ‘dashboard-display-icons-p’
fails to specify containing group
dashboard-widgets.el:198:1:Warning: defcustom for ‘dashboard-display-icons-p’
fails to specify containing group
Can you add the group for it? :)
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM, Thank you so much! :D
Closes #333.
Adds a customizeable function to restrict icon insertion and default it to display-graphic-p to retain legacy behavior. This will let terminal users who have icons properly configured use dashboard with icons without needing local modifications.