jadjoubran / laravel5-angular-material-starter

Get started with Laravel 5.3 and AngularJS (material)
https://laravel-angular.readme.io/
MIT License
1.66k stars 400 forks source link

Fixed gulpfile bug #417

Closed yazfield closed 7 years ago

yazfield commented 7 years ago

What kind of change does this PR introduce? (Bug fix, feature, docs update, ...)

What is the current behavior? (You can also link to an open issue here) Issue #414

What is the new behavior? Fixed the issue by removing ./ prefix from paths and setting Elixir paths in config.

Does this PR introduce a breaking change?

Other information: This fix is kind of a 'hack' for a bug in Elixir, it will have to be changed in the future. I changed the gulp file from using absolute paths with ./ prefix to using relative paths by setting elixir paths in config. However, the styles mixin needs to override elixir path config using ./ prefix (works just fine here!) which looks ugly.

jadjoubran commented 7 years ago

Awesome, I had this noted on my trello board and I saw your comment on the bug, thanks! 😄