macosx: Remove wrong "Change" button behavior in simple shortcut prefs
authorMarvin Scholz <epirat07@gmail.com>
Wed, 7 Oct 2015 18:18:29 +0000 (20:18 +0200)
committerFelix Paul Kühne <fkuehne@videolan.org>
Fri, 9 Oct 2015 10:26:55 +0000 (12:26 +0200)
Remove the primary state of the "Change" button in the simple preferences
shotcut view, as the window already has a primary button, which is the one
to apply the settings. Having two buttons as primary buttons is bad UX
and very confusing as it's unclear which button will be triggered when
hitting the enter key.
Given that the "Change" button was not triggered by the enter key anyway,
this should not cause any app behavior changes.

Signed-off-by: Felix Paul Kühne <fkuehne@videolan.org>
extras/package/macosx/Resources/English.lproj/SimplePreferences.xib

index 8ce38d8..fda675e 100644 (file)
@@ -1774,7 +1774,7 @@ Gw
                         <autoresizingMask key="autoresizingMask" widthSizable="YES" heightSizable="YES"/>
                         <subviews>
                             <tableView verticalHuggingPriority="750" allowsExpansionToolTips="YES" alternatingRowBackgroundColors="YES" columnReordering="NO" multipleSelection="NO" emptySelection="NO" autosaveName="sprefs_hotkeys_view" headerView="3541" id="2670">
-                                <rect key="frame" x="0.0" y="0.0" width="528" height="19"/>
+                                <rect key="frame" x="0.0" y="0.0" width="528" height="286"/>
                                 <autoresizingMask key="autoresizingMask"/>
                                 <animations/>
                                 <size key="intercellSpacing" width="3" height="2"/>
@@ -1841,9 +1841,6 @@ Gw
                     <buttonCell key="cell" type="push" title="Change" bezelStyle="rounded" alignment="center" borderStyle="border" inset="2" id="3526">
                         <behavior key="behavior" pushIn="YES" lightByBackground="YES" lightByGray="YES"/>
                         <font key="font" metaFont="system"/>
-                        <string key="keyEquivalent" base64-UTF8="YES">
-DQ
-</string>
                     </buttonCell>
                     <connections>
                         <action selector="hotkeySettingChanged:" target="-2" id="ryh-Ni-dNC"/>