{"action":"created","issue":{"url":"https://api.github.com/repos/livecode/livecode/issues/3114","labels_url":"https://api.github.com/repos/livecode/livecode/issues/3114/labels{/name}","comments_url":"https://api.github.com/repos/livecode/livecode/issues/3114/comments","events_url":"https://api.github.com/repos/livecode/livecode/issues/3114/events","html_url":"https://github.com/livecode/livecode/pull/3114","id":113024809,"number":3114,"title":"[[ Bugfix 16272 ]] Ignore clicks to menu items that anchor submenus","user":{"login":"livecodefraser","id":5311346,"avatar_url":"https://avatars.githubusercontent.com/u/5311346?v=3","gravatar_id":"","url":"https://api.github.com/users/livecodefraser","html_url":"https://github.com/livecodefraser","followers_url":"https://api.github.com/users/livecodefraser/followers","following_url":"https://api.github.com/users/livecodefraser/following{/other_user}","gists_url":"https://api.github.com/users/livecodefraser/gists{/gist_id}","starred_url":"https://api.github.com/users/livecodefraser/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/livecodefraser/subscriptions","organizations_url":"https://api.github.com/users/livecodefraser/orgs","repos_url":"https://api.github.com/users/livecodefraser/repos","events_url":"https://api.github.com/users/livecodefraser/events{/privacy}","received_events_url":"https://api.github.com/users/livecodefraser/received_events","type":"User","site_admin":false},"labels":[{"url":"https://api.github.com/repos/livecode/livecode/labels/bug","name":"bug","color":"fc2929"}],"state":"open","locked":false,"assignee":null,"milestone":null,"comments":4,"created_at":"2015-10-23T13:58:49Z","updated_at":"2015-10-23T14:19:03Z","closed_at":null,"pull_request":{"url":"https://api.github.com/repos/livecode/livecode/pulls/3114","html_url":"https://github.com/livecode/livecode/pull/3114","diff_url":"https://github.com/livecode/livecode/pull/3114.diff","patch_url":"https://github.com/livecode/livecode/pull/3114.patch"},"body":"This brings the LiveCode behaviour more in line with the behaviour of modern UIs.\r\n\r\nThe existing behaviour (closing the menu when a submenu anchor is clicked on) is retained in the case that there is an \"on mouseRelease\" handler for the menu in order to avoid breaking existing code that captures clicks to the anchor.\r\n\r\nFor full UI compliance, clicks to the submenu anchor should open and close the submenu. However, the current menu code cannot handle this case without a larger rework of the code.\r\n\r\nI'm marking this as a WIP as I'm not sure checking for ES_NOT_HANDLED is the right way of going about this; in particular, should ES_PASS also do the same? @runrevmark is sure to have an opinion ;)"},"comment":{"url":"https://api.github.com/repos/livecode/livecode/issues/comments/150587371","html_url":"https://github.com/livecode/livecode/pull/3114#issuecomment-150587371","issue_url":"https://api.github.com/repos/livecode/livecode/issues/3114","id":150587371,"user":{"login":"runrevmark","id":3985298,"avatar_url":"https://avatars.githubusercontent.com/u/3985298?v=3","gravatar_id":"","url":"https://api.github.com/users/runrevmark","html_url":"https://github.com/runrevmark","followers_url":"https://api.github.com/users/runrevmark/followers","following_url":"https://api.github.com/users/runrevmark/following{/other_user}","gists_url":"https://api.github.com/users/runrevmark/gists{/gist_id}","starred_url":"https://api.github.com/users/runrevmark/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/runrevmark/subscriptions","organizations_url":"https://api.github.com/users/runrevmark/orgs","repos_url":"https://api.github.com/users/runrevmark/repos","events_url":"https://api.github.com/users/runrevmark/events{/privacy}","received_events_url":"https://api.github.com/users/runrevmark/received_events","type":"User","site_admin":false},"created_at":"2015-10-23T14:19:03Z","updated_at":"2015-10-23T14:19:03Z","body":"You should probably record the behavior in terms of message passing / not-passing in the release notes... I'm not sure where that info should go in the dictionary :S"}}