Skip to content
This repository was archived by the owner on Apr 12, 2024. It is now read-only.

chore(styleDirective): remove an unneccessary directive definition #14983

Merged
merged 1 commit into from
Aug 8, 2016

Conversation

dcherman
Copy link
Contributor

@dcherman dcherman commented Aug 3, 2016

What kind of change does this PR introduce? (Bug fix, feature, docs update, ...)

Chore / Cleanup

Does this PR introduce a breaking change?

Nope

Please check if the PR fulfills these requirements

Other information:

Since the style directive is defined as a non-terminal element directive
with no behavior on link, we may as well not define anything at all.

Since the style directive is defined as a non-terminal element directive
with no behavior on link, we may as well not define anything at all.
@gkalpak
Copy link
Member

gkalpak commented Aug 5, 2016

Nice catch!

Some background:

  • The styleDirective was originally added with d656d11 to prevent the contents of <style> tags from being processed by Angular (for whatever reason - possibly because it didn't work in older browsers).
  • This restriction was removed in 0609453 (since support for IE8 was removed and all supported browsers worked with interpolations in <style> elements. Instead of changing terminal to false, the directive could as well have been removed in 0609453.

@gkalpak
Copy link
Member

gkalpak commented Aug 5, 2016

LGTM

@Narretz
Copy link
Contributor

Narretz commented Aug 5, 2016

Wow, the CI finally passed! LGTM

@Narretz Narretz merged commit b58a7f8 into angular:master Aug 8, 2016
gkalpak pushed a commit that referenced this pull request Aug 8, 2016
Since the style directive is defined as a non-terminal element directive
with no behavior on link, we may as well not define anything at all.
PR (#14983)
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants