Rated 4 out of 5 stars

by mrmr on Oct. 9, 2020 ·

Actually, looking at implementation.js, turns out it is pretty simple change to allow existing syntax with multiple commands, and not needing custom functions:

diff --git a/addon/implementation.js b/addon/implementation.js
index 5a740f2..f44bf40 100644
--- a/addon/implementation.js
+++ b/addon/implementation.js
@@ -143,7 +143,9 @@ var TBKeys = {
let cmdBody = command.slice(cmdType.length + 1)
switch (cmdType) {
case "cmd":
- window.goDoCommand(cmdBody)
+ for (let cmd of cmdBody.split(" ")) {
+ window.goDoCommand(cmd)
+ }
break
case "func":
window[cmdBody]()


Then, one can just do:
"n": "cmd:cmd_collapseAllThreads cmd_nextMsg",

-mr

This review is for a previous version of the add-on (2.1.0). 

Rated 4 out of 5 stars

by mrmr on Oct. 8, 2020 ·

Thanks, but I prefer not having to build my own, unless there is now way to specify multiple cmds in the addon itself.

So, if you or the author confirms there is now way to do that, I will try building myself. Thanks.

This review is for a previous version of the add-on (2.1.0). 

Rated 5 out of 5 stars

by mrmr on Oct. 7, 2020 ·

Great app, I would not be using tb without it.
One question, how do I define multiple commands? I tried:

"D": "cmd:cmd_selectThread,cmd:cmd_delete",

in various ways, like using ; instead of comma, space in between etc.. nothing works. In tb60, I had used dorando_keyconfig and was able to accomplish the same.

Thanks.

This review is for a previous version of the add-on (2.1.0).