Review fixes

This commit is contained in:
Julien Richard 2022-01-16 10:19:50 +01:00
commit 66703b5ae3
3 changed files with 20 additions and 16 deletions

View file

@ -186,7 +186,8 @@ namespace Greenshot.Editor.Drawing
var rotationAngle = GetRotationAngle();
if (rotationAngle != 0)
{
graphics.DrawImage(RotateImage(bitmap, rotationAngle), Bounds);
using var newBitmap = RotateImage(bitmap, rotationAngle);
graphics.DrawImage(RotateImage(newBitmap, rotationAngle), Bounds);
return;
}

View file

@ -179,19 +179,22 @@ namespace Greenshot.Editor.Drawing
/// <param name="matrix"></param>
public override void Transform(Matrix matrix)
{
if (image != null)
if (image == null)
{
int rotateAngle = CalculateAngle(matrix);
// we currently assume only one transformation has been made.
if (rotateAngle != 0)
base.Transform(matrix);
return;
}
int rotateAngle = CalculateAngle(matrix);
// we currently assume only one transformation has been made.
if (rotateAngle != 0)
{
Log.DebugFormat("Rotating element with {0} degrees.", rotateAngle);
DisposeShadow();
using var tmpMatrix = new Matrix();
using (image)
{
Log.DebugFormat("Rotating element with {0} degrees.", rotateAngle);
DisposeShadow();
using var tmpMatrix = new Matrix();
using (image)
{
image = ImageHelper.ApplyEffect(image, new RotateEffect(rotateAngle), tmpMatrix);
}
image = ImageHelper.ApplyEffect(image, new RotateEffect(rotateAngle), tmpMatrix);
}
}

View file

@ -65,6 +65,8 @@ namespace Greenshot.Editor.Forms
private static readonly List<IImageEditor> EditorList = new List<IImageEditor>();
private static bool? _emojifontInstalled;
private Surface _surface;
private GreenshotToolStripButton[] _toolbarButtons;
@ -190,10 +192,8 @@ namespace Greenshot.Editor.Forms
private void HideEmojiButtonWhenFontIsNotInstalled()
{
if (!FontFamily.Families.Any(f => string.Equals(f.Name, "Segoe UI Emoji", StringComparison.OrdinalIgnoreCase)))
{
btnEmoji.Visible = false;
}
_emojifontInstalled ??= FontFamily.Families.Any(f => string.Equals(f.Name, "Segoe UI Emoji", StringComparison.OrdinalIgnoreCase));
btnEmoji.Visible = _emojifontInstalled.Value;
}
/// <summary>