Fixed some more stuff

.appcast.xml

@@ -5,6 +5,11 @@
     <link>https://raw.githubusercontent.com/DWilliames/paddy-sketch-plugin/master/.appcast.xml</link>
     <description>Automated padding, spacing and alignment for your Sketch layers</description>
     <language>en</language>
+    <item>
+      <title>Version 1.0.6</title>
+      <description>Squashed some more bugs</description>
+      <enclosure url="https://github.com/DWilliames/paddy-sketch-plugin/releases/download/v1.0.6/Paddy.sketchplugin.zip" sparkle:version="1.0.6" />
+    </item>
     <item>
       <title>Version 1.0.5</title>
       <description>More bug fixes</description>

Paddy.sketchplugin/Contents/Sketch/async.js

@@ -1,3 +1,4 @@
+// Work in progress
 
 function loadFramework(name, className) {
   if (NSClassFromString(className) == null) {

Paddy.sketchplugin/Contents/Sketch/layers.js

@@ -17,7 +17,7 @@ function getContainerFrameForBGLayer(bg) {
       return false
     if (layer.isMemberOfClass(MSSliceLayer))
       return false
-    if (layer.name().startsWith('-'))
+    if (shouldLayerBeIgnored(layer))
       return false
     return true
   })
@@ -36,13 +36,24 @@ function getContainerFrameForBGLayer(bg) {
 }
 
 
+// If the layer should be ignored when calculating the rect of a layer
+function shouldLayerBeIgnored(layer) {
+  if (!layer) return
+
+  if (layer.name().startsWith('-')) {
+    return true
+  }
+
+  return false
+}
+
 // Whether to check for Stack Groups
 var takeIntoAccountStackViews = true
 
 
 // Return the rect for a layer as an MSRect
 function rectForLayer(layer, ignoreWithPrefix) {
-  if (!layer || (ignoreWithPrefix && layer.name().startsWith('-'))) {
+  if (!layer || (ignoreWithPrefix && shouldLayerBeIgnored(layer))) {
     return
   }
 
@@ -161,7 +172,7 @@ function getBGCandidateFromLayers(layers) {
 
   var candidate
   var existingBackground = layers.find(function(layer) {
-    if (layer.name().startsWith('-'))
+    if (shouldLayerBeIgnored(layer))
       return false
 
     if (canLayerHavePadding(layer)) {

Paddy.sketchplugin/Contents/Sketch/main.js

@@ -332,11 +332,15 @@ function selectionChanged(context) {
   startBenchmark()
   document = context.actionContext.document
 
+  // print('Selection changed - OLD/NEW')
+  // print(context.actionContext.oldSelection)
+  // print(context.actionContext.newSelection)
+
   if (newSelection.length == 1 && newSelection[0].name() == "PADDY-NULL-LAYER") {
     // The selection was becuase of the 'Null layer' used to resolve Craft's stupid Duplicator conflict!!!
 
     log('Selected the NULL layer; will reselect', context.actionContext.oldSelection)
-    
+
     context.actionContext.oldSelection.forEach(function(layer) {
       layer.select_byExpandingSelection(true, true)
     })
@@ -576,7 +580,7 @@ function updatePaddingAndSpacingForLayer(layer) {
 
       var updateInstances = true
 
-      if (total >= 5) {
+      if (total >= 10) {
         // if there are more than 10 to update... ask the user if this is what they want to do
 
         var iconImage = NSImage.alloc().initByReferencingFile(plugin.urlForResourceNamed("icon.png").path())

Paddy.sketchplugin/Contents/Sketch/manifest.json

@@ -21,7 +21,7 @@
     {
       "name": "Imply padding for selection",
       "identifier": "autoApplyPadding",
-      "shortcut": "command p",
+      "shortcut": "command shift p",
       "script": "main.js",
       "handler": "autoApplyPadding",
       "description": "Apply implied padding to the currently selected layers",

Paddy.sketchplugin/Contents/Sketch/spacing.js

@@ -234,7 +234,7 @@ function applySpacingToGroup(spacing, groupLayer) {
       return false
     if (layer.isMemberOfClass(MSSliceLayer))
       return false
-    if (layer.name().startsWith('-'))
+    if (shouldLayerBeIgnored(layer))
       return false
     return true
   })

README.md

@@ -84,7 +84,7 @@ Examples:
 
 Run the plugin command **'Enter padding for selection'** or **`Control + Alt + p`** to bring up an input field, to save padding to all your selected layers easier.
 
-If you don't want to be prompted to enter the specified padding, and instead want it to be automatically inferred from what it already is – run the command **'Imply padding for selection'** or **`Command + p`**.
+If you don't want to be prompted to enter the specified padding, and instead want it to be automatically inferred from what it already is – run the command **'Imply padding for selection'** or **`Command + shift + p`**.
 
 ![Padding example](.images/GIFs/padding.gif)