mrmr

À propos de moi

Information développeur
Nom mrmr
Utilisateur depuis oct. 7, 2020
Nombre de modules développés 0 modules
Moyenne des notes des modules du développeur Pas encore évalué

Mes critiques

New Email To All:reply, ready for new subject/body

Noté 3 sur 5 étoiles

>>yes, right click context menu is a good idea. I will add that and also keyboard activation.

If ++ doesn't work due to the other addon - why give 2 stars only to us, if it is them?
>>The button is inserted by the new TB addon system, not by us. I will send them a bug report.

I can't believe this site doesn't allow me to reply to a reply! Strange!
Anyways, first thank you for replying promptly. Well, we all, meaning my coworkers, use mail header addons which seem to be very popular. Once you add to either context menu and/or the mail toolbar, I promise to change it to 5 stars and also make a small donation :). Kindly post here when you are done. Thank you very much.

Cette critique concerne une version précédente du module (2.0).  Cet utilisateur a une critique précédente de ce module.

New Email To All:reply, ready for new subject/body

Noté 2 sur 5 étoiles

Great idea, but can't get it to work. After installing on tb 78.6.1, I see "++" in middle pane, but can't click on it (prob due to "message header toolbar customize" add on I have. Why can't it be put in the Mail Toolbar? Or right click context menu? (this on fedora 28).

Cette critique concerne une version précédente du module (2.0). 

tbkeys-lite

Noté 4 sur 5 étoiles

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

Cette critique concerne une version précédente du module (2.1.0).  Cet utilisateur a 2 critiques précédentes de ce module.

tbkeys-lite

Noté 4 sur 5 étoiles

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.

Cette critique concerne une version précédente du module (2.1.0).  Cet utilisateur a d’autres critiques de ce module.

tbkeys-lite

Noté 5 sur 5 étoiles

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.

Cette critique concerne une version précédente du module (2.1.0).