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

update watch ''app/pages/**/*.scss" #9

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

rdlabo
Copy link

@rdlabo rdlabo commented Apr 14, 2016

Ionic2 beta.4's gulpfile is watch 'app//*.scss'. so when ''app/pages//*.scss" is changed, browser live reloaded.
But now, don't make css file.

Angular2 have function css capsuled and Ionic2 can use this.

example.

@page({
styleUrls : ['build/css/index/index.css'],
})

so I think ionic-gulp-sass-build must watch ''app/pages/*/.scss'.

thanks.

Ionic2 beta.4's gulpfile is watch 'app/**/*.scss'. so when ''app/pages/**/*.scss"  is changed, browser live reloaded.
But now, don't make css file.

Angular2 have function css capsuled and Ionic2 can use this.

example.
> @page({
>   styleUrls  : ['build/css/index/index.css'],
> })

so I think ionic-gulp-sass-build must watch ''app/pages/**/*.scss'.

thanks.
@tlancina
Copy link
Contributor

tlancina commented May 2, 2016

Hey thanks for the PR. Right now Ionic's Sass isn't really set up to take advantage of style encapsulation, so I'm not sure this makes sense to do right now, but will definitely be a possibility in the future.

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.

2 participants