Please describe me what’s going wrong such as errors you’ve got in detail as possible so that I can help you.
Okay, below code won’t work because unfortunately plugins can’t access to the app’s node_modules folder.
import foldCode from 'codemirror/addon/fold/foldcode'
import foldGutter from 'codemirror/addon/fold/foldgutter'
import foldMd from 'codemirror/addon/fold/markdown-fold.js'
So you have to include these files in your plugins.
Hope that helps.
Hey I tried to clean up the code a little bit as I don’t quite understand how babel works.
And I also included CodeMirror
However, it looks like the required() function is still not working.
When I console.log() from code-fold.js L23, the options are set correctly
However, going into the generated HTML
.CodeMirror-linenumber are there, but .CodeMirror-foldmarker aren’t.
What should I change there to make it work?
Okay, I’ll look into it next week.
Thanks so much!
Hi,
I looked into your code.
If you looked into the addons, you find that they require CodeMirror at <your_plugin>/node_modules/codemirror/lib/codemirror but they get a CodeMirror class object different from Inkdrop’s one because it is a module in a different path.
So the addons are being installed to another CodeMirror.
That’s why it doesn’t work.
A workaround would be to import the modules from the app’s node_module in order for the addons to use the same CodeMirror class object, like so:
const app = require('electron').remote.app;
const modulePath = app.getAppPath() + '/node_modules/'
require(modulePath + 'codemirror/addon/fold/foldcode.js');
require(modulePath + 'codemirror/addon/fold/foldgutter.js');
require(modulePath + 'codemirror/addon/fold/markdown-fold.js');
And please remove the dependency on ‘codemirror’ from your package.json.
I confirmed it works.
Hope that helps!
Yey! Made the changes and published it as a plugin! 
Thanks so much!
Great!
https://app.inkdrop.info/plugins/code-fold
Could it have a screenshot in the description so that people can understand the plugin? 
Added
Cool.
Thank you, Samantha!
Hey craftzdog,
I’m trying to improve the plugin by adding foldAll() and unfoldAll() functionalities, but I couldn’t get keymaps to work right: https://github.com/moyicat/inkdrop-code-fold/commit/7d88da28f90ceffd68555d916cc174581efd1e9b
What am I missing there? Can you help? Thanks!
Hi Samantha,
Hm, it looks nice but I guess the command name should be codefold:fold-all / codefold:unfold-all.
Maybe export-print plugin would be helpful as an example.
I think the command is only triggered while you are on the editor, so it’d be better if selectors for keymaps are like:
{
".platform-darwin .CodeMirror": {
"cmd-r": "codefold:fold-all",
"cmd-t": "codefold:unfold-all"
},
".platform-linux .CodeMirror": {
"ctrl-r": "codefold:fold-all",
"ctrl-t": "codefold:unfold-all"
},
".platform-win32 .CodeMirror": {
"ctrl-r": "codefold:fold-all",
"ctrl-t": "codefold:unfold-all"
}
}
And I guess cmd-r and cmd-t are kind of too general key combination because it’s often used for reloading, running, compiling, tabs, etc.
So other key combination would be nice like cmd-shift-o / cmd-shift-u.
Hope that helps!
Made an update: https://github.com/moyicat/inkdrop-code-fold/commit/e7f1d9cbe356a1eafebf7a3c9c31c71581abc805
But I’m still not seeing keymap here:
Hi,
Thanks for the information.
I tried your plugin and found that keybindings are correctly loaded:
But there are two problems.
First, you should name commands starting with code-fold: which is your exact plugin name. Because the app tries to find commands that start with "<your plugin name>:" on your plugin detail screen. So please rename from codefold:fold-all to code-fold:fold-all.
Second, I found a bug that keybindings on the plugin screen doesn’t show them correctly if a selector has a platform. I’ll fix it in the next release!
Thanks again.
Gotcha! Updated and published a new version
Great! And v3.20.2 has resolved the second issue. Thanks again!
It’s cool plugin! Thanks!
This plugin seems not to work anymore.
Unfortunately, Samantha isn’t active with maintaining the plugin at the moment.
Would you be interested in taking over it?
Good day Takuya!
Unfortunately, I already commited to doing @anon20920130 Toolbar plugin first.
Perhaps maybe I can just combine the two ideas by putting in a code “fold all/unfold all” button on the toolbar plugin as a future update once the developer licenses are open again.
