[vlc-commits] macosx: Remove wrong "Change" button behavior in simple shortcut prefs

[Marvin Scholz]

vlc | branch: master | Marvin Scholz <epirat07 at gmail.com> | Wed Oct  7 20:18:29 2015 +0200| [e77a32e8d9ae067b53faa1a27558e8ba7b3a8ce4] | committer: Felix Paul Kühne

macosx: Remove wrong "Change" button behavior in simple shortcut prefs

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 at videolan.org>

> http://git.videolan.org/gitweb.cgi/vlc.git/?a=commit;h=e77a32e8d9ae067b53faa1a27558e8ba7b3a8ce4
---

 .../package/macosx/Resources/English.lproj/SimplePreferences.xib  |    5 +----
 1 file changed, 1 insertion(+), 4 deletions(-)

diff --git a/extras/package/macosx/Resources/English.lproj/SimplePreferences.xib b/extras/package/macosx/Resources/English.lproj/SimplePreferences.xib
index 8ce38d8..fda675e 100644
--- a/extras/package/macosx/Resources/English.lproj/SimplePreferences.xib
+++ b/extras/package/macosx/Resources/English.lproj/SimplePreferences.xib
@@ -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"/>