Skip to content
This repository was archived by the owner on Dec 15, 2022. It is now read-only.

Stop using shadow DOM selectors#615

Merged
as-cii merged 3 commits intomasterfrom
sm-remove-shadow-dom-styles
Oct 17, 2016
Merged

Stop using shadow DOM selectors#615
as-cii merged 3 commits intomasterfrom
sm-remove-shadow-dom-styles

Conversation

@simurai
Copy link
Contributor

@simurai simurai commented Oct 8, 2016

We are in the process of removing the shadow DOM boundary from atom-text-editor elements. This pull request upgrades existing selectors so that they:

  • Stop using :host.
  • Stop using atom-text-editor::shadow.
  • Prepend syntax class names with syntax--.

TODO

@simurai simurai changed the title Stop using shadow DOM selectors [WIP] Stop using shadow DOM selectors Oct 8, 2016
@simurai
Copy link
Contributor Author

simurai commented Oct 8, 2016

@as-cii I removed :host from the TextMate theme converter, but I guess we still need to add the syntax-- prefix. Is it somewhere here?

@as-cii as-cii removed their assignment Oct 10, 2016
@as-cii
Copy link
Contributor

as-cii commented Oct 10, 2016

but I guess we still need to add the syntax-- prefix.

Great catch, @simurai! ✨ I have updated this pull request's description with a link of the first-mate pull request we'll need to merge in order to address this. Thanks!

@as-cii as-cii changed the title [WIP] Stop using shadow DOM selectors Stop using shadow DOM selectors Oct 10, 2016
@as-cii as-cii force-pushed the sm-remove-shadow-dom-styles branch from d898626 to 84f4310 Compare October 17, 2016 14:23
@as-cii
Copy link
Contributor

as-cii commented Oct 17, 2016

Tests are failing, but this seems to be the case on master as well. I'll go ahead and 🚢 this, as it's blocking the shadow DOM removal pull request. Thanks, @simurai!

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants