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.

Sign up for GitHub

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

Jump to bottom

add stylus support for sprites #50

Closed
ibes opened this issue Jan 2, 2014 · 1 comment
Closed

add stylus support for sprites #50

ibes opened this issue Jan 2, 2014 · 1 comment
Assignees
@alpadev

Comments

@ibes
Copy link

ibes commented Jan 2, 2014

Hi,

would be nice (and I guess not much work) to have stylus support added. I am not sure enough about the code of the generator, but the output of the example in the readme would be (in the shortest possible form):

.Icon-MyImage1
.Icon-MyImage2
  background url(../img/icon36.png) no-repeat

.Icon-MyImage1
  background-position 0 -432px

.Icon-MyImage2
  background-position 0 -396px

Stylus Docs: http://learnboost.github.io/stylus/
Test my code (it's a form, so paste the code to the left): http://learnboost.github.io/stylus/try.html
@alpadev alpadev mentioned this issue Jan 2, 2014
Merged
@ghost ghost assigned alpadev Jan 2, 2014
@alpadev
Copy link
Collaborator

alpadev commented Jan 7, 2014

Just published v0.3.3 that adds stylus support :)

regards

@alpadev alpadev closed this as completed Jan 7, 2014
cybear added a commit to cybear/grunt-imagine that referenced this issue May 27, 2014
@cybear
(inlineImg) Rewrite of main loop
62c8928 
## Utilize an object FileMeta to contain everything needed for each
file.

It keeps the original path and supplies helper functions to do
everything necessary with each file.

## Use functional style to filter the file list

* `filterOnlyImage` Filter out fonts as well as CSS files
* `filterOnlyExists` Filter out files that don’t exist
* `filterOnlySmall` Filter away large files if IE8 mode activated

## Other fixes

* Verify that `fs.exists` instead of opening and using `try…catch`
* The path verification didn’t work in my project. I added a method of
detecting the path (asciidisco#52) but kept both the other methods (asciidisco#50, asciidisco#54) so
as not to break any earlier functionality.
@cybear cybear mentioned this issue May 27, 2014
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@alpadev alpadev

Labels
None yet
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@ibes @alpadev