Skip to content
This repository has been archived by the owner on Aug 7, 2021. It is now read-only.

fix(template): disable minification of css by css-loader #154

Merged
merged 1 commit into from
May 16, 2017

Conversation

sis0k0
Copy link
Contributor

@sis0k0 sis0k0 commented May 16, 2017

When uglify is used, it triggers the minification option of the
css-loader. The css-loader uses internally cssnano to optimize the size
of the stylesheets. Since NativeScript supports only a subset of the CSS language, many of these optimizations may be damaging for the user's css.
Such is the case with background-position: center, which gets
translated to background-position: 50%, which is currently not working
in NativeScript 3.0 (NativeScript/NativeScript#4201).

closes #135

When uglify is used, it triggers the minification option of the
css-loader. The css-loader uses internally cssnano to optimize the size
of the stylesheets. Since NativeScript supports only a subset of the CSS language, many of these optimizations may be damaging for the user's css.
Such is the case with `background-position`: center, which gets
translated to `background-position`: 50%, which is currently not working
in NativeScript 3.0 (NativeScript/NativeScript#4201).

closes #135
@sis0k0 sis0k0 force-pushed the vlaeva/css-loader-minify branch from 97b0edf to 5f5686b Compare May 16, 2017 10:02
@sis0k0 sis0k0 merged commit 30e9c97 into master May 16, 2017
@sis0k0 sis0k0 deleted the vlaeva/css-loader-minify branch May 16, 2017 10:22
sis0k0 added a commit that referenced this pull request Dec 28, 2017
sis0k0 added a commit that referenced this pull request Jan 2, 2018
sis0k0 added a commit that referenced this pull request Jan 9, 2018
sis0k0 added a commit that referenced this pull request Jan 9, 2018
* fix(css): disable minification when using uglify

related to #154, #135
fixes #377

* refactor: inline css-loader options
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

CSS background-position does not apply on webpack
1 participant