FEATURE-863: Fixed storing of the start value.

Updated SVG dependency, which fixes a few small issues with the Jira plug-in.

Greenshot/Drawing/StepLabelContainer.cs

@@ -57,6 +57,8 @@ namespace Greenshot.Drawing {
 		#region Number serializing
 		// Used to store the number of this label, so when deserializing it can be placed back to the StepLabels list in the right location
 		private int _number;
+		// Used to store the counter start of the Surface, as the surface is NOT stored.
+		private int _counterStart = 1;
 		public int Number {
 			get {
 				return _number;
@@ -74,6 +76,7 @@ namespace Greenshot.Drawing {
 		private void SetValuesOnSerializing(StreamingContext context) {
 			if (Parent != null) {
 				Number = ((Surface)Parent).CountStepLabels(this);
+				_counterStart = ((Surface) Parent).CounterStart;
 			}
 		}
 		#endregion
@@ -90,6 +93,7 @@ namespace Greenshot.Drawing {
 				Alignment = StringAlignment.Center,
 				LineAlignment = StringAlignment.Center
 			};
+
 		}
 
 		/// <summary>
@@ -103,10 +107,13 @@ namespace Greenshot.Drawing {
 			}
 			((Surface) Parent)?.RemoveStepLabel(this);
 			base.SwitchParent(newParent);
-			if (newParent != null) {
-				((Surface)Parent)?.AddStepLabel(this);
+			if (newParent == null)
+			{
+				return;
 			}
-
+			// Make sure the counter start is restored (this unfortunately happens multiple times... -> hack)
+			newParent.CounterStart = _counterStart;
+			newParent.AddStepLabel(this);
 		}
 
 		public override Size DefaultSize => new Size(30, 30);

Greenshot/Drawing/Surface.cs

@@ -269,12 +269,14 @@ namespace Greenshot.Drawing
 			get { return _counterStart; }
 			set
 			{
-				if (_propertyChanged != null && _counterStart != value)
+				if (_counterStart == value)
 				{
-					_counterStart = value;
-					_propertyChanged(this, new PropertyChangedEventArgs("CounterStart"));
-					Invalidate();
+					return;
 				}
+
+				_counterStart = value;
+				Invalidate();
+				_propertyChanged?.Invoke(this, new PropertyChangedEventArgs("CounterStart"));
 			}
 		}
 

Greenshot/Forms/ImageEditorForm.cs

@@ -207,6 +207,8 @@ namespace Greenshot {
 			// Workaround: for the MouseWheel event which doesn't get to the panel
 			MouseWheel += PanelMouseWheel;
 
+			// Make sure the value is set correctly when starting
+			counterUpDown.Value = Surface.CounterStart;
 			ApplyLanguage();
 		}
 

GreenshotJiraPlugin/GreenshotJiraPlugin.csproj

@@ -49,8 +49,8 @@
     <Reference Include="log4net">
       <HintPath>..\Greenshot\Lib\log4net.dll</HintPath>
     </Reference>
-    <Reference Include="Svg, Version=2.2.1.1459, Culture=neutral, PublicKeyToken=12a0bac221edeae2, processorArchitecture=MSIL">
-      <HintPath>..\packages\Svg.2.2.1\lib\net35\Svg.dll</HintPath>
+    <Reference Include="Svg, Version=2.2.1.38382, Culture=neutral, PublicKeyToken=12a0bac221edeae2, processorArchitecture=MSIL">
+      <HintPath>..\packages\Svg.2.2.2\lib\net35\Svg.dll</HintPath>
       <Private>True</Private>
     </Reference>
     <Reference Include="System" />

GreenshotJiraPlugin/packages.config

@@ -4,5 +4,5 @@
   <package id="Dapplo.Jira" version="0.1.65" targetFramework="net45" />
   <package id="Dapplo.Log.Facade" version="0.5.4" targetFramework="net45" />
   <package id="LibZ.Tool" version="1.2.0.0" targetFramework="net45" />
-  <package id="Svg" version="2.2.1" targetFramework="net45" />
+  <package id="Svg" version="2.2.2" targetFramework="net45" />
 </packages>

GreenshotPlugin/Interfaces/Generic.cs

@@ -111,6 +111,9 @@ namespace Greenshot.Plugin
 		event SurfaceDrawingModeEventHandler DrawingModeChanged;
 		event SurfaceElementEventHandler MovingElementChanged;
 
+		/// <summary>
+		/// Start valueof the step-labels (counts)
+		/// </summary>
 		int CounterStart { get; set; }
 
 		/// <summary>