-
Notifications
You must be signed in to change notification settings - Fork 24.8k
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
set encapsulation to None
when there is no style / set ng-version
attribute on root component
#27175
Closed
ocombe
wants to merge
2
commits into
angular:master
from
ocombe:fix/ivy/testbed-platform-server/render
Closed
set encapsulation to None
when there is no style / set ng-version
attribute on root component
#27175
ocombe
wants to merge
2
commits into
angular:master
from
ocombe:fix/ivy/testbed-platform-server/render
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
ocombe
added
type: bug/fix
action: review
The PR is still awaiting reviews from at least one requested reviewer
effort1: hours
area: server
Issues related to server-side rendering
target: major
This PR is targeted for the next major release
comp: ivy
risk: low
labels
Nov 19, 2018
alxhub
approved these changes
Nov 19, 2018
You can preview 4cf11f8 at https://pr27175-4cf11f8.ngbuilds.io/. |
mhevery
approved these changes
Nov 19, 2018
ocombe
force-pushed
the
fix/ivy/testbed-platform-server/render
branch
from
November 19, 2018 20:08
4cf11f8
to
e2bb32f
Compare
You can preview e2bb32f at https://pr27175-e2bb32f.ngbuilds.io/. |
alxhub
approved these changes
Nov 20, 2018
ocombe
added
action: merge
The PR is ready for merge by the caretaker
PR action: time-zone
and removed
action: review
The PR is still awaiting reviews from at least one requested reviewer
labels
Nov 20, 2018
mhevery
pushed a commit
that referenced
this pull request
Nov 20, 2018
FrederikSchlemmer
pushed a commit
to FrederikSchlemmer/angular
that referenced
this pull request
Jan 3, 2019
FrederikSchlemmer
pushed a commit
to FrederikSchlemmer/angular
that referenced
this pull request
Jan 3, 2019
ngfelixl
pushed a commit
to ngfelixl/angular
that referenced
this pull request
Jan 28, 2019
ngfelixl
pushed a commit
to ngfelixl/angular
that referenced
this pull request
Jan 28, 2019
This issue has been automatically locked due to inactivity. Read more about our automatic conversation locking policy. This action has been performed automatically by a bot. |
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: server
Issues related to server-side rendering
cla: yes
effort1: hours
risk: low
target: major
This PR is targeted for the next major release
type: bug/fix
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
PR Type
What kind of change does this PR introduce?
What is the current behavior?
We used to set view encapsulation to
None
when there was no styles, but it's not the case with ivy, which will create css attribute selectors on elements when it's not needed.Also we don't set the
ng-version
attribute on the root component.What is the new behavior?
We set encapsulation to
None
when there is no style.We set
ng-version
attribute on root component.Does this PR introduce a breaking change?