electron-userland / electron-spellchecker

Implement spellchecking, correctly
MIT License
237 stars 83 forks source link

Doesnt work on windows #93

Open m4rkmckenna opened 6 years ago

m4rkmckenna commented 6 years ago

The below works on mac mac but when run under windows fails ... am i missing something?

const {SpellCheckHandler, ContextMenuListener, ContextMenuBuilder} = require('electron-spellchecker');

window.spellCheckHandler = new SpellCheckHandler();
setTimeout(() => window.spellCheckHandler.attachToInput(), 1000);

window.spellCheckHandler.autoUnloadDictionariesOnBlur();
window.contextMenuBuilder = new ContextMenuBuilder(window.spellCheckHandler, null, true);
window.contextMenuListener = new ContextMenuListener((info) => {
    window.contextMenuBuilder.showPopupMenu(info);
});

This is added to render.js of electron-quick-start

The following is logged to the console onblur

E:\node_modules\rxjs\observable\PromiseObservable.js:76 Uncaught Error
    at MergeMapSubscriber.project (E:\node_modules\electron-remote\lib\renderer-require.js:34)
    at MergeMapSubscriber._tryNext (E:\node_modules\rxjs\operator\mergeMap.js:120)
    at MergeMapSubscriber._next (E:\node_modules\rxjs\operator\mergeMap.js:110)
    at MergeMapSubscriber.Subscriber.next (E:\node_modules\rxjs\Subscriber.js:89)
    at FilterSubscriber._next (E:\node_modules\rxjs\operator\filter.js:87)
    at FilterSubscriber.Subscriber.next (E:\node_modules\rxjs\Subscriber.js:89)
    at RefCountSubscriber.Subscriber._next (E:\node_modules\rxjs\Subscriber.js:125)
    at RefCountSubscriber.Subscriber.next (E:\node_modules\rxjs\Subscriber.js:89)
    at Subject.next (E:\node_modules\rxjs\Subject.js:55)
    at ConnectableSubscriber.Subscriber._next (E:\node_modules\rxjs\Subscriber.js:125)
(anonymous) @ E:\node_modules\electron-remote\lib\renderer-require.js:34
MergeMapSubscriber._tryNext @ E:\node_modules\rxjs\operator\mergeMap.js:120
MergeMapSubscriber._next @ E:\node_modules\rxjs\operator\mergeMap.js:110
Subscriber.next @ E:\node_modules\rxjs\Subscriber.js:89
FilterSubscriber._next @ E:\node_modules\rxjs\operator\filter.js:87
Subscriber.next @ E:\node_modules\rxjs\Subscriber.js:89
Subscriber._next @ E:\node_modules\rxjs\Subscriber.js:125
Subscriber.next @ E:\node_modules\rxjs\Subscriber.js:89
Subject.next @ E:\node_modules\rxjs\Subject.js:55
Subscriber._next @ E:\node_modules\rxjs\Subscriber.js:125
Subscriber.next @ E:\node_modules\rxjs\Subscriber.js:89
DoSubscriber._next @ E:\node_modules\rxjs\operator\do.js:88
Subscriber.next @ E:\node_modules\rxjs\Subscriber.js:89
(anonymous) @ E:\node_modules\rxjs\observable\FromEventObservable.js:132
emitTwo @ events.js:106
emit @ events.js:191
edmoresamewave commented 6 years ago

Hi, getting the same issue on Windows, except no errors in console. Any help would be appreciated.

snene commented 6 years ago

I am having issues on windows too. On right clicking the wrong word, the context menu appears momentarily and disappears immediately.