nostalgy-xpi icon indicating copy to clipboard operation
nostalgy-xpi copied to clipboard

3.0.7 interferes with (QNote++) plugins

Open DrBoym opened this issue 3 years ago • 18 comments

Hi there,

running TB 91.0.3, Nostalgy 3.0.7 and QNote++ 0.10.2 I end up with Nostalgy Hotkeys being active when editing a QNote++ entry for a mail or such. I expect that the same could happen with other plugins, not tested yet. Workarounds: A) Restrict command to email tabs works, but only after setting the option and restarting TB. B) Define hotkeys that are not used for typing text

Related to that is another issue: Hotkeys in Nostaly cannot be disabled anymore: Click on disable in Key menu does not have any effect what so ever.

DrBoym avatar Sep 01 '21 07:09 DrBoym

yes, A is the recommended solution for that, because at the moment, after a restartm TB does not recognise the mail window correctly.

opto avatar Sep 24 '21 18:09 opto

the second issue is a dublicate of #129

opto avatar Sep 24 '21 18:09 opto

yes, A is the recommended solution for that, because at the moment, after a restartm TB does not recognise the mail window correctly.

I am afraid in 3.0.8 but also 3.0.7 "restrict command to email tabs" does not solve this issue consistently for plugins like QNote. Thus this issue should probably not be closed as it requires a fix still. One could of cause use XNote (I guess @opto would agree ;-) ) as a workaround for QNote++.

DrBoym avatar Sep 27 '21 06:09 DrBoym

is the qnote edited in the main mail:3pane window? If yes, this will be diffucult to change, because normally, nobody edits in that window.

Can you reset the keys in options to something not used in the note?

opto avatar Sep 27 '21 20:09 opto

is the qnote edited in the main mail:3pane window? If yes, this will be diffucult to change, because normally, nobody edits in that window.

I am not 100% sure, but I would say no. It is an overlay that you edit in. Functionality is basically exactly the same as in XNote++, which I know you know way better than me @opto ;-). I only switched to QNote (really sorry for that!) a while back when TB 78 was released, as XNote didn't work for quiet a while and I had to get my old notes back in order to be able to work. QNote++ is capable of working on old notes taken with XNote or XNote++. Current XNote++ v3.2.4 on TB 91.1.2 does not show the bug described here. Typing text including a letter defined as a hotkey does not trigger Nostalgy, as it does in QNote++ currently.

Therefore, there is this workaround: I switched to use XNote++ again, but keep QNote++ running in parallel just to read my old QNotes with that. Of cause doing this is a bit annoying when you hit a mail with a new XNote++ it will show up twice, one note by XNote++ and the same one in a different GUI from QNote. Unfortunately XNote++ is not capable of reading/editing old QNote++ notes. Only the other way round...QNote is able to read XNote notes ^^

Maybe it is a nice feature idea for XNote++ to make it compatible with QNote notes? This way users could define the same storage path for notes in both plugins and work with that one, which is currently working best based on the TB Version used and user preference of cause. Wait... Yes, it would be nice, as others mentioned earlier ;-): https://github.com/xnotepp/xnote/issues/62

Can you reset the keys in options to something not used in the note?

In theory yes. But as you require all characters and numbers, as well as symbols, to write a meaningful note it is quiet difficult to define a set of useful hotkeys for Nostalgy. You might work with F1-12, but those are used for other plugins in my case (QuickText).

DrBoym avatar Sep 30 '21 10:09 DrBoym

I'm running into the same problem with QNote (https://addons.thunderbird.net/en-US/thunderbird/addon/qnote/?src=userprofile) and it would be great if XNote++ and QNote were interchangeable. That would fix my problem. For now, I have to type my note in Notepad++ and paste it into my QNote to avoid having the key presses stolen by Nostalgy++. If there was a way for Nostalgy++ to detect that the focus is in the QNote window and not steal the keypress, that would do it, but I have no idea if that is possible in the TB add-on world. Thanks for a great add-on!

SergeLalonde avatar Oct 17 '21 22:10 SergeLalonde

Maybe removing NostalgyStopEvent(ev); call after keypress event can resolve this?

mlazdans avatar Oct 18 '21 10:10 mlazdans

Is this repo up to date? I show latest update 11 months ago but 3.0.8 release 24 days ago...

mlazdans avatar Oct 18 '21 10:10 mlazdans

Version 3.2.16 of Nostalgy++ released (on TB 91.6.0) but this bug is still not corrected. I'm still not able to type QNotes while Nostalgy++ is active.

ThomasHeinrichSchmidt avatar Feb 15 '22 09:02 ThomasHeinrichSchmidt

It seems that changes in keystroke detection (perhaps in N++ or in TB or both) also have an impact on several other issues, like #121 #122, #127, #130

ThomasHeinrichSchmidt avatar Feb 15 '22 09:02 ThomasHeinrichSchmidt

see first post: A) Restrict command to email tabs works, but only after setting the option and restarting TB.

does that help? It should be the recommended way and is set as standard now, but old installations don't pick it up.

Klaus

opto avatar Feb 15 '22 09:02 opto

@ThomasHeinrichSchmidt

It seems that changes in keystroke detection (perhaps in N++ or in TB or both) also have an impact on several other issues, like #121 #122, #127, #130

#122, #127 is resolved, not working for you?

opto avatar Feb 15 '22 09:02 opto

I'm using "Restrict commands to email tab" all the time.

Nostalgy-Preferences

But as soon as I type a N++ command letter while editing a QNote (e.g. p q r s) N++ jumps in

QNote-typing-PQRS

ThomasHeinrichSchmidt avatar Feb 15 '22 09:02 ThomasHeinrichSchmidt

@ThomasHeinrichSchmidt

It seems that changes in keystroke detection (perhaps in N++ or in TB or both) also have an impact on several other issues, like #121 #122, #127, #130

#122, #127 is resolved, not working for you?

and #130 was also solved (for me) since 3.1.3 :-)

Olifair avatar Feb 15 '22 10:02 Olifair

I saw references to these issues when I read about problems similar to mine. Good if most of those problems have been resolved. But still I cannot type into QNote as I could before TB 91 and Nostalgy++ 3.0.7.

ThomasHeinrichSchmidt avatar Feb 15 '22 10:02 ThomasHeinrichSchmidt

I still have the same problem as @ThomasHeinrichSchmidt. The Restrict commands to email tab is on and makes no difference. I tried unchecking it, restarting, rechecking, restarting and there's no change. Is there really no fix for this? I installed XNote++ and it works fine. Unfortunately, I can't switch to it since it doesn't see all of the QNotes that I've already created. Keeping both installed is clumsy.

SergeLalonde avatar Mar 27 '22 17:03 SergeLalonde

In the end I tried to disable most of the N++ keys leaving just Ctrl+S and Shift+Ctrl+S for "Save message" and "Save as suggested" respectively. But this does not work either because N++ does not save the "disable" option. When I come back to the "Nostalgy Preferences" dialog all the keys are enabled again.

ThomasHeinrichSchmidt avatar Apr 13 '22 08:04 ThomasHeinrichSchmidt

Please add this on check to onNostalgyKeyPress(ev) function. I can not send merge request because this repo seems out to date compared with just released 3.3.19

...
function onNostalgyKeyPress(ev) {
  if(ev.target && ev.target.className == 'webextension-popup-browser'){
    return;
  }
....

mlazdans avatar Apr 15 '22 20:04 mlazdans