These are chat archives for thinkingmedia/iChrome

8th
Dec 2015
Avi Kohn
@AMKohn
Dec 08 2015 00:37
I just pulled your changes into a local branch so I can switch to grunt-sass (Travis and I need Ruby and SASS installed otherwise) and I'm getting:
Warning: It's not clear which file to import for '@import "widgets"'.
Candidates:
  widgets.scss
  widgets.css
Please delete or rename all but one of these files.
Ideas?
Sorry to bother you, but I have no experience with SASS. I'm not sure what the best approach is.
Mathew Foscarini
@thinkingmedia
Dec 08 2015 00:40
Can you run it with - - trace
Avi Kohn
@AMKohn
Dec 08 2015 00:40
The grunt task?
style.scss is calling @import "widgets";
Mathew Foscarini
@thinkingmedia
Dec 08 2015 00:41
Grunt sass:dev - - debug - - verbose - - trace
Yeah. It's that not right?
Avi Kohn
@AMKohn
Dec 08 2015 00:41
Running "sass:dev" (sass) task
Verifying property sass.dev exists in config...OK
Files: app/css/_app.scss -> app/css/_app.css
Files: app/css/_colors.scss -> app/css/_colors.css
Files: app/css/_mixins.scss -> app/css/_mixins.css
Files: app/css/_values.scss -> app/css/_values.css
Files: app/css/style.scss -> app/css/style.css
Files: app/css/widgets.scss -> app/css/widgets.css
Options: precision=10, compass, lineNumbers
Options: precision=10, compass, lineNumbers
Options: precision=10, compass, lineNumbers
Options: precision=10, compass, lineNumbers
Options: precision=10, compass, lineNumbers
Options: precision=10, compass, lineNumbers
>> It's not clear which file to import for '@import "widgets"'.
>> Candidates:
>>   widgets.scss
>>   widgets.css
>> Please delete or rename all but one of these files.
>>
>>   Line 12  Column 1  app\css\style.scss
Warning:  Use --force to continue.

Aborted due to warnings.
Mathew Foscarini
@thinkingmedia
Dec 08 2015 00:42
??????
Avi Kohn
@AMKohn
Dec 08 2015 00:42
This happened when I ran the watch task
Both files (scss and css) exist
Mathew Foscarini
@thinkingmedia
Dec 08 2015 00:42
yeah it shouldn't be doing that
Avi Kohn
@AMKohn
Dec 08 2015 00:42
one sec, brb
Back
Mathew Foscarini
@thinkingmedia
Dec 08 2015 00:46
Can you run sass --version on the command line
Avi Kohn
@AMKohn
Dec 08 2015 00:46
I don't have sass installed...
This is libsass
But I don't think that's the issue
There are two files with the same name
Mathew Foscarini
@thinkingmedia
Dec 08 2015 00:46
yes, one is the SASS file and the other is the output
SASS should not be conflicting with this.
libsass must need different settings
Avi Kohn
@AMKohn
Dec 08 2015 00:48
I'll install sass and see what happens
I need to get ruby
Mathew Foscarini
@thinkingmedia
Dec 08 2015 00:48
one sec.
I'm trying to get it to use grunt-sass my setup is default to my Ruby SASS install
npm installed node-sass but grunt isn't using it
Avi Kohn
@AMKohn
Dec 08 2015 00:49
After switching grunt-contrib-sass for grunt.loadNpmTasks("grunt-sass"); mine just took it
I ran sass:dev to test
then grunt watch
Mathew Foscarini
@thinkingmedia
Dec 08 2015 00:50
ah i must have missed that
still can't reproduce
Avi Kohn
@AMKohn
Dec 08 2015 00:51
I ran sass:dev first which created the css file, I just deleted it and ran watch which ran fine.
I'll make a change and see what happens
Nope, loop after the first run
Adding an explicit .scss in style.scss fixes it
I guess I'll do that
Mathew Foscarini
@thinkingmedia
Dec 08 2015 00:53
yeah, that will do it, but I don't understand why
Avi Kohn
@AMKohn
Dec 08 2015 00:53
Maybe SASS itself resolves scss if it has a choice between the two.
Or you have a config file somewhere that I don't
Mathew Foscarini
@thinkingmedia
Dec 08 2015 00:56
I uninstalled SASS and now can reproduce the issue
Avi Kohn
@AMKohn
Dec 08 2015 00:57
adding .scss works anyway. Being explicit is good
Looks good overall, I'm getting 240ms compilation times
Mathew Foscarini
@thinkingmedia
Dec 08 2015 00:57
okay
the options for grunt-sass are different from grunt-contrib-sass
I'll push an update
Avi Kohn
@AMKohn
Dec 08 2015 01:00
OK, I'll delete my branch
Mathew Foscarini
@thinkingmedia
Dec 08 2015 01:02
okay done
Avi Kohn
@AMKohn
Dec 08 2015 01:05
Could you update the dev config to this before I merge? I want to keep things consistent.
                compass: true,
                indentWidth: 1,
                indentType: "tab",
                lineNumbers: true,
                outputStyle: "expanded"
Actually it also needs sourceComments: true instead of lineNumbers: true
Avi Kohn
@AMKohn
Dec 08 2015 01:13
nvm, merged
Mathew Foscarini
@thinkingmedia
Dec 08 2015 01:14
remove compass: true iChrome doesn't use compass
Avi Kohn
@AMKohn
Dec 08 2015 01:15
grunt-sass doesn't support it either
Mathew Foscarini
@thinkingmedia
Dec 08 2015 01:15
mostly harmless than
Avi Kohn
@AMKohn
Dec 08 2015 01:15
I'll update the whole config, it needs a different sourcemap configuration
Mathew Foscarini
@thinkingmedia
Dec 08 2015 01:15
k
Avi Kohn
@AMKohn
Dec 08 2015 01:23
I see you're working on Bower. A number of the libraries iChrome uses have been patched
Mathew Foscarini
@thinkingmedia
Dec 08 2015 01:24
it wasn't going very well so I stopped.
lol
Avi Kohn
@AMKohn
Dec 08 2015 01:24
OK
np
I don't change them often
But SASS is great, thanks!
Mathew Foscarini
@thinkingmedia
Dec 08 2015 01:24
np, bower is great when you're starting a project.
I often have my work-in-progress always build with the latest jquery, angular, etc.. etc.. which is easier when you have bower
Avi Kohn
@AMKohn
Dec 08 2015 01:25
Yeah
Can Bower do custom jQuery builds? (slimmed down?)
Looks like it can't. Too bad
Mathew Foscarini
@thinkingmedia
Dec 08 2015 01:32
I don't know.
it's recommended to use the google cdn
Avi Kohn
@AMKohn
Dec 08 2015 01:33
Yeah, but not offline
Mathew Foscarini
@thinkingmedia
Dec 08 2015 01:33
oh
right
there is a way to do both
I think google has a script that will load it locally if the cdn can't be accessed
Avi Kohn
@AMKohn
Dec 08 2015 01:34
Loading locally is faster than the CDN, AFAIK, even if it's cached.
Mathew Foscarini
@thinkingmedia
Dec 08 2015 01:34
true i forgot it was an extension
lol
Avi Kohn
@AMKohn
Dec 08 2015 01:34
np
Plus it's bundled this way.
But I'm not working on performance for another week or so :P
Avi Kohn
@AMKohn
Dec 08 2015 01:35
Yup, that's on the list
I assembled a list of the sections iChrome uses
Anyway, I'm going to quit for the day I think
Good night!
Mathew Foscarini
@thinkingmedia
Dec 08 2015 01:36
night