{"action":"created","issue":{"url":"https://api.github.com/repos/racodond/sonar-css-plugin/issues/43","repository_url":"https://api.github.com/repos/racodond/sonar-css-plugin","labels_url":"https://api.github.com/repos/racodond/sonar-css-plugin/issues/43/labels{/name}","comments_url":"https://api.github.com/repos/racodond/sonar-css-plugin/issues/43/comments","events_url":"https://api.github.com/repos/racodond/sonar-css-plugin/issues/43/events","html_url":"https://github.com/racodond/sonar-css-plugin/issues/43","id":244258883,"number":43,"title":"/deep/ parse error","user":{"login":"HiperSoke","id":651821,"avatar_url":"https://avatars1.githubusercontent.com/u/651821?v=4","gravatar_id":"","url":"https://api.github.com/users/HiperSoke","html_url":"https://github.com/HiperSoke","followers_url":"https://api.github.com/users/HiperSoke/followers","following_url":"https://api.github.com/users/HiperSoke/following{/other_user}","gists_url":"https://api.github.com/users/HiperSoke/gists{/gist_id}","starred_url":"https://api.github.com/users/HiperSoke/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/HiperSoke/subscriptions","organizations_url":"https://api.github.com/users/HiperSoke/orgs","repos_url":"https://api.github.com/users/HiperSoke/repos","events_url":"https://api.github.com/users/HiperSoke/events{/privacy}","received_events_url":"https://api.github.com/users/HiperSoke/received_events","type":"User","site_admin":false},"labels":[{"id":551422874,"url":"https://api.github.com/repos/racodond/sonar-css-plugin/labels/question","name":"question","color":"cc317c","default":true}],"state":"open","locked":false,"assignee":null,"assignees":[],"milestone":null,"comments":3,"created_at":"2017-07-20T06:34:07Z","updated_at":"2017-07-21T11:12:51Z","closed_at":null,"body":"Hello,\r\n\r\nI saw that a few versions ago it was added support to /deep/ combinator, but for me is not working. I'm using version 4.7\r\nCode:\r\n` /deep/ .btngroupstyle {\r\n    background-color: black;\r\n    border-color: #8cd1fa;\r\n    border-style: solid;\r\n    border-width: 1.5px;\r\n    height: 35px;\r\n} `\r\nError:\r\n` Parse error at line 44 column 1: 34: border-color: #8cd1fa; 35: border-radius: 5px; 36: border-style: solid; 37: border-width: 1.5px; 38: height: 35px; 39: margin-bottom: 10px; 40: text-align: left; 41: width: 70px; 42: } 43: 44: /deep/ .btngroupstyle { ^ 45: background-color: #ffffff; 46: border-color: #8cd1fa; 47: border-style: solid; 48: border-width: 1.5px; 49: height: 35px; 50: } 51: 52: .buttonload { 53: background-color: #0098f3; 54: color: #ffffff; `\r\n\r\nRegards.\r\n"},"comment":{"url":"https://api.github.com/repos/racodond/sonar-css-plugin/issues/comments/316974828","html_url":"https://github.com/racodond/sonar-css-plugin/issues/43#issuecomment-316974828","issue_url":"https://api.github.com/repos/racodond/sonar-css-plugin/issues/43","id":316974828,"user":{"login":"racodond","id":1215713,"avatar_url":"https://avatars2.githubusercontent.com/u/1215713?v=4","gravatar_id":"","url":"https://api.github.com/users/racodond","html_url":"https://github.com/racodond","followers_url":"https://api.github.com/users/racodond/followers","following_url":"https://api.github.com/users/racodond/following{/other_user}","gists_url":"https://api.github.com/users/racodond/gists{/gist_id}","starred_url":"https://api.github.com/users/racodond/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/racodond/subscriptions","organizations_url":"https://api.github.com/users/racodond/orgs","repos_url":"https://api.github.com/users/racodond/repos","events_url":"https://api.github.com/users/racodond/events{/privacy}","received_events_url":"https://api.github.com/users/racodond/received_events","type":"User","site_admin":false},"created_at":"2017-07-21T11:12:51Z","updated_at":"2017-07-21T11:12:51Z","body":"> I changed the /deep/ combinator to ::ng-deep as is the one that we should use for Angular 4 projects and seems that is isnt supported by the plugin\r\n\r\n`/deep/` is considered as a CSS combinator. Thus, in CSS it cannot be the first element of a selector. Hence, the parse error you get.\r\n\r\nUnlike CSS, it is possible to have a CSS combinator as first element of a selector in SCSS. Hence the support of `/deep/` in SCSS when it is the first element of a selector.\r\n\r\nI'm not familiar with `/deep/`. Thus, it is really possible to have it as first element of a selector? If so, it'll require a change in the grammar.\r\n\r\n`::ng-deep` is considered as a pseudo-element, that's why it is properly parsed.\r\n\r\n> I think that maybe you could add ::ng-deep as a deprecated combinator\r\n\r\nOK. `::ng-deep` is considered as a pseudo-element for now. Does it make sense to mark it as obsolete/deprecated in rule \"Obsolete pseudo-elements and pseudo-classes should not be used\". Or doesn't it make sense to consider it as a pseudo-element in an Anglular context? And it should really be considered as combinator in terms of wording?\r\n\r\n"}}
{"actor":{"display_login":"racodond"}}