Browse Source

Merge branch 'master' into allow-auto-hide-attached

pull/6054/head
Dariusz Komosiński 5 years ago
committed by GitHub
parent
commit
d167b54260
No known key found for this signature in database GPG Key ID: 4AEE18F83AFDEB23
  1. 11
      native/Avalonia.Native/src/OSX/window.mm
  2. 5
      samples/ControlCatalog/MainWindow.xaml.cs
  3. 3
      samples/RenderDemo/MainWindow.xaml
  4. 25
      samples/RenderDemo/Pages/CustomAnimatorPage.xaml
  5. 27
      samples/RenderDemo/Pages/CustomAnimatorPage.xaml.cs
  6. 16
      samples/RenderDemo/Pages/CustomStringAnimator.cs
  7. 29
      src/Avalonia.Animation/Animation.cs
  8. 2
      src/Avalonia.Controls/Remote/RemoteWidget.cs
  9. 4
      src/Avalonia.Diagnostics/Diagnostics/DevTools.cs
  10. 5
      src/Avalonia.Diagnostics/Diagnostics/DevToolsOptions.cs
  11. 9
      src/Avalonia.Diagnostics/Diagnostics/ViewModels/MainViewModel.cs
  12. 20
      src/Avalonia.Diagnostics/Diagnostics/Views/MainWindow.xaml.cs
  13. 47
      src/Skia/Avalonia.Skia/DrawingContextImpl.cs
  14. 4
      src/Skia/Avalonia.Skia/FormattedTextImpl.cs
  15. 4
      src/Windows/Avalonia.Direct2D1/Media/AvaloniaTextRenderer.cs
  16. 32
      src/Windows/Avalonia.Direct2D1/Media/DrawingContextImpl.cs
  17. 7
      src/Windows/Avalonia.Direct2D1/Media/LinearGradientBrushImpl.cs
  18. 11
      src/Windows/Avalonia.Direct2D1/Media/RadialGradientBrushImpl.cs
  19. 40
      tests/Avalonia.RenderTests/Media/ConicGradientBrushTests.cs
  20. 36
      tests/Avalonia.RenderTests/Media/LinearGradientBrushTests.cs
  21. 34
      tests/Avalonia.RenderTests/Media/RadialGradientBrushTests.cs
  22. BIN
      tests/TestFiles/Direct2D1/Media/ConicGradientBrush/ConicGradientBrush_DrawingContext.expected.png
  23. BIN
      tests/TestFiles/Direct2D1/Media/LinearGradientBrush/LinearGradientBrush_DrawingContext.expected.png
  24. BIN
      tests/TestFiles/Direct2D1/Media/RadialGradientBrush/RadialGradientBrush_DrawingContext.expected.png
  25. BIN
      tests/TestFiles/Skia/Media/ConicGradientBrush/ConicGradientBrush_DrawingContext.expected.png
  26. BIN
      tests/TestFiles/Skia/Media/LinearGradientBrush/LinearGradientBrush_DrawingContext.expected.png
  27. BIN
      tests/TestFiles/Skia/Media/RadialGradientBrush/RadialGradientBrush_DrawingContext.expected.png

11
native/Avalonia.Native/src/OSX/window.mm

