Skip to content

feat(HMR): style views at runtime - #7012

Merged
vchimev merged 3 commits into
masterfrom
vchimev/style-view
Mar 13, 2019
Merged

vchimev merged 3 commits into
masterfrom
vchimev/style-view

Conversation

@vchimev

@vchimev vchimev commented Mar 8, 2019 •

Copy link
Copy Markdown
Contributor

PR Checklist

What is the current behavior?

A change in styles applies at runtime only for a Frame root applications.

What is the new behavior?

This pull request enables changes in styles to apply at runtime, no matter of the root application: layouts, Frame, TabView or SideDrawer.

Implements a part of #6953.

@cla-bot cla-bot Bot added the cla: yes label Mar 8, 2019
@ghost ghost assigned vchimev Mar 8, 2019
@ghost ghost added the in progress label Mar 8, 2019
@vchimev vchimev changed the title feat(HMR): style view at runtime feat(HMR): style views at runtime Mar 8, 2019
@vchimev
vchimev force-pushed the vchimev/style-view branch from 277db23 to 11aefb6 Compare March 11, 2019 11:51
@MartoYankov

Copy link
Copy Markdown
Contributor

We should add some more e2e hmr tests. For example, changing the CSS of TabView as root.

@vchimev
vchimev force-pushed the vchimev/style-view branch 2 times, most recently from 11aefb6 to 4c143cc Compare March 12, 2019 13:08
Comment thread tests/app/testRunner.ts Resolved
vchimev added 2 commits March 12, 2019 17:48
test: module root view comopnent

test: update livesync tests

refactor: _onLivesync function
@vchimev
vchimev force-pushed the vchimev/style-view branch from 4c143cc to b8e87ed Compare March 12, 2019 15:48
@vchimev
vchimev force-pushed the vchimev/style-view branch from b8e87ed to f9edb1b Compare March 12, 2019 15:51
@vchimev

vchimev commented Mar 13, 2019

Copy link
Copy Markdown
Contributor Author

@cla-bot check

@cla-bot

cla-bot Bot commented Mar 13, 2019

Copy link
Copy Markdown

The cla-bot has been summoned, and re-checked this pull request!

@vchimev
vchimev merged commit 3c2c1d9 into master Mar 13, 2019
@vchimev
vchimev deleted the vchimev/style-view branch March 13, 2019 11:24
@ghost ghost removed the in progress label Mar 13, 2019
@lock

lock Bot commented Mar 12, 2020

Copy link
Copy Markdown

This thread has been automatically locked since there has not been any recent activity after it was closed. Please open a new issue for related bugs.

@lock lock Bot locked and limited conversation to collaborators Mar 12, 2020
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants