-
Notifications
You must be signed in to change notification settings - Fork 99
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
Diff swallows black color in added areas #34
Comments
I had another case where the diff image's pixels are not distinguishable, the difference is there and correctly detected (a subtle one), but the resulting image is all black, probably the same issue. |
I've fixed the problem with a options.lightness for imagediff.diff, I should post a pull request this week end. |
…ows black color in added areas). Default +25 color light value.
Here are some screenshots illustrating the fix, all taken with Succss (The CSS regression testing tool I'm doing). I also factorised the rgb color diffing code and tested it for diffUnequal (with option.align set to 'top') as well:
While the previous PR can be used per project, another, much better and stable pull request is in the work. |
…dds options.ligthness, options.rgb and options.stack to customize the diff image.
…dds options.ligthness, options.rgb and options.stack to customize the diff image. Updates Jasmine ImageDiffSpec with new diff options
In a diff with two unequal images, added pixels in black are not distinguishable. They are swallowed by the otherwise black background, the result of the diff of same pixels.
Test case: http://jsfiddle.net/cburgmer/79pr2/4/
The text was updated successfully, but these errors were encountered: