bcoe / record-crate

index, organize, and search your music collection, DJ sick sets.
ISC License
25 stars 6 forks source link

Polymize #9

Closed DeviaVir closed 9 years ago

DeviaVir commented 9 years ago

Alright, I'm going to throw this out here as the base for the app frontend. I've already shared some screenshots, now it's time to let you and others also play around, leave feedback, add issues for missing features, etc. etc.

Currently features the following:

Some screenshots: screen shot 2014-12-17 at 23 36 34 screen shot 2014-12-17 at 23 36 27 screen shot 2014-12-17 at 23 36 50

bcoe commented 9 years ago

@DeviaVir this is looking awesome. I'm not super familiar with the Grunt/Bower workflow, what steps should I run to get the app up and running.

I think it would be smart to add any dependencies: grunt-cli, bower, to the project, and to allow them to be executed through the bin.

I think it would be great if we could keep it so the user can still simply install the module, and start the server -- perhaps we could use a prestart hook in the package.json to run all the necessary build steps, e.g., bower grunt.

zeke commented 9 years ago

Nice work, @DeviaVir! Can you update instructions in the README?

DeviaVir commented 9 years ago

@zeke Actually, I did already, check the changes! ;)

@bcoe this is my goal as well, but here's where I am less certain of how to accomplish this, hence the PR so we can start making it one asap

DeviaVir commented 9 years ago

Updated the package.json so that npm start starts both now!

DeviaVir commented 9 years ago

Small update to the "info page", when clicking an item. Preview below.

screen shot 2014-12-18 at 15 14 59

bcoe commented 9 years ago

@DeviaVir this is looking great!

A couple technical comments, to get it working on my computer:

Once I did this everything worked great with npm start. I'm really digging the initial design, and will have some comments shortly :+1:

DeviaVir commented 9 years ago

I pushed the fixes you had to apply to make it work. You want to go ahead and merge? I think it would give us more of an overview if you would add your comments in separate issues, and we would throw in different PR's per issue. But that's my 2 cents :)

DeviaVir commented 9 years ago

Ah, probably this error: OptionParser::InvalidOption: invalid option: --sourcemap showed up, because you have an old version of sass. We can probably use sourcemap again, if you update the ruby gem. But I disabled it for now, so there's no hurry to.

bcoe commented 9 years ago

Cool., that explains my problem. I'm going to merge what we have so far, so that we have a foundation and can move forward with smaller bits of functionality. Here's a thought I have:

I think it would be cool if we performed the sass and bower installation steps as an occasional operation which we check-in. That way someone could install redord-crate via npm install, without having to worry about dev-dependencies.