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

feat: allow to include shared styles #41

Merged
merged 3 commits into from
Jun 29, 2018
Merged

feat: allow to include shared styles #41

merged 3 commits into from
Jun 29, 2018

Conversation

ktsn
Copy link
Owner

@ktsn ktsn commented Jun 28, 2018

This PR let us to specify vueDesigner.sharedStyles config which allows to include global styles into the preview page.

It would be useful when we use some css frameworks, reset / normalize css or so on.

I guess I will update this PR to enable watching the changes for the shared styles so that update the preview style in realtime.

Currently, shared style affects globally not only user defined components but also app level (Vue Designer's) components. It should be avoided but seems the implementation will be a bit complicated. I would leave it as a further improvement.

The preprocessors are not supported as same as <style> block in SFC.

@codecov
Copy link

codecov bot commented Jun 29, 2018

Codecov Report

Merging #41 into master will decrease coverage by 1.85%.
The diff coverage is 5%.

Impacted file tree graph

@@            Coverage Diff             @@
##           master      #41      +/-   ##
==========================================
- Coverage   66.31%   64.46%   -1.86%     
==========================================
  Files          50       51       +1     
  Lines        1633     1683      +50     
  Branches      378      390      +12     
==========================================
+ Hits         1083     1085       +2     
- Misses        535      583      +48     
  Partials       15       15
Impacted Files Coverage Δ
src/view/components/Renderer.vue 79.48% <ø> (ø) ⬆️
src/view/components/PageMain.vue 0% <ø> (ø) ⬆️
src/view/components/Viewport.vue 50% <ø> (ø) ⬆️
src/message/bus.ts 0% <0%> (ø) ⬆️
src/view/store/modules/project/project-actions.ts 26.37% <0%> (-0.6%) ⬇️
...rc/view/store/modules/project/project-mutations.ts 40.74% <0%> (-0.77%) ⬇️
src/vscode/watcher.ts 0% <0%> (ø)
src/server/main.ts 0% <0%> (ø) ⬆️
src/vue-designer.ts 0% <0%> (ø) ⬆️
src/view/store/modules/project/project-state.ts 100% <100%> (ø) ⬆️
... and 2 more

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 90c47f5...b8e44e4. Read the comment docs.

@ktsn ktsn merged commit 6b07626 into master Jun 29, 2018
@ktsn ktsn deleted the shared-css branch June 29, 2018 05:24
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

1 participant