Nicola Squartini

Hi @tista500, Thank you for looking into this issue. 1 and 2. The original GTK2 code of Electron only had one background color, picked with: ``` GtkWidget* menu_bar = gtk_menu_bar_new();...

It's a good idea! Right now Atom looks for dictionaries in `/usr/lib/atom/node_modules/spellchecker/vendor/hunspell_dictionaries` ([code here](https://github.com/atom/node-spellchecker/blob/master/lib/spellchecker.js#L61)). I could make a patch and make it point at `/usr/share/hunspell` which is where Arch installs...

Well, then there is no need to patch. So the question is whether a package that has spellchecking features should have dictionaries among its optional dependencies. Seems like the arch-dev...

Thanks for reporting, I'll look into it.

@fusion809 thanks for reporting the issue. I tried `terminal-plus` right now and I get the exact same problem. I'm looking into it.

It must be related to the fact that we use electron 1.2.1, so the headers are different.

`npm` is system `/usr/bin/npm` `apm` uses its own npm in `/usr/lib/node_modules/atom-package-manager/node_modules/npm`

Sorry, I meant `/usr/lib/node_modules/atom-package-manager/node_modules/npm/bin/npm-cli.js`

It works if you upgrade `nan` to the latest version: ``` cd $HOME/.atom/packages/terminal-plus/node_modules/pty.js ``` in `package.json` change `"nan": "2.0.5"` into `"nan": "2.3.5"` and then run `npm install`. We are compiling...

In fact, it's safer to run `apm install` rather than `npm install` in `$HOME/.atom/packages/terminal-plus/node_modules/pty.js`. This way, it will compile against Electron's headers.