Minor refactoring and UI tweaks

Gifski/Assets.xcassets/ButtonTextColor.colorset/Contents.json

@@ -0,0 +1,38 @@
+{
+  "info" : {
+    "version" : 1,
+    "author" : "xcode"
+  },
+  "colors" : [
+    {
+      "idiom" : "mac",
+      "color" : {
+        "color-space" : "srgb",
+        "components" : {
+          "red" : "0xFF",
+          "alpha" : "1.000",
+          "blue" : "0xFF",
+          "green" : "0xFF"
+        }
+      }
+    },
+    {
+      "idiom" : "mac",
+      "appearances" : [
+        {
+          "appearance" : "luminosity",
+          "value" : "dark"
+        }
+      ],
+      "color" : {
+        "color-space" : "srgb",
+        "components" : {
+          "red" : "0x00",
+          "alpha" : "1.000",
+          "blue" : "0x00",
+          "green" : "0x00"
+        }
+      }
+    }
+  ]
+}

Gifski/Constants.swift

@@ -1,7 +1,7 @@
 import Cocoa
 
 extension NSColor {
-	static let appTheme = NSColor.controlAccentColorPolyfill
+	static let themeColor = NSColor.controlAccentColorPolyfill
 }
 
 extension Defaults.Keys {

Gifski/ConversionCompletedView.swift

@@ -28,19 +28,24 @@ final class ConversionCompletedView: SSView {
 		$0.spacing = 20
 	}
 
-	private lazy var showInFinderButton = with(CustomButton()) {
-		$0.title = "Show in Finder"
-		$0.textColor = .appTheme
-		$0.backgroundColor = .clear
-		$0.borderWidth = 1
+	private func createButton(title: String) -> CustomButton {
+		return with(CustomButton()) {
+			$0.title = title
+			$0.textColor = .themeColor
+			// TODO: Custombutton should have a better way of handling different color in dark and light mode
+			$0.activeTextColor = NSColor(named: "ButtonTextColor")!
+			$0.backgroundColor = .clear
+			$0.activeBackgroundColor = .themeColor
+			$0.borderColor = .themeColor
+			$0.activeBorderColor = .themeColor
+			$0.borderWidth = 1
+			$0.font = NSFont.systemFont(ofSize: 12, weight: .medium)
+		}
 	}
 
-	private lazy var shareButton = with(CustomButton()) {
-		$0.title = "Share"
-		$0.textColor = .appTheme
-		$0.backgroundColor = .clear
-		$0.borderWidth = 1
-	}
+	private lazy var showInFinderButton = createButton(title: "Show in Finder")
+
+	private lazy var shareButton = createButton(title: "Share")
 
 	var fileUrl: URL! {
 		didSet {

Gifski/MainWindowController.swift

@@ -5,7 +5,7 @@ import Crashlytics
 
 final class MainWindowController: NSWindowController {
 	private lazy var circularProgress = with(CircularProgress(size: 160)) {
-		$0.color = .appTheme
+		$0.color = .themeColor
 		$0.isHidden = true
 		$0.centerInWindow(window)
 	}
@@ -92,7 +92,7 @@ final class MainWindowController: NSWindowController {
 		window.makeKeyAndOrderFront(nil)
 		NSApp.activate(ignoringOtherApps: false)
 
-		DockProgress.style = .circle(radius: 55, color: .appTheme)
+		DockProgress.style = .circle(radius: 55, color: .themeColor)
 	}
 
 	/// Gets called when the Esc key is pressed.

Gifski/VideoDropView.swift

@@ -40,7 +40,7 @@ class DropView: SSView {
 		}
 	}
 
-	override init(frame: NSRect) {
+	override init(frame: CGRect) {
 		super.init(frame: frame)
 		autoresizingMask = [.width, .height]
 		registerForDraggedTypes(acceptedTypes)
@@ -64,7 +64,7 @@ class DropView: SSView {
 		}
 	}
 
-	override func draw(_ dirtyRect: NSRect) {
+	override func draw(_ dirtyRect: CGRect) {
 		super.draw(dirtyRect)
 
 		if isDraggingHighlighted {
@@ -107,7 +107,7 @@ final class VideoDropView: DropView {
 	var onComplete: (([URL]) -> Void)?
 
 	override var highlightColor: NSColor {
-		return .appTheme
+		return .themeColor
 	}
 
 	override var acceptedTypes: [NSPasteboard.PasteboardType] {

Gifski/util.swift

@@ -557,7 +557,7 @@ class Label: NSTextField {
 
 /// Use it in Interface Builder as a class or programmatically
 final class MonospacedLabel: Label {
-	override init(frame: NSRect) {
+	override init(frame: CGRect) {
 		super.init(frame: frame)
 		setup()
 	}