Anton Firszov

Results 169 comments of Anton Firszov

If filling polygons also reproduces the issue, please also try: - `ctx.Fill(Color.Red)` - `ctx.Fill(someNonSolidBrush)` (Eg. `Brushes.Percent20Pattern`)

Hah, this seems to be an easy question but it isn't. Currently the [`LinearLineSegment(PointF[])`](https://github.com/SixLabors/ImageSharp.Drawing/blob/c29e5de8bb1ed0afd11d71cb75ede5fd4bee0644/src/ImageSharp.Drawing/Shapes/LinearLineSegment.cs#L46-L53) constructor takes ownership of the array without copying, this makes sense since it avoids an allocation...

This is a reasonable request, but we won't be able to make it into 1.0 (see https://github.com/SixLabors/ImageSharp.Drawing/issues/134#issuecomment-839965138).

Actually, I haven't considered an options struct, we should explore that too. It could help avoiding the Pen Brush mutability asymmetry, which I don't like much, just proposed as a...

@tocsoft since you are active these days: any thoughts on this? Any alternative ideas to avoid the constructor bloat?

I also think now that the `PenOptions` direction would be better since it preserves immutability. +1 for turning `IBrush` into an abstract class for consistency.

I would consider this out of scope now, I believe we should focus on drawing & fonts feature gaps + performance issues. @tocsoft moving to Future for now, if you...

This looks like a very useful feature, but I doubt we can make it into 1.0, moving to Future.

@SirusDoma dumb, slow heuristic approach, but might be good enough: you can use repeated calls of `TextMeasurer.Measure` to get a size that approximately fits your destination area, then maybe using...

Makes sense, but low priority, moving to Future. Community PR-s are welcome.