@ -513,6 +513,7 @@ private:
bool _fullScreenActive; bool _fullScreenActive;
SystemDecorations _decorations; SystemDecorations _decorations;
AvnWindowState _lastWindowState; AvnWindowState _lastWindowState;
AvnWindowState _actualWindowState;
bool _inSetWindowState; bool _inSetWindowState;
NSRect _preZoomSize; NSRect _preZoomSize;
bool _transitioningWindowState; bool _transitioningWindowState;
@ -539,6 +540,7 @@ private:
_transitioningWindowState = false; _transitioningWindowState = false;
_inSetWindowState = false; _inSetWindowState = false;
_lastWindowState = Normal; _lastWindowState = Normal;
_actualWindowState = Normal;
WindowEvents = events; WindowEvents = events;
[Window setCanBecomeKeyAndMain]; [Window setCanBecomeKeyAndMain];
[Window disableCursorRects]; [Window disableCursorRects];
@ -633,7 +635,7 @@ private:
void WindowStateChanged () override void WindowStateChanged () override
{ {
if(!_inSetWindowState && !_transitioningWindowState) if(_shown && !_inSetWindowState && !_transitioningWindowState)
{ {
AvnWindowState state; AvnWindowState state;
GetWindowState(&state); GetWindowState(&state);
@ -963,14 +965,14 @@ private:
{ {
@autoreleasepool @autoreleasepool
{ {
if(_lastWindowState == state) if(_actualWindowState == state)
{ {
return S_OK; return S_OK;
} }
_inSetWindowState = true; _inSetWindowState = true;
auto currentState = _lastWindowState; auto currentState = _actualWindowState;
_lastWindowState = state; _lastWindowState = state;
if(currentState == Normal) if(currentState == Normal)
@ -1049,8 +1051,11 @@ private:
} }
break; break;
} }
_actualWindowState = _lastWindowState;
} }
_inSetWindowState = false; _inSetWindowState = false;
return S_OK; return S_OK;

5
samples/ControlCatalog/MainWindow.xaml.cs

@ -17,7 +17,10 @@ namespace ControlCatalog
public MainWindow() public MainWindow()
{ {
this.InitializeComponent(); this.InitializeComponent();
this.AttachDevTools(); this.AttachDevTools(new Avalonia.Diagnostics.DevToolsOptions()
{
StartupScreenIndex = 1,
});
//Renderer.DrawFps = true; //Renderer.DrawFps = true;
//Renderer.DrawDirtyRects = Renderer.DrawFps = true; //Renderer.DrawDirtyRects = Renderer.DrawFps = true;

3
samples/RenderDemo/MainWindow.xaml

@ -36,6 +36,9 @@
<TabItem Header="Transitions"> <TabItem Header="Transitions">
<pages:TransitionsPage/> <pages:TransitionsPage/>
</TabItem> </TabItem>
<TabItem Header="Custom Animator">
<pages:CustomAnimatorPage/>
</TabItem>
<TabItem Header="Clipping"> <TabItem Header="Clipping">
<pages:ClippingPage/> <pages:ClippingPage/>
</TabItem> </TabItem>

25
samples/RenderDemo/Pages/CustomAnimatorPage.xaml

@ -0,0 +1,25 @@
<UserControl
xmlns="https://github.com/avaloniaui"
xmlns:x="http://schemas.microsoft.com/winfx/2006/xaml"
xmlns:pages="clr-namespace:RenderDemo.Pages"
x:Class="RenderDemo.Pages.CustomAnimatorPage"
MaxWidth="600">
<Grid>
<TextBlock HorizontalAlignment="Center" VerticalAlignment="Center">
<TextBlock.Styles>
<Style Selector="TextBlock">
<Style.Animations>
<Animation Duration="0:0:1" IterationCount="Infinite">
<KeyFrame Cue="0%">
<Setter Property="Text" Value="" Animation.Animator="{x:Type pages:CustomStringAnimator}"/>
</KeyFrame>
<KeyFrame Cue="100%">
<Setter Property="Text" Value="0123456789" Animation.Animator="{x:Type pages:CustomStringAnimator}"/>
</KeyFrame>
</Animation>
</Style.Animations>
</Style>
</TextBlock.Styles>
</TextBlock>
</Grid>
</UserControl>

27
samples/RenderDemo/Pages/CustomAnimatorPage.xaml.cs

@ -0,0 +1,27 @@
using System.Reactive.Linq;
using Avalonia;
using Avalonia.Animation;
using Avalonia.Controls;
using Avalonia.Controls.Shapes;
using Avalonia.Data;
using Avalonia.Input;
using Avalonia.Interactivity;
using Avalonia.Markup.Xaml;
using Avalonia.Media;
using RenderDemo.ViewModels;
namespace RenderDemo.Pages
{
public class CustomAnimatorPage : UserControl
{
public CustomAnimatorPage()
{
InitializeComponent();
}
private void InitializeComponent()
{
AvaloniaXamlLoader.Load(this);
}
}
}

16
samples/RenderDemo/Pages/CustomStringAnimator.cs

@ -0,0 +1,16 @@
using Avalonia.Animation.Animators;
namespace RenderDemo.Pages
{
public class CustomStringAnimator : Animator<string>
{
public override string Interpolate(double progress, string oldValue, string newValue)
{
if (newValue.Length == 0) return "";
var step = 1.0 / newValue.Length;
var length = (int)(progress / step);
var result = newValue.Substring(0, length + 1);
return result;
}
}
}

29
src/Avalonia.Animation/Animation.cs

@ -194,6 +194,33 @@ namespace Avalonia.Animation
[Content] [Content]
public KeyFrames Children { get; } = new KeyFrames(); public KeyFrames Children { get; } = new KeyFrames();
// Store values for the Animator attached properties for IAnimationSetter objects.
private static readonly Dictionary<IAnimationSetter, Type> s_animators = new Dictionary<IAnimationSetter, Type>();
/// <summary>
/// Gets the value of the Animator attached property for a setter.
/// </summary>
/// <param name="setter">The animation setter.</param>
/// <returns>The property animator type.</returns>
public static Type GetAnimator(IAnimationSetter setter)
{
if (s_animators.TryGetValue(setter, out var type))
{
return type;
}
return null;
}
/// <summary>
/// Sets the value of the Animator attached property for a setter.
/// </summary>
/// <param name="setter">The animation setter.</param>
/// <param name="value">The property animator value.</param>
public static void SetAnimator(IAnimationSetter setter, Type value)
{
s_animators[setter] = value;
}
private readonly static List<(Func<AvaloniaProperty, bool> Condition, Type Animator)> Animators = new List<(Func<AvaloniaProperty, bool>, Type)> private readonly static List<(Func<AvaloniaProperty, bool> Condition, Type Animator)> Animators = new List<(Func<AvaloniaProperty, bool>, Type)>
{ {
( prop => typeof(bool).IsAssignableFrom(prop.PropertyType), typeof(BoolAnimator) ), ( prop => typeof(bool).IsAssignableFrom(prop.PropertyType), typeof(BoolAnimator) ),
@ -248,7 +275,7 @@ namespace Avalonia.Animation
{ {
foreach (var setter in keyframe.Setters) foreach (var setter in keyframe.Setters)
{ {
var handler = GetAnimatorType(setter.Property); var handler = Animation.GetAnimator(setter) ?? GetAnimatorType(setter.Property);
if (handler == null) if (handler == null)
{ {

2
src/Avalonia.Controls/Remote/RemoteWidget.cs

@ -70,7 +70,7 @@ namespace Avalonia.Controls.Remote
public override void Render(DrawingContext context) public override void Render(DrawingContext context)
{ {
if (_lastFrame != null) if (_lastFrame != null && _lastFrame.Width != 0 && _lastFrame.Height != 0)
{ {
var fmt = (PixelFormat) _lastFrame.Format; var fmt = (PixelFormat) _lastFrame.Format;
if (_bitmap == null || _bitmap.PixelSize.Width != _lastFrame.Width || if (_bitmap == null || _bitmap.PixelSize.Width != _lastFrame.Width ||

4
src/Avalonia.Diagnostics/Diagnostics/DevTools.cs

@ -10,7 +10,8 @@ namespace Avalonia.Diagnostics
{ {
public static class DevTools public static class DevTools
{ {
private static readonly Dictionary<TopLevel, Window> s_open = new Dictionary<TopLevel, Window>(); private static readonly Dictionary<TopLevel, MainWindow> s_open =
new Dictionary<TopLevel, MainWindow>();
public static IDisposable Attach(TopLevel root, KeyGesture gesture) public static IDisposable Attach(TopLevel root, KeyGesture gesture)
{ {
@ -52,6 +53,7 @@ namespace Avalonia.Diagnostics
Width = options.Size.Width, Width = options.Size.Width,
Height = options.Size.Height, Height = options.Size.Height,
}; };
window.SetOptions(options);
window.Closed += DevToolsClosed; window.Closed += DevToolsClosed;
s_open.Add(root, window); s_open.Add(root, window);

5
src/Avalonia.Diagnostics/Diagnostics/DevToolsOptions.cs

@ -22,5 +22,10 @@ namespace Avalonia.Diagnostics
/// Gets or sets the initial size of the DevTools window. The default value is 1280x720. /// Gets or sets the initial size of the DevTools window. The default value is 1280x720.
/// </summary> /// </summary>
public Size Size { get; set; } = new Size(1280, 720); public Size Size { get; set; } = new Size(1280, 720);
/// <summary>
/// Get or set the startup screen index where the DevTools window will be displayed.
/// </summary>
public int? StartupScreenIndex { get; set; }
} }
} }

9
src/Avalonia.Diagnostics/Diagnostics/ViewModels/MainViewModel.cs

@ -151,7 +151,7 @@ namespace Avalonia.Diagnostics.ViewModels
get { return _pointerOverElement; } get { return _pointerOverElement; }
private set { RaiseAndSetIfChanged(ref _pointerOverElement, value); } private set { RaiseAndSetIfChanged(ref _pointerOverElement, value); }
} }
private void UpdateConsoleContext(ConsoleContext context) private void UpdateConsoleContext(ConsoleContext context)
{ {
context.root = _root; context.root = _root;
@ -213,5 +213,12 @@ namespace Avalonia.Diagnostics.ViewModels
tree.SelectControl(control); tree.SelectControl(control);
} }
} }
public int? StartupScreenIndex { get; private set; } = default;
public void SetOptions(DevToolsOptions options)
{
StartupScreenIndex = options.StartupScreenIndex;
}
} }
} }

20
src/Avalonia.Diagnostics/Diagnostics/Views/MainWindow.xaml.cs

@ -24,6 +24,23 @@ namespace Avalonia.Diagnostics.Views
_keySubscription = InputManager.Instance.Process _keySubscription = InputManager.Instance.Process
.OfType<RawKeyEventArgs>() .OfType<RawKeyEventArgs>()
.Subscribe(RawKeyDown); .Subscribe(RawKeyDown);
EventHandler? lh = default;
lh = (s, e) =>
{
this.Opened -= lh;
if ((DataContext as MainViewModel)?.StartupScreenIndex is int index)
{
var screens = this.Screens;
if (index > -1 && index < screens.ScreenCount)
{
var screen = screens.All[index];
this.Position = screen.Bounds.TopLeft;
this.WindowState = WindowState.Maximized;
}
}
};
this.Opened += lh;
} }
public TopLevel? Root public TopLevel? Root
@ -115,5 +132,8 @@ namespace Avalonia.Diagnostics.Views
} }
private void RootClosed(object? sender, EventArgs e) => Close(); private void RootClosed(object? sender, EventArgs e) => Close();
public void SetOptions(DevToolsOptions options) =>
(DataContext as MainViewModel)?.SetOptions(options);
} }
} }

47
src/Skia/Avalonia.Skia/DrawingContextImpl.cs

@ -164,7 +164,7 @@ namespace Avalonia.Skia
/// <inheritdoc /> /// <inheritdoc />
public void DrawLine(IPen pen, Point p1, Point p2) public void DrawLine(IPen pen, Point p1, Point p2)
{ {
using (var paint = CreatePaint(_strokePaint, pen, new Size(Math.Abs(p2.X - p1.X), Math.Abs(p2.Y - p1.Y)))) using (var paint = CreatePaint(_strokePaint, pen, new Rect(p1, p2).Normalize()))
{ {
if (paint.Paint is object) if (paint.Paint is object)
{ {
@ -177,10 +177,10 @@ namespace Avalonia.Skia
public void DrawGeometry(IBrush brush, IPen pen, IGeometryImpl geometry) public void DrawGeometry(IBrush brush, IPen pen, IGeometryImpl geometry)
{ {
var impl = (GeometryImpl) geometry; var impl = (GeometryImpl) geometry;
var size = geometry.Bounds.Size; var rect = geometry.Bounds;
using (var fill = brush != null ? CreatePaint(_fillPaint, brush, size) : default(PaintWrapper)) using (var fill = brush != null ? CreatePaint(_fillPaint, brush, rect) : default(PaintWrapper))
using (var stroke = pen?.Brush != null ? CreatePaint(_strokePaint, pen, size) : default(PaintWrapper)) using (var stroke = pen?.Brush != null ? CreatePaint(_strokePaint, pen, rect) : default(PaintWrapper))
{ {
if (fill.Paint != null) if (fill.Paint != null)
{ {
@ -354,7 +354,7 @@ namespace Avalonia.Skia
if (brush != null) if (brush != null)
{ {
using (var paint = CreatePaint(_fillPaint, brush, rect.Rect.Size)) using (var paint = CreatePaint(_fillPaint, brush, rect.Rect))
{ {
if (isRounded) if (isRounded)
{ {
@ -397,7 +397,7 @@ namespace Avalonia.Skia
if (pen?.Brush != null) if (pen?.Brush != null)
{ {
using (var paint = CreatePaint(_strokePaint, pen, rect.Rect.Size)) using (var paint = CreatePaint(_strokePaint, pen, rect.Rect))
{ {
if (paint.Paint is object) if (paint.Paint is object)
{ {
@ -417,7 +417,7 @@ namespace Avalonia.Skia
/// <inheritdoc /> /// <inheritdoc />
public void DrawText(IBrush foreground, Point origin, IFormattedTextImpl text) public void DrawText(IBrush foreground, Point origin, IFormattedTextImpl text)
{ {
using (var paint = CreatePaint(_fillPaint, foreground, text.Bounds.Size)) using (var paint = CreatePaint(_fillPaint, foreground, text.Bounds))
{ {
var textImpl = (FormattedTextImpl) text; var textImpl = (FormattedTextImpl) text;
textImpl.Draw(this, Canvas, origin.ToSKPoint(), paint, _canTextUseLcdRendering); textImpl.Draw(this, Canvas, origin.ToSKPoint(), paint, _canTextUseLcdRendering);
@ -427,7 +427,7 @@ namespace Avalonia.Skia
/// <inheritdoc /> /// <inheritdoc />
public void DrawGlyphRun(IBrush foreground, GlyphRun glyphRun) public void DrawGlyphRun(IBrush foreground, GlyphRun glyphRun)
{ {
using (var paintWrapper = CreatePaint(_fillPaint, foreground, glyphRun.Size)) using (var paintWrapper = CreatePaint(_fillPaint, foreground, new Rect(glyphRun.Size)))
{ {
var glyphRunImpl = (GlyphRunImpl)glyphRun.GlyphRunImpl; var glyphRunImpl = (GlyphRunImpl)glyphRun.GlyphRunImpl;
@ -537,7 +537,7 @@ namespace Avalonia.Skia
var paint = new SKPaint(); var paint = new SKPaint();
Canvas.SaveLayer(paint); Canvas.SaveLayer(paint);
_maskStack.Push(CreatePaint(paint, mask, bounds.Size, true)); _maskStack.Push(CreatePaint(paint, mask, bounds, true));
} }
/// <inheritdoc /> /// <inheritdoc />
@ -593,18 +593,19 @@ namespace Avalonia.Skia
/// <param name="paintWrapper">Paint wrapper.</param> /// <param name="paintWrapper">Paint wrapper.</param>
/// <param name="targetSize">Target size.</param> /// <param name="targetSize">Target size.</param>
/// <param name="gradientBrush">Gradient brush.</param> /// <param name="gradientBrush">Gradient brush.</param>
private void ConfigureGradientBrush(ref PaintWrapper paintWrapper, Size targetSize, IGradientBrush gradientBrush) private void ConfigureGradientBrush(ref PaintWrapper paintWrapper, Rect targetRect, IGradientBrush gradientBrush)
{ {
var tileMode = gradientBrush.SpreadMethod.ToSKShaderTileMode(); var tileMode = gradientBrush.SpreadMethod.ToSKShaderTileMode();
var stopColors = gradientBrush.GradientStops.Select(s => s.Color.ToSKColor()).ToArray(); var stopColors = gradientBrush.GradientStops.Select(s => s.Color.ToSKColor()).ToArray();
var stopOffsets = gradientBrush.GradientStops.Select(s => (float)s.Offset).ToArray(); var stopOffsets = gradientBrush.GradientStops.Select(s => (float)s.Offset).ToArray();
var position = targetRect.Position.ToSKPoint();
switch (gradientBrush) switch (gradientBrush)
{ {
case ILinearGradientBrush linearGradient: case ILinearGradientBrush linearGradient:
{ {
var start = linearGradient.StartPoint.ToPixels(targetSize).ToSKPoint(); var start = position + linearGradient.StartPoint.ToPixels(targetRect.Size).ToSKPoint();
var end = linearGradient.EndPoint.ToPixels(targetSize).ToSKPoint(); var end = position + linearGradient.EndPoint.ToPixels(targetRect.Size).ToSKPoint();
// would be nice to cache these shaders possibly? // would be nice to cache these shaders possibly?
using (var shader = using (var shader =
@ -617,10 +618,10 @@ namespace Avalonia.Skia
} }
case IRadialGradientBrush radialGradient: case IRadialGradientBrush radialGradient:
{ {
var center = radialGradient.Center.ToPixels(targetSize).ToSKPoint(); var center = position + radialGradient.Center.ToPixels(targetRect.Size).ToSKPoint();
var radius = (float)(radialGradient.Radius * targetSize.Width); var radius = (float)(radialGradient.Radius * targetRect.Width);
var origin = radialGradient.GradientOrigin.ToPixels(targetSize).ToSKPoint(); var origin = position + radialGradient.GradientOrigin.ToPixels(targetRect.Size).ToSKPoint();
if (origin.Equals(center)) if (origin.Equals(center))
{ {
@ -665,7 +666,7 @@ namespace Avalonia.Skia
} }
case IConicGradientBrush conicGradient: case IConicGradientBrush conicGradient:
{ {
var center = conicGradient.Center.ToPixels(targetSize).ToSKPoint(); var center = position + conicGradient.Center.ToPixels(targetRect.Size).ToSKPoint();
// Skia's default is that angle 0 is from the right hand side of the center point // Skia's default is that angle 0 is from the right hand side of the center point
// but we are matching CSS where the vertical point above the center is 0. // but we are matching CSS where the vertical point above the center is 0.
@ -867,10 +868,10 @@ namespace Avalonia.Skia
/// </summary> /// </summary>
/// <param name="paint">The paint to wrap.</param> /// <param name="paint">The paint to wrap.</param>
/// <param name="brush">Source brush.</param> /// <param name="brush">Source brush.</param>
/// <param name="targetSize">Target size.</param> /// <param name="targetRect">Target rect.</param>
/// <param name="disposePaint">Optional dispose of the supplied paint.</param> /// <param name="disposePaint">Optional dispose of the supplied paint.</param>
/// <returns>Paint wrapper for given brush.</returns> /// <returns>Paint wrapper for given brush.</returns>
internal PaintWrapper CreatePaint(SKPaint paint, IBrush brush, Size targetSize, bool disposePaint = false) internal PaintWrapper CreatePaint(SKPaint paint, IBrush brush, Rect targetRect, bool disposePaint = false)
{ {
var paintWrapper = new PaintWrapper(paint, disposePaint); var paintWrapper = new PaintWrapper(paint, disposePaint);
@ -889,7 +890,7 @@ namespace Avalonia.Skia
if (brush is IGradientBrush gradient) if (brush is IGradientBrush gradient)
{ {
ConfigureGradientBrush(ref paintWrapper, targetSize, gradient); ConfigureGradientBrush(ref paintWrapper, targetRect, gradient);
return paintWrapper; return paintWrapper;
} }
@ -909,7 +910,7 @@ namespace Avalonia.Skia
if (tileBrush != null && tileBrushImage != null) if (tileBrush != null && tileBrushImage != null)
{ {
ConfigureTileBrush(ref paintWrapper, targetSize, tileBrush, tileBrushImage); ConfigureTileBrush(ref paintWrapper, targetRect.Size, tileBrush, tileBrushImage);
} }
else else
{ {
@ -924,10 +925,10 @@ namespace Avalonia.Skia
/// </summary> /// </summary>
/// <param name="paint">The paint to wrap.</param> /// <param name="paint">The paint to wrap.</param>
/// <param name="pen">Source pen.</param> /// <param name="pen">Source pen.</param>
/// <param name="targetSize">Target size.</param> /// <param name="targetRect">Target rect.</param>
/// <param name="disposePaint">Optional dispose of the supplied paint.</param> /// <param name="disposePaint">Optional dispose of the supplied paint.</param>
/// <returns></returns> /// <returns></returns>
private PaintWrapper CreatePaint(SKPaint paint, IPen pen, Size targetSize, bool disposePaint = false) private PaintWrapper CreatePaint(SKPaint paint, IPen pen, Rect targetRect, bool disposePaint = false)
{ {
// In Skia 0 thickness means - use hairline rendering // In Skia 0 thickness means - use hairline rendering
// and for us it means - there is nothing rendered. // and for us it means - there is nothing rendered.
@ -936,7 +937,7 @@ namespace Avalonia.Skia
return default; return default;
} }
var rv = CreatePaint(paint, pen.Brush, targetSize, disposePaint); var rv = CreatePaint(paint, pen.Brush, targetRect, disposePaint);
paint.IsStroke = true; paint.IsStroke = true;
paint.StrokeWidth = (float) pen.Thickness; paint.StrokeWidth = (float) pen.Thickness;

4
src/Skia/Avalonia.Skia/FormattedTextImpl.cs

@ -278,9 +278,9 @@ namespace Avalonia.Skia
if (fb != null) if (fb != null)
{ {
//TODO: figure out how to get the brush size //TODO: figure out how to get the brush rect
currentWrapper = context.CreatePaint(new SKPaint { IsAntialias = true }, fb, currentWrapper = context.CreatePaint(new SKPaint { IsAntialias = true }, fb,
new Size()); default);
} }
else else
{ {

4
src/Windows/Avalonia.Direct2D1/Media/AvaloniaTextRenderer.cs

@ -34,10 +34,10 @@ namespace Avalonia.Direct2D1.Media
{ {
var wrapper = clientDrawingEffect as BrushWrapper; var wrapper = clientDrawingEffect as BrushWrapper;
// TODO: Work out how to get the size below rather than passing new Size(). // TODO: Work out how to get the rect below rather than passing default.
var brush = (wrapper == null) ? var brush = (wrapper == null) ?
_foreground : _foreground :
_context.CreateBrush(wrapper.Brush, new Size()).PlatformBrush; _context.CreateBrush(wrapper.Brush, default).PlatformBrush;
_renderTarget.DrawGlyphRun( _renderTarget.DrawGlyphRun(
new RawVector2 { X = baselineOriginX, Y = baselineOriginY }, new RawVector2 { X = baselineOriginX, Y = baselineOriginY },

32
src/Windows/Avalonia.Direct2D1/Media/DrawingContextImpl.cs

@ -192,7 +192,7 @@ namespace Avalonia.Direct2D1.Media
{ {
using (var d2dSource = ((BitmapImpl)source.Item).GetDirect2DBitmap(_deviceContext)) using (var d2dSource = ((BitmapImpl)source.Item).GetDirect2DBitmap(_deviceContext))
using (var sourceBrush = new BitmapBrush(_deviceContext, d2dSource.Value)) using (var sourceBrush = new BitmapBrush(_deviceContext, d2dSource.Value))
using (var d2dOpacityMask = CreateBrush(opacityMask, opacityMaskRect.Size)) using (var d2dOpacityMask = CreateBrush(opacityMask, opacityMaskRect))
using (var geometry = new SharpDX.Direct2D1.RectangleGeometry(Direct2D1Platform.Direct2D1Factory, destRect.ToDirect2D())) using (var geometry = new SharpDX.Direct2D1.RectangleGeometry(Direct2D1Platform.Direct2D1Factory, destRect.ToDirect2D()))
{ {
if (d2dOpacityMask.PlatformBrush != null) if (d2dOpacityMask.PlatformBrush != null)
@ -217,9 +217,7 @@ namespace Avalonia.Direct2D1.Media
{ {
if (pen != null) if (pen != null)
{ {
var size = new Rect(p1, p2).Size; using (var d2dBrush = CreateBrush(pen.Brush, new Rect(p1, p2).Normalize()))
using (var d2dBrush = CreateBrush(pen.Brush, size))
using (var d2dStroke = pen.ToDirect2DStrokeStyle(_deviceContext)) using (var d2dStroke = pen.ToDirect2DStrokeStyle(_deviceContext))
{ {
if (d2dBrush.PlatformBrush != null) if (d2dBrush.PlatformBrush != null)
@ -245,7 +243,7 @@ namespace Avalonia.Direct2D1.Media
{ {
if (brush != null) if (brush != null)
{ {
using (var d2dBrush = CreateBrush(brush, geometry.Bounds.Size)) using (var d2dBrush = CreateBrush(brush, geometry.Bounds))
{ {
if (d2dBrush.PlatformBrush != null) if (d2dBrush.PlatformBrush != null)
{ {
@ -257,7 +255,7 @@ namespace Avalonia.Direct2D1.Media
if (pen != null) if (pen != null)
{ {
using (var d2dBrush = CreateBrush(pen.Brush, geometry.GetRenderBounds(pen).Size)) using (var d2dBrush = CreateBrush(pen.Brush, geometry.GetRenderBounds(pen)))
using (var d2dStroke = pen.ToDirect2DStrokeStyle(_deviceContext)) using (var d2dStroke = pen.ToDirect2DStrokeStyle(_deviceContext))
{ {
if (d2dBrush.PlatformBrush != null) if (d2dBrush.PlatformBrush != null)
@ -282,7 +280,7 @@ namespace Avalonia.Direct2D1.Media
if (brush != null) if (brush != null)
{ {
using (var b = CreateBrush(brush, rect.Size)) using (var b = CreateBrush(brush, rect))
{ {
if (b.PlatformBrush != null) if (b.PlatformBrush != null)
{ {
@ -311,7 +309,7 @@ namespace Avalonia.Direct2D1.Media
if (pen?.Brush != null) if (pen?.Brush != null)
{ {
using (var wrapper = CreateBrush(pen.Brush, rect.Size)) using (var wrapper = CreateBrush(pen.Brush, rect))
using (var d2dStroke = pen.ToDirect2DStrokeStyle(_deviceContext)) using (var d2dStroke = pen.ToDirect2DStrokeStyle(_deviceContext))
{ {
if (wrapper.PlatformBrush != null) if (wrapper.PlatformBrush != null)
@ -349,7 +347,7 @@ namespace Avalonia.Direct2D1.Media
{ {
var impl = (FormattedTextImpl)text; var impl = (FormattedTextImpl)text;
using (var brush = CreateBrush(foreground, impl.Bounds.Size)) using (var brush = CreateBrush(foreground, impl.Bounds))
using (var renderer = new AvaloniaTextRenderer(this, _deviceContext, brush.PlatformBrush)) using (var renderer = new AvaloniaTextRenderer(this, _deviceContext, brush.PlatformBrush))
{ {
if (brush.PlatformBrush != null) if (brush.PlatformBrush != null)
@ -367,7 +365,7 @@ namespace Avalonia.Direct2D1.Media
/// <param name="glyphRun">The glyph run.</param> /// <param name="glyphRun">The glyph run.</param>
public void DrawGlyphRun(IBrush foreground, GlyphRun glyphRun) public void DrawGlyphRun(IBrush foreground, GlyphRun glyphRun)
{ {
using (var brush = CreateBrush(foreground, glyphRun.Size)) using (var brush = CreateBrush(foreground, new Rect(glyphRun.Size)))
{ {
var glyphRunImpl = (GlyphRunImpl)glyphRun.GlyphRunImpl; var glyphRunImpl = (GlyphRunImpl)glyphRun.GlyphRunImpl;
@ -458,9 +456,9 @@ namespace Avalonia.Direct2D1.Media
/// Creates a Direct2D brush wrapper for a Avalonia brush. /// Creates a Direct2D brush wrapper for a Avalonia brush.
/// </summary> /// </summary>
/// <param name="brush">The avalonia brush.</param> /// <param name="brush">The avalonia brush.</param>
/// <param name="destinationSize">The size of the brush's target area.</param> /// <param name="destinationRect">The brush's target area.</param>
/// <returns>The Direct2D brush wrapper.</returns> /// <returns>The Direct2D brush wrapper.</returns>
public BrushImpl CreateBrush(IBrush brush, Size destinationSize) public BrushImpl CreateBrush(IBrush brush, Rect destinationRect)
{ {
var solidColorBrush = brush as ISolidColorBrush; var solidColorBrush = brush as ISolidColorBrush;
var linearGradientBrush = brush as ILinearGradientBrush; var linearGradientBrush = brush as ILinearGradientBrush;
@ -475,11 +473,11 @@ namespace Avalonia.Direct2D1.Media
} }
else if (linearGradientBrush != null) else if (linearGradientBrush != null)
{ {
return new LinearGradientBrushImpl(linearGradientBrush, _deviceContext, destinationSize); return new LinearGradientBrushImpl(linearGradientBrush, _deviceContext, destinationRect);
} }
else if (radialGradientBrush != null) else if (radialGradientBrush != null)
{ {
return new RadialGradientBrushImpl(radialGradientBrush, _deviceContext, destinationSize); return new RadialGradientBrushImpl(radialGradientBrush, _deviceContext, destinationRect);
} }
else if (conicGradientBrush != null) else if (conicGradientBrush != null)
{ {
@ -492,7 +490,7 @@ namespace Avalonia.Direct2D1.Media
imageBrush, imageBrush,
_deviceContext, _deviceContext,
(BitmapImpl)imageBrush.Source.PlatformImpl.Item, (BitmapImpl)imageBrush.Source.PlatformImpl.Item,
destinationSize); destinationRect.Size);
} }
else if (visualBrush != null) else if (visualBrush != null)
{ {
@ -523,7 +521,7 @@ namespace Avalonia.Direct2D1.Media
visualBrush, visualBrush,
_deviceContext, _deviceContext,
new D2DBitmapImpl(intermediate.Bitmap), new D2DBitmapImpl(intermediate.Bitmap),
destinationSize); destinationRect.Size);
} }
} }
} }
@ -574,7 +572,7 @@ namespace Avalonia.Direct2D1.Media
ContentBounds = PrimitiveExtensions.RectangleInfinite, ContentBounds = PrimitiveExtensions.RectangleInfinite,
MaskTransform = PrimitiveExtensions.Matrix3x2Identity, MaskTransform = PrimitiveExtensions.Matrix3x2Identity,
Opacity = 1, Opacity = 1,
OpacityBrush = CreateBrush(mask, bounds.Size).PlatformBrush OpacityBrush = CreateBrush(mask, bounds).PlatformBrush
}; };
var layer = _layerPool.Count != 0 ? _layerPool.Pop() : new Layer(_deviceContext); var layer = _layerPool.Count != 0 ? _layerPool.Pop() : new Layer(_deviceContext);
_deviceContext.PushLayer(ref parameters, layer); _deviceContext.PushLayer(ref parameters, layer);

7
src/Windows/Avalonia.Direct2D1/Media/LinearGradientBrushImpl.cs

@ -8,7 +8,7 @@ namespace Avalonia.Direct2D1.Media
public LinearGradientBrushImpl( public LinearGradientBrushImpl(
ILinearGradientBrush brush, ILinearGradientBrush brush,
SharpDX.Direct2D1.RenderTarget target, SharpDX.Direct2D1.RenderTarget target,
Size destinationSize) Rect destinationRect)
{ {
if (brush.GradientStops.Count == 0) if (brush.GradientStops.Count == 0)
{ {
@ -21,8 +21,9 @@ namespace Avalonia.Direct2D1.Media
Position = (float)s.Offset Position = (float)s.Offset
}).ToArray(); }).ToArray();
var startPoint = brush.StartPoint.ToPixels(destinationSize); var position = destinationRect.Position;
var endPoint = brush.EndPoint.ToPixels(destinationSize); var startPoint = position + brush.StartPoint.ToPixels(destinationRect.Size);
var endPoint = position + brush.EndPoint.ToPixels(destinationRect.Size);
using (var stops = new SharpDX.Direct2D1.GradientStopCollection( using (var stops = new SharpDX.Direct2D1.GradientStopCollection(
target, target,

11
src/Windows/Avalonia.Direct2D1/Media/RadialGradientBrushImpl.cs

@ -8,7 +8,7 @@ namespace Avalonia.Direct2D1.Media
public RadialGradientBrushImpl( public RadialGradientBrushImpl(
IRadialGradientBrush brush, IRadialGradientBrush brush,
SharpDX.Direct2D1.RenderTarget target, SharpDX.Direct2D1.RenderTarget target,
Size destinationSize) Rect destinationRect)
{ {
if (brush.GradientStops.Count == 0) if (brush.GradientStops.Count == 0)
{ {
@ -21,12 +21,13 @@ namespace Avalonia.Direct2D1.Media
Position = (float)s.Offset Position = (float)s.Offset
}).ToArray(); }).ToArray();
var centerPoint = brush.Center.ToPixels(destinationSize); var position = destinationRect.Position;
var gradientOrigin = brush.GradientOrigin.ToPixels(destinationSize) - centerPoint; var centerPoint = position + brush.Center.ToPixels(destinationRect.Size);
var gradientOrigin = position + brush.GradientOrigin.ToPixels(destinationRect.Size) - centerPoint;
// Note: Direct2D supports RadiusX and RadiusY but Cairo backend supports only Radius property // Note: Direct2D supports RadiusX and RadiusY but Cairo backend supports only Radius property
var radiusX = brush.Radius * destinationSize.Width; var radiusX = brush.Radius * destinationRect.Width;
var radiusY = brush.Radius * destinationSize.Height; var radiusY = brush.Radius * destinationRect.Height;
using (var stops = new SharpDX.Direct2D1.GradientStopCollection( using (var stops = new SharpDX.Direct2D1.GradientStopCollection(
target, target,

40
tests/Avalonia.RenderTests/Media/ConicGradientBrushTests.cs

@ -1,5 +1,6 @@
using Avalonia.Controls; using Avalonia.Controls;
using Avalonia.Media; using Avalonia.Media;
using System;
using System.Threading.Tasks; using System.Threading.Tasks;
using Xunit; using Xunit;
@ -174,5 +175,44 @@ namespace Avalonia.Direct2D1.RenderTests.Media
await RenderToFile(target); await RenderToFile(target);
CompareImages(); CompareImages();
} }
[Fact]
public async Task ConicGradientBrush_DrawingContext()
{
var brush = new ConicGradientBrush
{
GradientStops =
{
new GradientStop { Color = Colors.Red, Offset = 0 },
new GradientStop { Color = Colors.Yellow, Offset = 0.1667 },
new GradientStop { Color = Colors.Lime, Offset = 0.3333 },
new GradientStop { Color = Colors.Aqua, Offset = 0.5000 },
new GradientStop { Color = Colors.Blue, Offset = 0.6667 },
new GradientStop { Color = Colors.Magenta, Offset = 0.8333 },
new GradientStop { Color = Colors.Red, Offset = 1 },
}
};
Decorator target = new Decorator
{
Width = 200,
Height = 200,
Child = new DrawnControl(c =>
{
c.DrawRectangle(brush, null, new Rect(0, 0, 100, 100));
c.DrawRectangle(brush, null, new Rect(100, 100, 100, 100));
}),
};
await RenderToFile(target);
CompareImages();
}
private class DrawnControl : Control
{
private readonly Action<DrawingContext> _render;
public DrawnControl(Action<DrawingContext> render) => _render = render;
public override void Render(DrawingContext context) => _render(context);
}
} }
} }

36
tests/Avalonia.RenderTests/Media/LinearGradientBrushTests.cs

@ -72,5 +72,41 @@ namespace Avalonia.Direct2D1.RenderTests.Media
await RenderToFile(target); await RenderToFile(target);
CompareImages(); CompareImages();
} }
[Fact]
public async Task LinearGradientBrush_DrawingContext()
{
var brush = new LinearGradientBrush
{
StartPoint = new RelativePoint(0, 0, RelativeUnit.Relative),
EndPoint = new RelativePoint(1, 1, RelativeUnit.Relative),
GradientStops =
{
new GradientStop { Color = Colors.Red, Offset = 0 },
new GradientStop { Color = Colors.Blue, Offset = 1 }
}
};
Decorator target = new Decorator
{
Width = 200,
Height = 200,
Child = new DrawnControl(c =>
{
c.DrawRectangle(brush, null, new Rect(0, 0, 100, 100));
c.DrawRectangle(brush, null, new Rect(100, 100, 100, 100));
}),
};
await RenderToFile(target);
CompareImages();
}
private class DrawnControl : Control
{
private readonly Action<DrawingContext> _render;
public DrawnControl(Action<DrawingContext> render) => _render = render;
public override void Render(DrawingContext context) => _render(context);
}
} }
} }

34
tests/Avalonia.RenderTests/Media/RadialGradientBrushTests.cs

@ -165,5 +165,39 @@ namespace Avalonia.Direct2D1.RenderTests.Media
await RenderToFile(target); await RenderToFile(target);
CompareImages(); CompareImages();
} }
[Fact]
public async Task RadialGradientBrush_DrawingContext()
{
var brush = new RadialGradientBrush
{
GradientStops =
{
new GradientStop { Color = Colors.Red, Offset = 0 },
new GradientStop { Color = Colors.Blue, Offset = 1 }
}
};
Decorator target = new Decorator
{
Width = 200,
Height = 200,
Child = new DrawnControl(c =>
{
c.DrawRectangle(brush, null, new Rect(0, 0, 100, 100));
c.DrawRectangle(brush, null, new Rect(100, 100, 100, 100));
}),
};
await RenderToFile(target);
CompareImages();
}
private class DrawnControl : Control
{
private readonly Action<DrawingContext> _render;
public DrawnControl(Action<DrawingContext> render) => _render = render;
public override void Render(DrawingContext context) => _render(context);
}
} }
} }

BIN
tests/TestFiles/Direct2D1/Media/ConicGradientBrush/ConicGradientBrush_DrawingContext.expected.png

Binary file not shown.

After

Width:  |  Height:  |  Size: 631 B

BIN
tests/TestFiles/Direct2D1/Media/LinearGradientBrush/LinearGradientBrush_DrawingContext.expected.png

Binary file not shown.

After

Width:  |  Height:  |  Size: 2.9 KiB

BIN
tests/TestFiles/Direct2D1/Media/RadialGradientBrush/RadialGradientBrush_DrawingContext.expected.png

Binary file not shown.

After

Width:  |  Height:  |  Size: 8.9 KiB

BIN
tests/TestFiles/Skia/Media/ConicGradientBrush/ConicGradientBrush_DrawingContext.expected.png

Binary file not shown.

After

Width:  |  Height:  |  Size: 9.8 KiB

BIN
tests/TestFiles/Skia/Media/LinearGradientBrush/LinearGradientBrush_DrawingContext.expected.png

Binary file not shown.

After

Width:  |  Height:  |  Size: 1.9 KiB

BIN
tests/TestFiles/Skia/Media/RadialGradientBrush/RadialGradientBrush_DrawingContext.expected.png

Binary file not shown.

After

Width:  |  Height:  |  Size: 5.8 KiB

Loading…
Cancel
Save