Skip to content
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

refactor(ivy): remove styling state storage and introduce direct style writing #32259

Closed
wants to merge 3 commits into from

Conversation

matsko
Copy link
Contributor

@matsko matsko commented Aug 22, 2019

This patch is a final major refactor in styling Angular.

This PR includes three main fixes:

  • All temporary state taht is persisted between template style/class application
    and style/class application in host bindings is now removed.
  • Removes the styling() and stylingApply() instructions.
  • Introduces a "direct apply" mode that is used apply prop-based
    style/class in the event that there are no map-based bindings as
    well as property collisions.

@atscott atscott added area: core Issues related to the framework runtime comp: ivy labels Aug 28, 2019
@ngbot ngbot bot modified the milestone: needsTriage Aug 28, 2019
@matsko matsko changed the title Final styling prototype refactor(ivy): remove styling state storage and introduce direct style writing Aug 29, 2019
@matsko matsko force-pushed the final_styling_prototype branch 9 times, most recently from 1e0db05 to 19326d7 Compare August 30, 2019 20:20
@matsko matsko requested a review from mhevery August 30, 2019 20:53
@matsko
Copy link
Contributor Author

matsko commented Aug 30, 2019

@matsko matsko force-pushed the final_styling_prototype branch 7 times, most recently from 2b5d511 to d34d536 Compare September 5, 2019 17:42
@angular angular deleted a comment from pkozlowski-opensource Sep 5, 2019
@angular angular deleted a comment from pkozlowski-opensource Sep 5, 2019
@angular angular deleted a comment from pkozlowski-opensource Sep 5, 2019
@matsko matsko requested review from a team as code owners September 10, 2019 19:23
@matsko matsko added the action: merge The PR is ready for merge by the caretaker label Sep 10, 2019
@matsko matsko closed this in 3b37469 Sep 10, 2019
matsko added a commit that referenced this pull request Sep 10, 2019
matsko added a commit to matsko/angular that referenced this pull request Sep 10, 2019
…e writing (angular#32259)

This patch is a final major refactor in styling Angular.

This PR includes three main fixes:

All temporary state taht is persisted between template style/class application
and style/class application in host bindings is now removed.
Removes the styling() and stylingApply() instructions.
Introduces a "direct apply" mode that is used apply prop-based
style/class in the event that there are no map-based bindings as
well as property collisions.

PR Close angular#32259
matsko added a commit to matsko/angular that referenced this pull request Sep 11, 2019
…e writing (angular#32259)

This patch is a final major refactor in styling Angular.

This PR includes three main fixes:

All temporary state taht is persisted between template style/class application
and style/class application in host bindings is now removed.
Removes the styling() and stylingApply() instructions.
Introduces a "direct apply" mode that is used apply prop-based
style/class in the event that there are no map-based bindings as
well as property collisions.

PR Close angular#32259
matsko added a commit to matsko/angular that referenced this pull request Sep 11, 2019
matsko added a commit to matsko/angular that referenced this pull request Sep 11, 2019
matsko added a commit to matsko/angular that referenced this pull request Sep 11, 2019
matsko added a commit that referenced this pull request Sep 11, 2019
…e writing (#32259) (#32596)

This patch is a final major refactor in styling Angular.

This PR includes three main fixes:

All temporary state taht is persisted between template style/class application
and style/class application in host bindings is now removed.
Removes the styling() and stylingApply() instructions.
Introduces a "direct apply" mode that is used apply prop-based
style/class in the event that there are no map-based bindings as
well as property collisions.

PR Close #32259

PR Close #32596
matsko added a commit that referenced this pull request Sep 11, 2019
matsko added a commit to matsko/angular that referenced this pull request Sep 11, 2019
…e writing

This patch is a final major refactor in styling Angular.

This PR includes three main fixes:

All temporary state taht is persisted between template style/class application
and style/class application in host bindings is now removed.
Removes the styling() and stylingApply() instructions.
Introduces a "direct apply" mode that is used apply prop-based
style/class in the event that there are no map-based bindings as
well as property collisions.

PR Close angular#32259
matsko added a commit to matsko/angular that referenced this pull request Sep 12, 2019
…e writing

This patch is a final major refactor in styling Angular.

This PR includes three main fixes:

All temporary state taht is persisted between template style/class application
and style/class application in host bindings is now removed.
Removes the styling() and stylingApply() instructions.
Introduces a "direct apply" mode that is used apply prop-based
style/class in the event that there are no map-based bindings as
well as property collisions.

PR Close angular#32259
matsko added a commit to matsko/angular that referenced this pull request Sep 12, 2019
…e writing

This patch is a final major refactor in styling Angular.

This PR includes three main fixes:

All temporary state taht is persisted between template style/class application
and style/class application in host bindings is now removed.
Removes the styling() and stylingApply() instructions.
Introduces a "direct apply" mode that is used apply prop-based
style/class in the event that there are no map-based bindings as
well as property collisions.

PR Close angular#32259
matsko added a commit to matsko/angular that referenced this pull request Sep 13, 2019
…e writing

This patch is a final major refactor in styling Angular.

This PR includes three main fixes:

All temporary state taht is persisted between template style/class application
and style/class application in host bindings is now removed.
Removes the styling() and stylingApply() instructions.
Introduces a "direct apply" mode that is used apply prop-based
style/class in the event that there are no map-based bindings as
well as property collisions.

PR Close angular#32259
matsko added a commit to matsko/angular that referenced this pull request Sep 14, 2019
…e writing

This patch is a final major refactor in styling Angular.

This PR includes three main fixes:

All temporary state taht is persisted between template style/class application
and style/class application in host bindings is now removed.
Removes the styling() and stylingApply() instructions.
Introduces a "direct apply" mode that is used apply prop-based
style/class in the event that there are no map-based bindings as
well as property collisions.

PR Close angular#32259
matsko added a commit to matsko/angular that referenced this pull request Sep 16, 2019
…e writing

This patch is a final major refactor in styling Angular.

This PR includes three main fixes:

All temporary state taht is persisted between template style/class application
and style/class application in host bindings is now removed.
Removes the styling() and stylingApply() instructions.
Introduces a "direct apply" mode that is used apply prop-based
style/class in the event that there are no map-based bindings as
well as property collisions.

PR Close angular#32259
matsko added a commit to matsko/angular that referenced this pull request Sep 16, 2019
…e writing

This patch is a final major refactor in styling Angular.

This PR includes three main fixes:

All temporary state taht is persisted between template style/class application
and style/class application in host bindings is now removed.
Removes the styling() and stylingApply() instructions.
Introduces a "direct apply" mode that is used apply prop-based
style/class in the event that there are no map-based bindings as
well as property collisions.

PR Close angular#32259
AndrewKushnir pushed a commit that referenced this pull request Sep 16, 2019
…e writing (#32591)

This patch is a final major refactor in styling Angular.

This PR includes three main fixes:

All temporary state taht is persisted between template style/class application
and style/class application in host bindings is now removed.
Removes the styling() and stylingApply() instructions.
Introduces a "direct apply" mode that is used apply prop-based
style/class in the event that there are no map-based bindings as
well as property collisions.

PR Close #32259

PR Close #32591
arnehoek pushed a commit to arnehoek/angular that referenced this pull request Sep 26, 2019
…e writing (angular#32259)

This patch is a final major refactor in styling Angular.

This PR includes three main fixes:

All temporary state taht is persisted between template style/class application
and style/class application in host bindings is now removed.
Removes the styling() and stylingApply() instructions.
Introduces a "direct apply" mode that is used apply prop-based
style/class in the event that there are no map-based bindings as
well as property collisions.

PR Close angular#32259
arnehoek pushed a commit to arnehoek/angular that referenced this pull request Sep 26, 2019
arnehoek pushed a commit to arnehoek/angular that referenced this pull request Sep 26, 2019
…e writing (angular#32259) (angular#32596)

This patch is a final major refactor in styling Angular.

This PR includes three main fixes:

All temporary state taht is persisted between template style/class application
and style/class application in host bindings is now removed.
Removes the styling() and stylingApply() instructions.
Introduces a "direct apply" mode that is used apply prop-based
style/class in the event that there are no map-based bindings as
well as property collisions.

PR Close angular#32259

PR Close angular#32596
arnehoek pushed a commit to arnehoek/angular that referenced this pull request Sep 26, 2019
arnehoek pushed a commit to arnehoek/angular that referenced this pull request Sep 26, 2019
…e writing (angular#32591)

This patch is a final major refactor in styling Angular.

This PR includes three main fixes:

All temporary state taht is persisted between template style/class application
and style/class application in host bindings is now removed.
Removes the styling() and stylingApply() instructions.
Introduces a "direct apply" mode that is used apply prop-based
style/class in the event that there are no map-based bindings as
well as property collisions.

PR Close angular#32259

PR Close angular#32591
@angular-automatic-lock-bot
Copy link

This issue has been automatically locked due to inactivity.
Please file a new issue if you are encountering a similar or related problem.

Read more about our automatic conversation locking policy.

This action has been performed automatically by a bot.

@angular-automatic-lock-bot angular-automatic-lock-bot bot locked and limited conversation to collaborators Oct 11, 2019
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
action: merge The PR is ready for merge by the caretaker area: core Issues related to the framework runtime cla: yes target: major This PR is targeted for the next major release
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

5 participants