Skip to content

Update Capturer.cs - new points + gradient#6

Open
Selmanovich wants to merge 1 commit intoreisxd:mainfrom
Selmanovich:patch-1
Open

Update Capturer.cs - new points + gradient#6
Selmanovich wants to merge 1 commit intoreisxd:mainfrom
Selmanovich:patch-1

Conversation

@Selmanovich
Copy link

Created a new mapping for the points where pixel colors are captured. Fixed the incorrect left and bottom segments positioning. Rewrote the frame generation function for the image—now it uses rectangles filled with a linear gradient according to the colors captured from the screen pixels.
screenshoot01
screenshoot02

Created a new mapping for the points where pixel colors are captured. 
Fixed the incorrect left and bottom segments positioning.
Rewrote the frame generation function for the image—now it uses rectangles filled with a linear gradient according to the colors captured from the screen pixels.
@moonkracker
Copy link

Yeah, I'm having the same problem, so really looking forward to this change being added to the main branch.

Copy link

@GaetanPierre GaetanPierre left a comment

Choose a reason for hiding this comment

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

I forked your work to test it, but I encountered some build issues.


paint.Shader = shader_right;
canvas.DrawRect(new SKRect(imgWidth - borderThick, 0, imgWidth, imgHeight), paint);

Choose a reason for hiding this comment

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

The code seems not to build. Maybe there is a missing '}' here ?

image.SetPixel(x, y, sKColor);
}
}
SKColor[] skColors = colors.Select(ClampColor).ToArray();

Choose a reason for hiding this comment

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

It seems to be missing "using System.Linq;" as well to build properly

This comment was marked as off-topic.

This comment was marked as off-topic.

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.

5 participants