Skip to content

Remove all uses of selector & to support nested import?#17352

Closed
glebm wants to merge 1 commit intov4-devfrom
avoid-parent-selector-suffix
Closed

Remove all uses of selector & to support nested import?#17352
glebm wants to merge 1 commit intov4-devfrom
avoid-parent-selector-suffix

Conversation

@glebm
Copy link
Copy Markdown
Member

@glebm glebm commented Aug 26, 2015

See sass/sass#1817 for the context on this.


If we want to support namespaced import, i.e.:

.twbs {
  @import 'bootstrap';
}

We cannot use selector &, see sass/sass#1817.
This is one ugly workaround, but it works.

Requires libsass 3.3 (due to selector-unify and selector-nest) which has not been released yet (but soon hopefully), so travis will fail.

If we want to support namespaced import:

    .twbs {
      @import 'bootstrap';
    }

We cannot use `selector &`, see sass/sass#1817.
This is one ugly workaround, but it works.

Requires libsass 3.3 which has not been released yet (but soon hopefully).
@cvrebert
Copy link
Copy Markdown
Collaborator

Ugh. I still think CSS postprocessing is the better way to do that. This pass-the-selector-as-a-string business is whack.

@glebm
Copy link
Copy Markdown
Member Author

glebm commented Aug 27, 2015

So.. I just tried disabling a fieldset, and all its children automatically become :disabled. I think we can just remove the fieldset[disabled] & selector, must have been something done for IE8-compat? /cc @mdo

@glebm
Copy link
Copy Markdown
Member Author

glebm commented Aug 27, 2015

Closing, opened #17364

@glebm glebm closed this Aug 27, 2015
@glebm glebm deleted the avoid-parent-selector-suffix branch August 31, 2015 00:44
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants