{"action":"created","issue":{"url":"https://api.github.com/repos/ag-grid/ag-grid/issues/1915","repository_url":"https://api.github.com/repos/ag-grid/ag-grid","labels_url":"https://api.github.com/repos/ag-grid/ag-grid/issues/1915/labels{/name}","comments_url":"https://api.github.com/repos/ag-grid/ag-grid/issues/1915/comments","events_url":"https://api.github.com/repos/ag-grid/ag-grid/issues/1915/events","html_url":"https://github.com/ag-grid/ag-grid/issues/1915","id":257540060,"number":1915,"title":"Unable to suppress browser context menu as of 10.0.0","user":{"login":"Yoranium238","id":24553369,"avatar_url":"https://avatars0.githubusercontent.com/u/24553369?v=4","gravatar_id":"","url":"https://api.github.com/users/Yoranium238","html_url":"https://github.com/Yoranium238","followers_url":"https://api.github.com/users/Yoranium238/followers","following_url":"https://api.github.com/users/Yoranium238/following{/other_user}","gists_url":"https://api.github.com/users/Yoranium238/gists{/gist_id}","starred_url":"https://api.github.com/users/Yoranium238/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/Yoranium238/subscriptions","organizations_url":"https://api.github.com/users/Yoranium238/orgs","repos_url":"https://api.github.com/users/Yoranium238/repos","events_url":"https://api.github.com/users/Yoranium238/events{/privacy}","received_events_url":"https://api.github.com/users/Yoranium238/received_events","type":"User","site_admin":false},"labels":[{"id":160277035,"url":"https://api.github.com/repos/ag-grid/ag-grid/labels/bug","name":"bug","color":"fc2929","default":true}],"state":"closed","locked":false,"assignee":null,"assignees":[],"milestone":null,"comments":1,"created_at":"2017-09-13T22:26:10Z","updated_at":"2017-10-12T09:50:28Z","closed_at":"2017-10-12T09:50:28Z","author_association":"NONE","body":"**I'm submitting a ...**  (check one with \"x\")\r\n```\r\n[x ] bug report => search github for a similar issue or PR before submitting\r\n[ ] feature request\r\n[ ] support request => Please do not submit support request here, instead see https://github.com/ceolter/ag-grid/blob/master/CONTRIBUTING.md#question\r\n```\r\n\r\n**Current behavior**\r\nAs of ag-grid 10.0.0, we are no longer able to suppress the default context menu.  Prior to this (in 9.1.0), we were able to handle this suppression in the onCellContextMenu grid option.  Here, we used the event's preventDefault method to handle this and display a different context menu (non-ag-grid).\r\n\r\nAs a result, the default browser context menu displays upon right-click.  A second click displays our custom context menu.\r\n\r\ngridOptions.suppressContextMenu=true does not prevent this either.\r\n\r\n**We are using angular (4.2.4) as a development framework.**\r\n\r\n**Expected behavior**\r\nSelecting right-click for the context menu should display our context menu as it did previously without displaying the browser's default menu.\r\n\r\n**Minimal reproduction of the problem with instructions**\r\nThis is an example.  We actually show our (non ag-grid) context menu here.  The effect is the same.\r\n\r\n`\r\nthis.gridOptions.suppressContextMenu = true;\r\n\r\nthis.gridOptions.onCellContextMenu = (event: any) => {\r\n      event.event.cancelBubble = true;\r\n      event.event.preventDefault();\r\n      alert(\"I should see only this\");\r\n};\r\n`\r\n\r\n\r\n\r\n**What is the motivation / use case for changing the behavior?**\r\nIt appears to be a bug.  The result is an unusable user experience.\r\n\r\n**Please tell us about your environment:**\r\nWindows, IntelliJ, NodeJS, NPM, Angular (4.2.4), Tomcat, PrimeNg \r\n\r\n* **ag-Grid version:** 13.1.2 (ag-grid-angular: 13.1.1)\r\n\r\n\r\n* **Browser:**\r\nChrome, IE, Edge, Firefox\r\n\r\n* **Language:** [all | TypeScript 2.3.4 ]\r\n\r\n"},"comment":{"url":"https://api.github.com/repos/ag-grid/ag-grid/issues/comments/336079177","html_url":"https://github.com/ag-grid/ag-grid/issues/1915#issuecomment-336079177","issue_url":"https://api.github.com/repos/ag-grid/ag-grid/issues/1915","id":336079177,"user":{"login":"makinggoodsoftware","id":505400,"avatar_url":"https://avatars0.githubusercontent.com/u/505400?v=4","gravatar_id":"","url":"https://api.github.com/users/makinggoodsoftware","html_url":"https://github.com/makinggoodsoftware","followers_url":"https://api.github.com/users/makinggoodsoftware/followers","following_url":"https://api.github.com/users/makinggoodsoftware/following{/other_user}","gists_url":"https://api.github.com/users/makinggoodsoftware/gists{/gist_id}","starred_url":"https://api.github.com/users/makinggoodsoftware/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/makinggoodsoftware/subscriptions","organizations_url":"https://api.github.com/users/makinggoodsoftware/orgs","repos_url":"https://api.github.com/users/makinggoodsoftware/repos","events_url":"https://api.github.com/users/makinggoodsoftware/events{/privacy}","received_events_url":"https://api.github.com/users/makinggoodsoftware/received_events","type":"User","site_admin":false},"created_at":"2017-10-12T09:50:28Z","updated_at":"2017-10-12T09:50:28Z","author_association":"CONTRIBUTOR","body":"Hi guys,\r\n\r\nWe are going to close this one.\r\n\r\nHave a look at postProcessPopup(params) | Allows user to process popups after they are created. Applications can use this if they want to, for example, reposition the popup.\r\n-- | --\r\n\r\nYou should be able to hide the popup from there and do your own thing.\r\n\r\n\r\n"}}
{"actor":{"display_login":"makinggoodsoftware"}}