Skip to content

feat(css): add attribute scoped css without global refresh (#7907)#7947

Merged
manoldonev merged 1 commit intoreleasefrom
amiorkov/scoped-css-patch
Oct 14, 2019
Merged

feat(css): add attribute scoped css without global refresh (#7907)#7947
manoldonev merged 1 commit intoreleasefrom
amiorkov/scoped-css-patch

Conversation

@VladimirAmiorkov
Copy link
Copy Markdown
Contributor

This is a patch PR for #7907

@VladimirAmiorkov VladimirAmiorkov self-assigned this Oct 11, 2019
@cla-bot cla-bot bot added the cla: yes label Oct 11, 2019
@MartoYankov
Copy link
Copy Markdown
Contributor

This PR is basically adding a new feature. I'm not sure this is good for a patch. Also, I've communicated with the guy who initially reported the issue and helped me isolate it that it will come out with {N} 6.2 at the end of the month.

@manoldonev
Copy link
Copy Markdown
Contributor

manoldonev commented Oct 14, 2019

@MartoYankov Generally I agree with you but as this change is triggered by a nativescript-angular fix, we need to release it with a patch in tns-core-modules as with 6.2.0 we'll be releasing a patch version of nativescript-angular (nativescript-angular versioning is tied to angular major and minor releases) that will rely on this fix and otherwise it will break all users (e.g. if you have a dependency like "nativescript-angular": "~8.2.0").

@manoldonev manoldonev merged commit b377eb6 into release Oct 14, 2019
@manoldonev manoldonev deleted the amiorkov/scoped-css-patch branch October 14, 2019 08:25
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants