Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Bower is a pain w/customization #66

Closed
grayghostvisuals opened this issue Sep 5, 2013 · 10 comments
Closed

Bower is a pain w/customization #66

grayghostvisuals opened this issue Sep 5, 2013 · 10 comments
Milestone

Comments

@grayghostvisuals
Copy link
Contributor

Love this, but one problem I have is bower. Bower is very powerful, but for things u really won't touch.

Tried using this project on bower, but wanted to change a whole load of stuff. Padding on the inner modal div, border color, width, breakpoints etc.

Maybe one of these days we can actually take advantage of a better package mgt. system. I just don't think we are there yet with them.

@drublic
Copy link
Owner

drublic commented Sep 13, 2013

Thanks so much for bringing up this issue. With #58 we will be trying to make the modal more customizable and make it easier to use it with Bower.

We could use some help with defining the customizable variables in Sass – so if you are up for it we would be very thankful.

@grayghostvisuals
Copy link
Contributor Author

Yeah I'm down. What I am doing right now with my project Typeplate (also on bower) is taking advantage of a separate var file. The main project file contains vars with Sass' !default flag. I don't know how I still feel about this method though :\ It really blows that bower rewrites the file when you update it.

Where do u want me to start? Don't wanna begin on something someone else is doing or does not agree with.

@drublic
Copy link
Owner

drublic commented Sep 16, 2013

Thanks for your ideas.

I am not sure where we should start with this.
Maybe a structure like this would be great:

|-- components
   |-- css-modal
      |-- _modal-config.scss
      |-- _modal-core.scss
      |-- _modal-theme.scss
      |-- modal.scss
|-- scss
   |-- _modal-config.scss
   |-- main.scss

While _modal-config.scss in css-modal (Bower component) holds all of the modal's default configuration, maybe with a !default directive (might be needed). The _modal-config.scss file in scss holds all the custom overwrites we need. We could include a template for this in the package too.

main.scss would have the following inclusions:

@import 'modal-config';
@import '../components/css-modal/modal';

All configuration in the local file overwrites the global defaults.

What do you think? Does this make sense?

@grayghostvisuals
Copy link
Contributor Author

Yup makes sense. Kind of the only way around this atm w/Bower. Users will have to duplicate the modal-config file and manually bring it over into their Sass folder if they want customization.

I can start on creating the config files. What's the workflow you want? Thinking a branch called development that pulls in branches from other contributors might be a good flow.

@drublic
Copy link
Owner

drublic commented Sep 17, 2013

We use "v1.0.0" as our "development" branch atm. It would be awesome if you could create a branch from there and use the existing work in #58. You can then do a PR against v1.0.0 and we will review from there. Sounds good?

@grayghostvisuals
Copy link
Contributor Author

yup that works.

@grayghostvisuals
Copy link
Contributor Author

@drublic Can u explain briefly on your build process with the v1.0.0 branch? Seems like we can also tackle a good contributing doc from this discussion.

@drublic
Copy link
Owner

drublic commented Sep 25, 2013

I've added a little bit of contribution guidelines in CONTRIBUTE.md. Which information is missing?

@grayghostvisuals
Copy link
Contributor Author

@drublic

Very, very minor as of right now…

  1. npm install for fresh projects
  2. What current version of Sass you're using

drublic added a commit that referenced this issue Oct 7, 2013
Closes #58. Thanks to @fabien-d.
Closes #66.
Closes #73.
drublic added a commit that referenced this issue Oct 7, 2013
drublic added a commit that referenced this issue Oct 7, 2013
@drublic
Copy link
Owner

drublic commented Oct 7, 2013

I've updated the CONTRIBUTE.md file. Thanks for the discussion and your ideas. Was applied in #73.

@drublic drublic closed this as completed Oct 7, 2013
drublic added a commit that referenced this issue Oct 7, 2013
Closes #58. Thanks to @fabien-d.
Closes #66.
Closes #73.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants