Brooklyn 2.0.1 (#31)

* Fixed Done button on High Sierra
* Fixed Window not becoming key on High Sierra

Author: pedrommcarrasco

Brooklyn/Contexts/Preferences/PreferencesWindowController.swift

@@ -31,16 +31,14 @@ final class PreferencesWindowController: NSWindowController, NSWindowDelegate {
 
 // MARK: - Lifecycle
 extension PreferencesWindowController {
-   
+
     override func windowDidLoad() {
         super.windowDidLoad()
         configure()
     }
 
     override func keyUp(with event: NSEvent) {
-        super.keyUp(with: event)
-        
-        if event.keyCode == Constant.spacebarKeyCode {
+        if event.keyCode == Constant.spacebarKeyCode, !event.isARepeat {
             guard !animationsTableView.selectedRowIndexes.isEmpty else { return }
             animationsTableView.selectedRowIndexes.forEach {
                 guard $0 >= 0 else { return }
@@ -136,6 +134,6 @@ private extension PreferencesWindowController {
     @IBAction func doneAction(_ sender: NSButton) {
         animationsTableView.deselectAll(nil)
         guard let window = window else { return }
-        window.endSheet(window)
+        window.sheetParent?.endSheet(window)
     }
 }

Brooklyn/Contexts/Preferences/PreferencesWindowController.xib

@@ -20,9 +20,10 @@
         <customObject id="-1" userLabel="First Responder" customClass="FirstResponder"/>
         <customObject id="-3" userLabel="Application" customClass="NSObject"/>
         <window title="Preferences" allowsToolTipsWhenApplicationIsInactive="NO" autorecalculatesKeyViewLoop="NO" restorable="NO" releasedWhenClosed="NO" visibleAtLaunch="NO" animationBehavior="default" id="QvC-M9-y7g">
+            <windowStyleMask key="styleMask" titled="YES"/>
             <windowPositionMask key="initialPositionMask" leftStrut="YES" rightStrut="YES" topStrut="YES" bottomStrut="YES"/>
             <rect key="contentRect" x="196" y="240" width="800" height="512"/>
-            <rect key="screenRect" x="0.0" y="0.0" width="1680" height="1050"/>
+            <rect key="screenRect" x="0.0" y="0.0" width="1920" height="1200"/>
             <view key="contentView" wantsLayer="YES" id="EiT-Mj-1SZ">
                 <rect key="frame" x="0.0" y="0.0" width="800" height="512"/>
                 <autoresizingMask key="autoresizingMask" widthSizable="YES" heightSizable="YES"/>
@@ -59,7 +60,7 @@
                                                     <subviews>
                                                         <button verticalHuggingPriority="750" translatesAutoresizingMaskIntoConstraints="NO" id="YlX-oQ-WQ2">
                                                             <rect key="frame" x="365" y="-2" width="18" height="18"/>
-                                                            <buttonCell key="cell" type="check" bezelStyle="regularSquare" imagePosition="only" inset="2" id="lWN-QX-1EU">
+                                                            <buttonCell key="cell" type="check" bezelStyle="regularSquare" imagePosition="only" refusesFirstResponder="YES" inset="2" id="lWN-QX-1EU">
                                                                 <behavior key="behavior" changeContents="YES" doesNotDimImage="YES" lightByContents="YES"/>
                                                                 <font key="font" metaFont="system"/>
                                                             </buttonCell>
@@ -163,8 +164,8 @@ DQ
                         </connections>
                     </button>
                     <textField horizontalHuggingPriority="251" verticalHuggingPriority="750" translatesAutoresizingMaskIntoConstraints="NO" id="U06-cj-0bE">
-                        <rect key="frame" x="432" y="186" width="93" height="17"/>
-                        <textFieldCell key="cell" lineBreakMode="clipping" title="Random Order" id="i0q-Ty-Q3X">
+                        <rect key="frame" x="432" y="186" width="263" height="17"/>
+                        <textFieldCell key="cell" lineBreakMode="clipping" title="Play Selected Animations in Random Order" id="i0q-Ty-Q3X">
                             <font key="font" metaFont="system"/>
                             <color key="textColor" name="secondaryLabelColor" catalog="System" colorSpace="catalog"/>
                             <color key="backgroundColor" name="textBackgroundColor" catalog="System" colorSpace="catalog"/>
@@ -310,15 +311,6 @@ DQ
             </connections>
             <point key="canvasLocation" x="251" y="248"/>
         </window>
-        <textField horizontalHuggingPriority="251" verticalHuggingPriority="750" id="piF-LM-Z4F">
-            <rect key="frame" x="0.0" y="0.0" width="37" height="17"/>
-            <autoresizingMask key="autoresizingMask"/>
-            <textFieldCell key="cell" lineBreakMode="clipping" title="Label" id="eWd-PK-ZV9">
-                <font key="font" usesAppearanceFont="YES"/>
-                <color key="textColor" name="labelColor" catalog="System" colorSpace="catalog"/>
-                <color key="backgroundColor" name="textBackgroundColor" catalog="System" colorSpace="catalog"/>
-            </textFieldCell>
-        </textField>
     </objects>
     <resources>
         <image name="github" width="686.4000244140625" height="409.60000610351562"/>

Brooklyn/Info.plist

@@ -15,9 +15,9 @@
 	<key>CFBundlePackageType</key>
 	<string>BNDL</string>
 	<key>CFBundleShortVersionString</key>
-	<string>2.0.0</string>
+	<string>2.0.1</string>
 	<key>CFBundleVersion</key>
-	<string>2</string>
+	<string>3</string>
 	<key>NSPrincipalClass</key>
 	<string>Brooklyn.BrooklynView</string>
 </dict>