search

sabof / svg-mode-line-themes

SVG-based themes for mode-line
45 stars 3 forks source link

Add user error message for smt/enable|next-theme|set-theme when SVG not available #8

Closed jasonm23 closed 9 years ago

sabof commented 9 years ago

Perhaps use something like (display-warning 'svg-mode-line-themes "SVG support is not available") followed by return in smt/enable? An error may potentially halt the loading of an init file, which seems undesirable.

I'm also not sure that adding checks in something other than smt/enable is necessary.

jasonm23 commented 9 years ago

Seems fair, I thought user-error didn't halt anything other than the current defun, but return-from sounds safer with a warning.

jasonm23 commented 9 years ago

updated, take a look.

sabof commented 9 years ago

Merged