Skip to content

Decompose sprites #1771

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

Open
wants to merge 44 commits into
base: master
Choose a base branch
from
Open

Decompose sprites #1771

wants to merge 44 commits into from

Conversation

scottdavis
Copy link
Member

WIP: Moves sprites to a compass plugin.

name = Compass::SpriteImporter.sprite_name(relative_uri)
sprites = Compass::SpriteImporter.new
name = Compass::Sprites::Importer.sprite_name(relative_uri)
sprites = Compass::Sprites::Importer.new

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Useless assignment to variable - sprites.

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

what is this object for?

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

no idea looks like it was caught in the global rename macro i ran

@lolmaus
Copy link
Contributor

lolmaus commented Aug 19, 2014

Thank you for your work Scott. O_O

@scottdavis
Copy link
Member Author

ping @chriseppstein this is massive so review it well all tests are passing

).each do |lib|
require "compass/#{lib}"
end
#tem
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Not sure what this comment is for. Delete it?

@scottdavis
Copy link
Member Author

also i botched up travis some how https://travis-ci.org/Compass/compass/jobs/36176909

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants