Skip to content

Fix errors that occur when you try to yarn watch:#1

Open
aimeehainesTJ wants to merge 1 commit into
masterfrom
feature/fix-yarn-errors
Open

Fix errors that occur when you try to yarn watch:#1
aimeehainesTJ wants to merge 1 commit into
masterfrom
feature/fix-yarn-errors

Conversation

@aimeehainesTJ

Copy link
Copy Markdown
  • Modal.twig doesn't exist so needs removing
  • SVG doesn't exist so needs removing
  • Missing fluid-unit mixin
  • Normalize import fails
  • Missing file editor.js which is required

- Modal.twig doesn't exist so needs removing
- SVG doesn't exist so needs removing
- Missing fluid-unit mixin
- Normalize import fails
- Missing file editor.js which is required
Comment thread assets/js/editor.js
@@ -0,0 +1,10 @@
import { FontLoader } from './tools';

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

We probably don't want to load the theme fonts into the editor/CMS - perhaps we can add a placeholder or just console log for the editor file to be generated for now?

Comment thread assets/js/tools/fonts.js
document.documentElement.classList.add('fonts-loaded');
}
}
} No newline at end of file

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Can you add a newline at the end of this file (keeps git happy) - Looks like you can enable this across all files in PHPStorm like so https://stackoverflow.com/a/36020003/2166894

Comment thread assets/js/tools/index.js
@@ -0,0 +1 @@
export FontLoader from './fonts'; No newline at end of file

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Same here re: newline

@@ -1 +1 @@
@import 'normalize.css'; No newline at end of file
@import '~normalize.css'; No newline at end of file

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Same here re: newline


@function fluid-unit($pixel_value) {
@return #{strip-unit($pixel_value) / strip-unit($max-font)}rem;
} No newline at end of file

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Same here re: newline

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.

2 participants