[Enter steps to reproduce:]
Atom: 1.21.0-beta0 x64
Electron: 1.6.9
OS: Mac OS X 10.12.6
Thrown From: atom-ide-ui package 0.3.0
Failed to activate the atom-ide-ui package
At variable @font-size is undefined in /Users/chris/.atom/packages/atom-ide-ui/modules/nuclide-commons-ui/styles/CodeSnippet.less:10:15
LessError: variable @font-size is undefined
at /packages/atom-ide-ui/modules/nuclide-commons-ui/styles/CodeSnippet.less:10:15
atom-ide-ui 0.3.0
close-on-left 0.9.2
goto-definition 1.3.4
language-gherkin 1.0.4
language-lua 0.9.11
xatom-syntax 0.3.0
xatom-ui 0.2.0
Thanks for the report. The @font-size variable seems to be something that's declared across all the default Atom themes and others like Material UI. I'm inclined to think that this is something xatom-ui should declare here:
https://github.com/willyelm/xatom-ui/blob/master/styles/ui-variables.less#L68
For comparison:
https://github.com/atom/one-dark-ui/blob/master/styles/ui-variables.less#L83
https://github.com/atom-material/atom-material-ui/blob/master/styles/ui-variables.less#L78
Although there's no official spec for what ui-variables should contain, this seems like a reasonably standard thing to have.
Admittedly, after digging a bit deeper and trying a few more 3rd party themes after submitting this issue it does appear you're right. This is probably an issue with Xatom. Thanks for quick response.
I get a slight variation on this:
LessError: variable @button-background-color-selected is undefined
at /Users/captbackfire/.atom/packages/atom-ide-ui/modules/nuclide-commons-ui/styles/Table.less:112:16
@almostsultry This is a problem with the theme you are using.
Have the same problem as well
Using Seti UI theme
and New Moon syntax theme
.atom\packages\atom-ide-ui\modules\nuclide-commons-ui\styles\TextEditorBanner.less:15:15
LessError: variable @font-family is undefined
at C:\Users\jonathan.yee\.atom\packages\atom-ide-ui\modules\nuclide-commons-ui\styles\TextEditorBanner.less:15:15
I'm also using this package fonts
@jonyeezs I'm using seti theme too. Switching to "Atom Dark" Theme resolved the issues for me.
@jonyeezs @dotmagic I just opened a PR on seti-ui which resolved the issue for me: https://github.com/jesseweed/seti-ui/pull/459
@forestgagnon , how? I'm using seti for the UI theme but I'm still getting the error
Hello
I get also this error. I use adwait pro ui theme (https://github.com/forivall/adwaita-pro-ui/). Error is come with latest atom-ide update
I got this error with a different variable. But my syntax theme already defines the variable:
LessError: variable @syntax-background-color is undefined
at /Users/parasyte/.atom/packages/atom-ide-ui/modules/nuclide-commons-ui/styles/LinePreview.less:6:23
The variable is defined here: https://github.com/parasyte/pretty-pastels-syntax/blob/master/styles/syntax-variables.less#L10
Found that this is still related to the UI theme, since mine doesn't import syntax-variables, I get this error. The core themes atom-dark-ui and atom-light-ui are affected by this bug as well.
The core themes atom-one-dark-ui and atom-one-light-ui both import syntax-variables, and they both work fine with atom-ide-ui.
Adding @import "syntax-variables"; to the UI theme is a workaround. But note there are core themes missing this.
Oh nice, there is a PR that fixes the issue I described -> https://github.com/facebook-atom/atom-ide-ui/pull/160 Just awaiting release.
This should be fixed here by https://github.com/facebook-atom/atom-ide-ui/pull/160.
Sorry, just published v0.8.1 to fix this!
Can confirm, I had the issue even though I use Atom Dark (for both window and syntax), updating atom-ide-ui to v0.8.1 fixed it. Nice work!
Most helpful comment
@jonyeezs @dotmagic I just opened a PR on seti-ui which resolved the issue for me: https://github.com/jesseweed/seti-ui/pull/459