-
Notifications
You must be signed in to change notification settings - Fork 3.4k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Less throws an error with the ">>>" combinator #2623
Comments
any update on this issue other than the answer above? |
I don't think so. Or will you submit a PR (adding the new combinator syntax and removing an old one) every time they change the draft? |
Well, this has been stable for over a year, so honestly saying "every time they change the draft' is a bit misleading |
@tigerhawkvok A year is just a twink in the CSS realm. I usually like to recall of CSS vars that were hanging there for decades and yet changed the syntax at Draft->CR transition almost at the last moment... |
Can we have |
Maybe less should consider about support >>> now? |
I'm with @seven-phases-max. There's still some debate on if shadow-piercing combinators are going to ever make it, since there are some that debate the philosophy of it (preferring locally-scoped styles). It's not tracked on caniuse.com, it doesn't have browser support, so that's why it's not supported (natively) in Less. But, as pointed out, it can be hacked in. I recommend closing this issue and re-opening only if this reaches CR status. |
I'm working on an angular app and believe we should start switching over to using Why not support |
@IEvangelist Regarding Angular stuff according to this post apparently you don't need |
::ng-deep is not working for me, because I'm dealing with a vue app, not an
angular app.
…On Wed, 30 Aug 2017, 05:37 Amin Pakseresht ***@***.***> wrote:
According to this post
<https://angular.io/guide/component-styles#deprecated-deep--and-ng-deep>
apparently you don't need >>> if you use ::ng-deep instead. And bottom
line is >>> or /deep/ will be deprecated from css as well as far as I
understood.
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
<#2623 (comment)>, or mute
the thread
<https://github.com/notifications/unsubscribe-auth/AFJBf4ZyVbcJ0spORtCXTC9aSEXCOqvkks5sdISggaJpZM4FOiQI>
.
|
@Jack-Works it's already explained for now you can use escape string as: @deep: ~'>>>';
// use @deep value in your selectors
.base-class @{deep} .deep-selector-class { } |
Closing until they make a final decision (according to latest rumors TODO: Open a generic issue to make combinator parsing more "relaxed" to accomodate some anticipated syntax (e.g. https://drafts.csswg.org/selectors-4/#descendant-combinators etc., possibly including even not existing things like |
@seven-phases-max I agree it would be forward-thinking to think of a a regex that would just capture any experimental combinators. I'm not sure there's a lot of reason for Less to be terribly discriminating with selectors. But it's true that browsers are intent now on removing shadow-piercing of any kind as an unwanted complexity. |
Eg,
However, this is valid.
See:
This should be correctly passed through.
The text was updated successfully, but these errors were encountered: