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

Images measurements support #54

Merged
merged 6 commits into from
Apr 4, 2014
Merged

Conversation

alpadev
Copy link
Collaborator

@alpadev alpadev commented Mar 31, 2014

Version bump to 0.3.4
Adds width and height to the spriter output. Also some code cleanup.

See #52

alpadev added 5 commits April 1, 2014 01:26
(also the old code been a lil dirty imo :))

Returns now a "meta" field instead of two - height and maxheight - fields.
The meta object contains these fields:
```
{
  maxHeight: int,
  maxWidth: int,
  images: [{
      height: int,
      offsetY: int,
      totalHeight: int,
      width: int
    },...]
}
```
the spriter script. See asciidisco#52

Also did some cleanup to the output string building.
### 0.3.4
+ Added width and height properties to the spriter output [@alpadev] (https://github.com/alpadev)
 - 85a04f5 Updates the generator functions to adopt the changes introduced to the spriter script. See asciidisco#52 [@alpadev]
 - 6416c26 Rewrote most of it.. to accomplish asciidisco#52 (also the old code been a lil dirty imo :)) [@alpadev]
@alpadev
Copy link
Collaborator Author

alpadev commented Apr 3, 2014

Just pushed an update to make the output optional - defaults to false.

@asciidisco
Copy link
Owner

Just checked it, seems fine for me.

asciidisco added a commit that referenced this pull request Apr 4, 2014
Images measurements support
@asciidisco asciidisco merged commit 551d9ad into asciidisco:master Apr 4, 2014
@asciidisco
Copy link
Owner

@alpadev @LaurentGoderre
Can one of you guys push a new version?
I´m behind a proxy that doesn't allow me to publish smth. to nom :/

@alpadev
Copy link
Collaborator Author

alpadev commented Apr 4, 2014

@asciidisco @LaurentGoderre

Sorry just got home phew.
Update done:
[17:48:17] $ npm publish
npm http PUT https://registry.npmjs.org/grunt-imagine
npm http 201 https://registry.npmjs.org/grunt-imagine

  • grunt-imagine@0.3.4

cybear added a commit to cybear/grunt-imagine that referenced this pull request May 27, 2014
## 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.
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.

2 participants