{"action":"created","issue":{"url":"https://api.github.com/repos/riot/riot/issues/2246","repository_url":"https://api.github.com/repos/riot/riot","labels_url":"https://api.github.com/repos/riot/riot/issues/2246/labels{/name}","comments_url":"https://api.github.com/repos/riot/riot/issues/2246/comments","events_url":"https://api.github.com/repos/riot/riot/issues/2246/events","html_url":"https://github.com/riot/riot/issues/2246","id":205369076,"number":2246,"title":"Attribute 'if' doesn't toggle visibility","user":{"login":"bmikeb","id":3843642,"avatar_url":"https://avatars.githubusercontent.com/u/3843642?v=3","gravatar_id":"","url":"https://api.github.com/users/bmikeb","html_url":"https://github.com/bmikeb","followers_url":"https://api.github.com/users/bmikeb/followers","following_url":"https://api.github.com/users/bmikeb/following{/other_user}","gists_url":"https://api.github.com/users/bmikeb/gists{/gist_id}","starred_url":"https://api.github.com/users/bmikeb/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/bmikeb/subscriptions","organizations_url":"https://api.github.com/users/bmikeb/orgs","repos_url":"https://api.github.com/users/bmikeb/repos","events_url":"https://api.github.com/users/bmikeb/events{/privacy}","received_events_url":"https://api.github.com/users/bmikeb/received_events","type":"User","site_admin":false},"labels":[{"id":211217517,"url":"https://api.github.com/repos/riot/riot/labels/enhancement","name":"enhancement","color":"eb6420","default":true}],"state":"closed","locked":false,"assignee":null,"assignees":[],"milestone":null,"comments":2,"created_at":"2017-02-04T19:01:58Z","updated_at":"2017-02-05T17:16:42Z","closed_at":"2017-02-05T17:16:42Z","body":"## Help us to manage our issues by answering the following:\r\n\r\n1. Describe your issue:\r\nDOM element must be hidden provided `if={false}`\r\nHowever its not the case\r\n\r\n\r\n2. Can you reproduce the issue?\r\n\r\nSee the link\r\nhttp://plnkr.co/edit/sZ4TeoFC8DTybwsKuRFS\r\n\r\n3. On which browser/OS does the issue appear?\r\nMac OS, Chrome\r\n\r\n4. Which version of Riot does it affect?\r\nver 3 only. On ver.2 it works fine.\r\n\r\n5. How would you tag this issue?\r\n  - [x] Bug\r\n"},"comment":{"url":"https://api.github.com/repos/riot/riot/issues/comments/277533692","html_url":"https://github.com/riot/riot/issues/2246#issuecomment-277533692","issue_url":"https://api.github.com/repos/riot/riot/issues/2246","id":277533692,"user":{"login":"GianlucaGuarini","id":879612,"avatar_url":"https://avatars.githubusercontent.com/u/879612?v=3","gravatar_id":"","url":"https://api.github.com/users/GianlucaGuarini","html_url":"https://github.com/GianlucaGuarini","followers_url":"https://api.github.com/users/GianlucaGuarini/followers","following_url":"https://api.github.com/users/GianlucaGuarini/following{/other_user}","gists_url":"https://api.github.com/users/GianlucaGuarini/gists{/gist_id}","starred_url":"https://api.github.com/users/GianlucaGuarini/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/GianlucaGuarini/subscriptions","organizations_url":"https://api.github.com/users/GianlucaGuarini/orgs","repos_url":"https://api.github.com/users/GianlucaGuarini/repos","events_url":"https://api.github.com/users/GianlucaGuarini/events{/privacy}","received_events_url":"https://api.github.com/users/GianlucaGuarini/received_events","type":"User","site_admin":false},"created_at":"2017-02-05T17:16:42Z","updated_at":"2017-02-05T17:16:42Z","body":"In riot@3 the `if` directive if false will no longer mount the tags so what you are trying to do it will not work. What you are asking might be implemented doing the following steps:\r\n\r\n1. mount the tag\r\n2. parse the expressions\r\n3. find out that the `if` directive is false\r\n4. unmount the tag\r\n\r\nSince all these things will lead to really poor performances I would recommend to use a `show` directive if you only need to hide a child tag and an `if` on the parent tag if you want mount/unmount it depending on your condition result."}}
{"actor":{"display_login":"GianlucaGuarini"}}