-
Notifications
You must be signed in to change notification settings - Fork 276
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
corrected display problems in MoveSelectedTool and LivePreviewManager #115
Conversation
I no longer feel confident about making decisions on things not not directly related to addins, so @cameronwhite will have to make an opinion here I think. But I can say that there is a problem with indentation caused by tabs vs. spaces here too, which will need correcting. 👌 |
Updated intends (by replacing 4 spaces with tab, don't know why mono used spaces...) |
@@ -421,7 +421,9 @@ public void FinishSelection () | |||
Layer layer = SelectionLayer; | |||
|
|||
using (Cairo.Context g = new Cairo.Context (CurrentUserLayer.Surface)) { | |||
layer.Draw(g); | |||
//layer.Draw(g); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Can you remove this line instead of leaving it commented out? Otherwise, the changes look good
@cameronwhite |
Looks good - I just didn't see the new commit. |
no problem. Thank you for merging it :-) |
The error can be seen e.g. when having a colored semi-transparent image and apply a black and white filter. In the regions of the image where alpha is != 255 there remain colored parts.