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

Fixing positioning issue #873

Closed
wants to merge 4 commits into from
Closed

Conversation

vbichkovsky
Copy link
Contributor

When there are some repeat-x images in the sprite and some other images have position set to 100%, there is a gap between sprite's right side and images with position = 100% (because width-fix was applied after they had been positioned).
I tried to figure out how to write a test for such case and I must say that your tests are difficult to comprehend ;)

@chriseppstein
Copy link
Member

@scottdavis can you review and merge when convenient?

@@ -1,3 +1,5 @@
require 'rational'
Copy link
Member

Choose a reason for hiding this comment

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

Do we need this require?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Nope, it just happened that my commit contained these changes and I created a pull request from it. You only need to change layout_methods.rb

Copy link
Member

Choose a reason for hiding this comment

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

manually changing something kinda makes using pull requests pointless =p

@chriseppstein
Copy link
Member

@scottdavis what's the status here? Are you holding this up so that a single require can be removed? >_<

@scottdavis
Copy link
Member

No I'm holding it up because I'm busy I'll get to it this week I'm going to cherry-pick it into master

Sent from my iPhone

On Jun 4, 2012, at 2:28 AM, Chris Eppsteinreply@reply.github.com wrote:

@scottdavis what's the status here? Are you holding this up so that a single require can be removed? >_<


Reply to this email directly or view it on GitHub:
#873 (comment)

@scottdavis
Copy link
Member

merged 5e42b8f

@scottdavis scottdavis closed this Jun 4, 2012
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.

None yet

3 participants