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

Adding automatic installation script #21

Merged
merged 2 commits into from
Oct 17, 2013
Merged

Conversation

rudisimo
Copy link
Contributor

I've added a Rakefile for easier installation into the user stylesheet directory. The script supports Chrome and Chrome Canary, plus it will install the stylesheet on all available profiles. To use it, run rake from the source directory.

- Supports Chrome and Chrome Canary
- Installs on all user profiles
@s10wen
Copy link
Owner

s10wen commented Oct 17, 2013

A_W_E_S_O_M_E

Would it be possible to do the reverse as well?

s10wen added a commit that referenced this pull request Oct 17, 2013
Adding automatic installation script
@s10wen s10wen merged commit 5a2a65d into s10wen:master Oct 17, 2013
@rudisimo
Copy link
Contributor Author

@simonowendesign Sure, the only thing to watch out for is a user with multiple profiles. I could give them the choice to pull the Custom.css file from a specific profile, like so:

Chrome Profiles
  [1] Default
  [2] Profile 1
  [3] Profile 2

Please choose your profile: [1]

@s10wen
Copy link
Owner

s10wen commented Oct 17, 2013

Sgtm :squirrel:

@mauricecruz
Copy link

Hey @rudisimo, Thanks for building this!

I tested this on Windows. There's a few issues:

  1. Not sure why but on Windows, Chrome Canary directory is called "Chrome SxS". Simple fix though.
  2. The backup feature is not working.

Also, it would be nice to implement the choice interface you suggested. I don't think it's fair to assume every profile would want to use the same theme.

@s10wen
Copy link
Owner

s10wen commented Oct 17, 2013

Yeah, sorry that got fixed after I'd done the PR - febebd6

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

Successfully merging this pull request may close these issues.

3 participants