Fork me on GitHub
#emacs
<
2024-01-31
>
Ed18:01:25

Hey. Dumb question, but I upgraded emacs on one of my machines and the theme I had installed there is now generating an error. It's running this

(set-face-attribute (car face) nil :weight 'regular :height (cadr face) :foreground "#88AAAA" :box `(:line-width (0 . ,(caddr face)) :color "#1e1e1e" :style nil))
which is generating this error (error "Invalid face box" :line-width (0 . 8) :color "#1e1e1e" :style nil) If I remove the :color "#1e1e1e" :style nil the error goes away, but according to the docs
VALUE must be a property list of the form `(:line-width WIDTH
:color COLOR :style STYLE)'.  If a keyword/value pair is missing from
the property list, a default value will be used for the value, as
specified below.  WIDTH specifies the width of the lines to draw; it
defaults to 1.  If WIDTH is negative, the absolute value is the width
of the lines, and draw top/bottom lines inside the characters area,
not around it.  COLOR is the name of the color to draw in, default is
the foreground color of the face for simple boxes, and the background
color of the face for 3D boxes.  STYLE specifies whether a 3D box
should be draw.  If STYLE is `released-button', draw a box looking
like a released 3D button.  If STYLE is `pressed-button' draw a box
that appears like a pressed button.  If STYLE is nil, the default if
the property list doesn't contain a style specification, draw a 2D
box.
which seems valid to me. What am I missing?

John21:01:01

doom emacs? if so check the issues there

John21:01:11

there were some fixes and then i think something else changed again

Ed22:01:26

No. Vanilla Emacs. https://github.com/bbatsov/solarized-emacs/issues/447 ... Looks like other people are having some issues too... But I can't tell what I've done wrong in trying to fix it