View Issue Details
|ID||Project||Category||View Status||Date Submitted||Last Update|
|0001855||Ham Radio Deluxe||Bug||public||2016-01-11 03:16||2017-09-18 00:08|
|Target Version||Fixed in Version||188.8.131.527|
|Summary||0001855: Neither of Ctrl-V or Shift-Ins work for callsign field in logbook filter window.|
|Description||Neither of Ctrl-V or Shift-Ins work for callsign field in logbook filter window.|
Right-click and then 'Paste' option works.
|Steps To Reproduce||ClLick on the Filter button|
try to paste in a callsign to the callsign text box using keyboard shortcuts.
Only Right click>paste seem to work.
|Tags||No tags attached.|
||I reproduced it tonight. Not only does Ctrl-V not work, it also can cause a crash if someone tries it.|
The problem is that CTRL+V and Shift+INS end up being routed to the frame window, not the edit control. On the frame, they cause the application to (try to) paste a row into the logbook directly.
There's a lot of undesired behaviour here:
1) command routing isn't working correctly. It'll be hard to fix because there are lots of windows involved, in two or three layers. Focus won't be easy to track down or correctly fix.
2) Even when we intend to paste into the logbook, the code will add the new row and then completely drain memory of all records, then re-load the database. This takes an unacceptably long time.
3) Same problem with cut commands (like CTRL+X)
I think the issue may be compounded by the design of the window layout, which hides and shows the filter bar rather than creating and destroying it.
If you're able to create a crash, please provide a minidump.
Looks like this is caused by the main frame window having an accelerator that includes CTRL+V, CTRL+INS, CTRL+X, and so on. PreTranslateMessage ends up interpreting the accelerator and routing the the command to the main frame, always.
I'll have to figure out if the easiest fix is to ditch the entries in the accelerator table; doing so will let the edit control handle its own focused keystrokes. But then we'll need a hook to handle the keystrokes for the desired row-by-row copy pasta.
Fixed with this change set
||Closed as part of the 184.108.40.2067 release.|
|2017-03-05 18:22||WA9PIE||Note Added: 0003100|
|2017-08-16 16:50||WA9PIE||View Status||private => public|
|2017-09-04 20:15||K7ZCZ||Note Added: 0004107|
|2017-09-04 21:41||K7ZCZ||Note Added: 0004108|
|2017-09-04 21:41||K7ZCZ||Project||2 - Next Dev List (Holding Area) => 3 - Current Dev List|
|2017-09-04 22:06||K7ZCZ||Assigned To||=> K7ZCZ|
|2017-09-04 22:06||K7ZCZ||Status||new => assigned|
|2017-09-07 16:46||K7ZCZ||Status||assigned => resolved|
|2017-09-07 16:46||K7ZCZ||Resolution||open => fixed|
|2017-09-07 16:46||K7ZCZ||Testing||=> Not Started|
|2017-09-07 16:46||K7ZCZ||Note Added: 0004132|
|2017-09-07 18:18||K7ZCZ||Fixed in Version||=> 220.127.116.114|
|2017-09-08 03:13||g3ucq||Note Added: 0004137|
|2017-09-11 21:32||K7ZCZ||Fixed in Version||18.104.22.1684 => 22.214.171.1245|
|2017-09-14 19:23||K7ZCZ||Fixed in Version||126.96.36.1995 => 188.8.131.527|
|2017-09-18 00:06||WA9PIE||Project||3 - Current Dev List => Ham Radio Deluxe|
|2017-09-18 00:08||WA9PIE||Note Added: 0004229|
|2017-09-18 00:08||WA9PIE||Status||resolved => closed|