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

Improvements and more stability in support HMR/live-reload #6948

Merged
merged 1 commit into from
Jan 16, 2023

Conversation

ixrock
Copy link
Member

@ixrock ixrock commented Jan 16, 2023

For currently used webpack-dev-server@4.x:

Before: in some cases HMR/page-reload didn't happen, e.g. editing welcome.tsx with follow up manual page reload:

Screen.Recording.2023-01-16.at.16.36.05.mov

After: (no need to do Cmd+R when HMR cannot be applied in some cases, see screenshot with error below)
On first editing welcome.tsx a whole app might be restarted, but just once. Next edits of welcome.tsx behaves like auto Cmd+R.

Screen.Recording.2023-01-16.at.16.29.42.mov

Caught error from video why page-reload happens instead of HMR:

Screenshot 2023-01-16 at 16 26 59

…v-server` config

Signed-off-by: Roman <ixrock@gmail.com>
@ixrock ixrock added the enhancement New feature or request label Jan 16, 2023
@ixrock ixrock added this to the 6.4.0 milestone Jan 16, 2023
@ixrock ixrock requested a review from jakolehm January 16, 2023 14:48
@ixrock ixrock requested a review from a team as a code owner January 16, 2023 14:48
@ixrock ixrock self-assigned this Jan 16, 2023
@ixrock ixrock requested review from jansav and aleksfront and removed request for a team January 16, 2023 14:48
@jakolehm jakolehm merged commit 05d3b2e into master Jan 16, 2023
@jakolehm jakolehm deleted the webpack-dev-server-tweaks branch January 16, 2023 16:12
@Nokel81 Nokel81 mentioned this pull request Jan 18, 2023
@Nokel81 Nokel81 added chore and removed enhancement New feature or request labels Jan 18, 2023
ixrock added a commit that referenced this pull request Feb 3, 2023
…tion while reloading, part of #6948

Signed-off-by: Roman <ixrock@gmail.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants