{"action":"closed","issue":{"url":"https://api.github.com/repos/WrenSecurity/wrenidm/issues/8","repository_url":"https://api.github.com/repos/WrenSecurity/wrenidm","labels_url":"https://api.github.com/repos/WrenSecurity/wrenidm/issues/8/labels{/name}","comments_url":"https://api.github.com/repos/WrenSecurity/wrenidm/issues/8/comments","events_url":"https://api.github.com/repos/WrenSecurity/wrenidm/issues/8/events","html_url":"https://github.com/WrenSecurity/wrenidm/issues/8","id":247694907,"number":8,"title":"Backgrid: Clicking on filter reset button sorts the column","user":{"login":"smolafilip","id":29750221,"avatar_url":"https://avatars2.githubusercontent.com/u/29750221?v=4","gravatar_id":"","url":"https://api.github.com/users/smolafilip","html_url":"https://github.com/smolafilip","followers_url":"https://api.github.com/users/smolafilip/followers","following_url":"https://api.github.com/users/smolafilip/following{/other_user}","gists_url":"https://api.github.com/users/smolafilip/gists{/gist_id}","starred_url":"https://api.github.com/users/smolafilip/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/smolafilip/subscriptions","organizations_url":"https://api.github.com/users/smolafilip/orgs","repos_url":"https://api.github.com/users/smolafilip/repos","events_url":"https://api.github.com/users/smolafilip/events{/privacy}","received_events_url":"https://api.github.com/users/smolafilip/received_events","type":"User","site_admin":false},"labels":[{"id":634687584,"url":"https://api.github.com/repos/WrenSecurity/wrenidm/labels/bug","name":"bug","color":"ee0701","default":true}],"state":"closed","locked":false,"assignee":{"login":"karelmaxa","id":13997406,"avatar_url":"https://avatars1.githubusercontent.com/u/13997406?v=4","gravatar_id":"","url":"https://api.github.com/users/karelmaxa","html_url":"https://github.com/karelmaxa","followers_url":"https://api.github.com/users/karelmaxa/followers","following_url":"https://api.github.com/users/karelmaxa/following{/other_user}","gists_url":"https://api.github.com/users/karelmaxa/gists{/gist_id}","starred_url":"https://api.github.com/users/karelmaxa/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/karelmaxa/subscriptions","organizations_url":"https://api.github.com/users/karelmaxa/orgs","repos_url":"https://api.github.com/users/karelmaxa/repos","events_url":"https://api.github.com/users/karelmaxa/events{/privacy}","received_events_url":"https://api.github.com/users/karelmaxa/received_events","type":"User","site_admin":false},"assignees":[{"login":"karelmaxa","id":13997406,"avatar_url":"https://avatars1.githubusercontent.com/u/13997406?v=4","gravatar_id":"","url":"https://api.github.com/users/karelmaxa","html_url":"https://github.com/karelmaxa","followers_url":"https://api.github.com/users/karelmaxa/followers","following_url":"https://api.github.com/users/karelmaxa/following{/other_user}","gists_url":"https://api.github.com/users/karelmaxa/gists{/gist_id}","starred_url":"https://api.github.com/users/karelmaxa/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/karelmaxa/subscriptions","organizations_url":"https://api.github.com/users/karelmaxa/orgs","repos_url":"https://api.github.com/users/karelmaxa/repos","events_url":"https://api.github.com/users/karelmaxa/events{/privacy}","received_events_url":"https://api.github.com/users/karelmaxa/received_events","type":"User","site_admin":false}],"milestone":null,"comments":0,"created_at":"2017-08-03T12:47:01Z","updated_at":"2018-02-26T01:01:17Z","closed_at":"2018-02-26T01:01:17Z","author_association":"NONE","body":"If you click on filter reset button, the click is propagated to the column header and it is interpreted as a sort request. This degrades user experience in two ways: it is confusing (user might not notice that the sorting took place), and it makes two requests (one for changed filter and one for sort request).\r\n\r\nA bit of background we were able to get:\r\n* Backgrid handles any click on `<a>` element (see https://github.com/wyuenho/backgrid/blob/master/src/header.js#L24) as a sorting request.\r\n* Commons UI contains `ThemeableServerSideFilter` with `<a>` as reset button - this component probably has to catch click event and stop the propagation.\r\n* It might be also viewed as a bug of the parent component (`ServerSideFilter`), which does not prevent event bubbling (see https://github.com/wyuenho/backgrid-filter/blob/master/backgrid-filter.js#L189)... but as I understand that component, it was never intended to be included in the column header.\r\n\r\nThere is an issue for this on the FR Bugster: [OPENIDM-5851](https://bugster.forgerock.org/jira/browse/OPENIDM-5851)"}}
{"actor":{"display_login":"Kortanul"}}