{"action":"created","issue":{"url":"https://api.github.com/repos/DoodleScheduling/android-material-chips/issues/8","repository_url":"https://api.github.com/repos/DoodleScheduling/android-material-chips","labels_url":"https://api.github.com/repos/DoodleScheduling/android-material-chips/issues/8/labels{/name}","comments_url":"https://api.github.com/repos/DoodleScheduling/android-material-chips/issues/8/comments","events_url":"https://api.github.com/repos/DoodleScheduling/android-material-chips/issues/8/events","html_url":"https://github.com/DoodleScheduling/android-material-chips/issues/8","id":141424842,"number":8,"title":"Fixed height ChipsView does not scroll","user":{"login":"jlindenbaum","id":197034,"avatar_url":"https://avatars.githubusercontent.com/u/197034?v=3","gravatar_id":"","url":"https://api.github.com/users/jlindenbaum","html_url":"https://github.com/jlindenbaum","followers_url":"https://api.github.com/users/jlindenbaum/followers","following_url":"https://api.github.com/users/jlindenbaum/following{/other_user}","gists_url":"https://api.github.com/users/jlindenbaum/gists{/gist_id}","starred_url":"https://api.github.com/users/jlindenbaum/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/jlindenbaum/subscriptions","organizations_url":"https://api.github.com/users/jlindenbaum/orgs","repos_url":"https://api.github.com/users/jlindenbaum/repos","events_url":"https://api.github.com/users/jlindenbaum/events{/privacy}","received_events_url":"https://api.github.com/users/jlindenbaum/received_events","type":"User","site_admin":false},"labels":[],"state":"closed","locked":false,"assignee":null,"milestone":null,"comments":2,"created_at":"2016-03-16T22:48:05Z","updated_at":"2016-04-08T15:59:40Z","closed_at":"2016-04-08T15:59:40Z","body":"When setting the `layout_height` to a fixed height, instead of `wrap_content` it's not possible to scroll through the chips added.\r\n\r\nEDIT\r\nIt's easy to wrap the `ChipsView` in a `ScrollView` with a fixed height, and setting the `ChipsView` to `match_parent`. This almost gives the desired result. An odd case where if the scroll view is not scrolled it is not possible to remove chips, the second \"remove\" tap is swallowed. A work around is to `fullScroll(VIEW.FOCUS_DOWN)` on the containing scroll view each time a chip is added in the `ChipsListener` call back. This properly allows the removal of chips again.\r\n\r\nI stepped through this in the debugger a bit, and it's a difficult problem to express. If the `ChipsView` is contained in a `ScrollView` the `unSelectChipsExcept()` function acts up. It always marks all chips as unselected, but the UI never updates to reflect that. The chip remains visually selected, but its selected flag is set to false. For whatever reason this is mitigated when the `ScrollView` is scrolled."},"comment":{"url":"https://api.github.com/repos/DoodleScheduling/android-material-chips/issues/comments/207493060","html_url":"https://github.com/DoodleScheduling/android-material-chips/issues/8#issuecomment-207493060","issue_url":"https://api.github.com/repos/DoodleScheduling/android-material-chips/issues/8","id":207493060,"user":{"login":"myacxy","id":4597425,"avatar_url":"https://avatars.githubusercontent.com/u/4597425?v=3","gravatar_id":"","url":"https://api.github.com/users/myacxy","html_url":"https://github.com/myacxy","followers_url":"https://api.github.com/users/myacxy/followers","following_url":"https://api.github.com/users/myacxy/following{/other_user}","gists_url":"https://api.github.com/users/myacxy/gists{/gist_id}","starred_url":"https://api.github.com/users/myacxy/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/myacxy/subscriptions","organizations_url":"https://api.github.com/users/myacxy/orgs","repos_url":"https://api.github.com/users/myacxy/repos","events_url":"https://api.github.com/users/myacxy/events{/privacy}","received_events_url":"https://api.github.com/users/myacxy/received_events","type":"User","site_admin":false},"created_at":"2016-04-08T15:59:40Z","updated_at":"2016-04-08T15:59:40Z","body":"Hey @jlindenbaum,\r\nthanks for your feedback. Due to your suggestions we decided to make the `ChipsView` itself a `ScrollView`. Therefore you should consider that you do not need to wrap it in your own `ScrollView` anymore after you upgrade to the new version and use `app:cv_max_height=\"${dimension}\"` instead."}}