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

Initial Work toward Webpack 5 with Svue #89

Closed
wants to merge 11 commits into from
Closed

Initial Work toward Webpack 5 with Svue #89

wants to merge 11 commits into from

Conversation

thadk
Copy link
Contributor

@thadk thadk commented Feb 22, 2022

@thadk
Copy link
Contributor Author

thadk commented Feb 22, 2022

  • Error from $isLoading

image

{#if $isLoading}

@thadk thadk changed the base branch from master to staging February 22, 2022 11:28
Copy link
Member

@mitchelljkotler mitchelljkotler left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This should be based against master not staging - staging is not updated consistently.

@@ -131,8 +131,8 @@
background: rgba($primary, 0.18);
left: 50%;
top: 50%;
margin-left: -$circleSize / 2;
margin-top: -$circleSize / 2;
margin-left: -$circleSize * 0.5;
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Why is multiplying by 0.5 better than dividing by 2?

@@ -83,6 +83,7 @@
on:click={() => dismiss(i)}
on:mouseover={reset}
on:mousemove={reset}
on:focus={() => null}
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

What does this do?

@@ -250,7 +250,10 @@
}
}

:global(h5, h6) {
:global(h5) {
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Why split these up?

@@ -1,22 +1,45 @@
const MiniCssExtractPlugin = require("mini-css-extract-plugin");
/* TODO: switch to imports instead of require https://github.com/baileyherbert/svelte-webpack-starter/blob/main/webpack.config.ts */
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Review

hotReload: false, /* Turned off for newer versions */
hotOptions: { /*
TK none of these alternated, non-default settings seem fix the
underlying problem with Svue
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

What is the problem with Svue?

@thadk thadk changed the base branch from staging to master February 24, 2022 20:47
@thadk thadk changed the base branch from master to staging February 24, 2022 20:49
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants