{"action":"created","issue":{"url":"https://api.github.com/repos/Virtual-TreeView/Virtual-TreeView/issues/458","labels_url":"https://api.github.com/repos/Virtual-TreeView/Virtual-TreeView/issues/458/labels{/name}","comments_url":"https://api.github.com/repos/Virtual-TreeView/Virtual-TreeView/issues/458/comments","events_url":"https://api.github.com/repos/Virtual-TreeView/Virtual-TreeView/issues/458/events","html_url":"https://github.com/Virtual-TreeView/Virtual-TreeView/issues/458","id":54836474,"number":458,"title":"Virtual TreeView should not react on standard actions for TEdit controls","user":{"login":"vincentparrett","id":1037262,"avatar_url":"https://avatars.githubusercontent.com/u/1037262?v=3","gravatar_id":"","url":"https://api.github.com/users/vincentparrett","html_url":"https://github.com/vincentparrett","followers_url":"https://api.github.com/users/vincentparrett/followers","following_url":"https://api.github.com/users/vincentparrett/following{/other_user}","gists_url":"https://api.github.com/users/vincentparrett/gists{/gist_id}","starred_url":"https://api.github.com/users/vincentparrett/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/vincentparrett/subscriptions","organizations_url":"https://api.github.com/users/vincentparrett/orgs","repos_url":"https://api.github.com/users/vincentparrett/repos","events_url":"https://api.github.com/users/vincentparrett/events{/privacy}","received_events_url":"https://api.github.com/users/vincentparrett/received_events","type":"User","site_admin":false},"labels":[{"url":"https://api.github.com/repos/Virtual-TreeView/Virtual-TreeView/labels/bug","name":"bug","color":"fc2929"},{"url":"https://api.github.com/repos/Virtual-TreeView/Virtual-TreeView/labels/imported","name":"imported","color":"FFFFFF"}],"state":"open","locked":false,"assignee":null,"milestone":null,"comments":16,"created_at":"2015-01-20T02:30:04Z","updated_at":"2015-01-20T02:31:24Z","closed_at":null,"body":"_From [CWBudde](https://code.google.com/u/CWBudde/) on August 06, 2014 06:50:10_\n\nCan you attach a sample project that replicates the problem, or can you provide detailed steps how to use or modify one of the demo projects that come with VirtualTreeView?\r\n\r\nAdd a TEditDelete action to the project with a hotkey assigned to [DEL] key. Now focus a node in the virtual treeview and hit [DEL]. As expected, the node is deleted.\r\n\r\nNow if one wants to add a warning dialog like 'Really delete this node?, it has to be done in the TEditDelete action handler. While this works, it can be complicated if the same action is shared by other components (ExecuteAction is handled automatically by the component, which has the focus).\r\n\r\nWhat is the expected output? What do you want to see instead?\r\n\r\nIt would be nice to have a way to filter/cancel the nodes marked for deletion. Something like an OnNodeDeleting (similar to OnNodeCopying). Performance wise this might not be worse as it could be added in a higher level (just after the action is called) What version of the product are you using? On what operating system? I'm using the trunk Can you supply a patch (most appreciated would be a patch file created with TortoiseSVN)? This will significantly increase the likelyhood that the fix is included in the next release. I can do the change and create a patch if this is something that is not possible otherwise (I'm no expert when it comes to VTV, but I couldn't see any chance for this so far).\n\n_Original issue: http://code.google.com/p/virtual-treeview/issues/detail?id=457_"},"comment":{"url":"https://api.github.com/repos/Virtual-TreeView/Virtual-TreeView/issues/comments/70595248","html_url":"https://github.com/Virtual-TreeView/Virtual-TreeView/issues/458#issuecomment-70595248","issue_url":"https://api.github.com/repos/Virtual-TreeView/Virtual-TreeView/issues/458","id":70595248,"user":{"login":"vincentparrett","id":1037262,"avatar_url":"https://avatars.githubusercontent.com/u/1037262?v=3","gravatar_id":"","url":"https://api.github.com/users/vincentparrett","html_url":"https://github.com/vincentparrett","followers_url":"https://api.github.com/users/vincentparrett/followers","following_url":"https://api.github.com/users/vincentparrett/following{/other_user}","gists_url":"https://api.github.com/users/vincentparrett/gists{/gist_id}","starred_url":"https://api.github.com/users/vincentparrett/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/vincentparrett/subscriptions","organizations_url":"https://api.github.com/users/vincentparrett/orgs","repos_url":"https://api.github.com/users/vincentparrett/repos","events_url":"https://api.github.com/users/vincentparrett/events{/privacy}","received_events_url":"https://api.github.com/users/vincentparrett/received_events","type":"User","site_admin":false},"created_at":"2015-01-20T02:31:24Z","updated_at":"2015-01-20T02:31:24Z","body":"_From [CWBudde](https://code.google.com/u/CWBudde/) on September 15, 2014 04:25:55_\n\n\\>>  which only states 'edit control' and not TCustomEdit or TEdit.\r\n\r\n\\> And Virtual treeview clearly is not an edit control, while SynEdit is.\r\n\r\nI agree that it's probably not the typical edit control, but since 'edit' does not imply 'text edit', I would not exclude it.\r\n\r\n\r\n\\>> It could also be useful to have an option which reads 'DisableAutoActionHandling'\r\n\\>> or similar\r\n\r\n\\> When this issue is implemented, this will be handled by the actions, like it is\r\n\\> done in similar cases in the VCL. So there won't be the need for such a property,\r\n\\> and it doesn't make sense to add it just for a few weeks.\r\n\r\nI'd rather have in mind to have this option still in place for the existing code. This means the existing code should stay in place, as I don't consider this as a defect (see above).\r\n\r\nIn the end it might boil down to the understanding of the word edit (see http://www.thesaurus.com/browse/edit ), which I understand probably a bit wider ('bearbeiten' in the German language).\r\n\r\nThis however is opposed to a static control, only serving for display purpose (such as a TLabel). While a treeview (containing at least 'view') is typical (historically) used to view a tree, I can however also understand your point of view."}}