From 5ab593ca1197d5cfd2255ad846db61cf65ad631d Mon Sep 17 00:00:00 2001
From: Damiano Galassi <damiog@gmail.com>
Date: Fri, 21 Oct 2016 14:43:35 +0200
Subject: [PATCH] MacGui: add some accessibility labels to the preview window
 buttons.

---
 macosx/HBPictureHUDController.xib | 11 +++++++----
 macosx/HBPlayerHUDController.xib  | 24 ++++++++++++++++++------
 2 files changed, 25 insertions(+), 10 deletions(-)

diff --git a/macosx/HBPictureHUDController.xib b/macosx/HBPictureHUDController.xib
index 22da3ff17..99690a921 100644
--- a/macosx/HBPictureHUDController.xib
+++ b/macosx/HBPictureHUDController.xib
@@ -1,8 +1,9 @@
-<?xml version="1.0" encoding="UTF-8" standalone="no"?>
-<document type="com.apple.InterfaceBuilder3.Cocoa.XIB" version="3.0" toolsVersion="11129.15" systemVersion="15G26a" targetRuntime="MacOSX.Cocoa" propertyAccessControl="none" useAutolayout="YES">
+<?xml version="1.0" encoding="UTF-8"?>
+<document type="com.apple.InterfaceBuilder3.Cocoa.XIB" version="3.0" toolsVersion="11535.1" systemVersion="16B2553a" targetRuntime="MacOSX.Cocoa" propertyAccessControl="none" useAutolayout="YES">
     <dependencies>
         <deployment identifier="macosx"/>
-        <plugIn identifier="com.apple.InterfaceBuilder.CocoaPlugin" version="11129.15"/>
+        <development version="7000" identifier="xcode"/>
+        <plugIn identifier="com.apple.InterfaceBuilder.CocoaPlugin" version="11535.1"/>
     </dependencies>
     <objects>
         <customObject id="-2" userLabel="File's Owner" customClass="HBPictureHUDController">
@@ -87,15 +88,17 @@
                         </menu>
                     </popUpButtonCell>
                     <connections>
+                        <accessibilityConnection property="title" destination="5T3-Vx-mC1" id="G7o-Sr-rhm"/>
                         <action selector="previewDurationPopUpChanged:" target="-2" id="JSm-Eo-Ijw"/>
                     </connections>
                 </popUpButton>
                 <slider verticalHuggingPriority="750" fixedFrame="YES" translatesAutoresizingMaskIntoConstraints="NO" id="wUk-SQ-GhS">
-                    <rect key="frame" x="18" y="39" width="420" height="17"/>
+                    <rect key="frame" x="18" y="39" width="420" height="15"/>
                     <autoresizingMask key="autoresizingMask" flexibleMaxX="YES" flexibleMinY="YES"/>
                     <sliderCell key="cell" controlSize="mini" continuous="YES" alignment="left" maxValue="9" tickMarkPosition="above" numberOfTickMarks="10" allowsTickMarkValuesOnly="YES" sliderType="linear" id="dPk-jb-w9o">
                         <font key="font" metaFont="miniSystem"/>
                     </sliderCell>
+                    <accessibility description="preview image"/>
                     <connections>
                         <action selector="pictureSliderChanged:" target="-2" id="576-54-fMW"/>
                     </connections>
diff --git a/macosx/HBPlayerHUDController.xib b/macosx/HBPlayerHUDController.xib
index ba6386846..db5cbac7a 100644
--- a/macosx/HBPlayerHUDController.xib
+++ b/macosx/HBPlayerHUDController.xib
@@ -1,8 +1,9 @@
-<?xml version="1.0" encoding="UTF-8" standalone="no"?>
-<document type="com.apple.InterfaceBuilder3.Cocoa.XIB" version="3.0" toolsVersion="10117" systemVersion="15G7a" targetRuntime="MacOSX.Cocoa" propertyAccessControl="none">
+<?xml version="1.0" encoding="UTF-8"?>
+<document type="com.apple.InterfaceBuilder3.Cocoa.XIB" version="3.0" toolsVersion="11535.1" systemVersion="16B2553a" targetRuntime="MacOSX.Cocoa" propertyAccessControl="none">
     <dependencies>
         <deployment identifier="macosx"/>
-        <plugIn identifier="com.apple.InterfaceBuilder.CocoaPlugin" version="10117"/>
+        <development version="7000" identifier="xcode"/>
+        <plugIn identifier="com.apple.InterfaceBuilder.CocoaPlugin" version="11535.1"/>
     </dependencies>
     <objects>
         <customObject id="-2" userLabel="File's Owner" customClass="HBPlayerHUDController">
@@ -26,6 +27,7 @@
                     <rect key="frame" x="69" y="18" width="322" height="17"/>
                     <autoresizingMask key="autoresizingMask" widthSizable="YES" flexibleMaxY="YES"/>
                     <sliderCell key="cell" controlSize="small" continuous="YES" alignment="left" maxValue="100" tickMarkPosition="above" sliderType="linear" id="mMk-re-3u5"/>
+                    <accessibility description="timeline"/>
                     <connections>
                         <action selector="sliderChanged:" target="-2" id="YM4-mh-fr5"/>
                     </connections>
@@ -37,6 +39,7 @@
                         <behavior key="behavior" pushIn="YES" lightByBackground="YES" lightByGray="YES"/>
                         <font key="font" metaFont="system"/>
                     </buttonCell>
+                    <accessibility description="play/pause"/>
                     <connections>
                         <action selector="playPauseToggle:" target="-2" id="LLB-0k-P6K"/>
                     </connections>
@@ -48,6 +51,7 @@
                         <behavior key="behavior" pushIn="YES" lightByBackground="YES" lightByGray="YES"/>
                         <font key="font" metaFont="system"/>
                     </buttonCell>
+                    <accessibility description="go to beginning"/>
                     <connections>
                         <action selector="goToBeginning:" target="-2" id="m24-WZ-vGD"/>
                     </connections>
@@ -59,6 +63,7 @@
                         <behavior key="behavior" pushIn="YES" lightByBackground="YES" lightByGray="YES"/>
                         <font key="font" metaFont="miniSystem"/>
                     </buttonCell>
+                    <accessibility description="go to end"/>
                     <connections>
                         <action selector="goToEnd:" target="-2" id="Rp0-53-4gw"/>
                     </connections>
@@ -71,6 +76,7 @@
                         <color key="textColor" name="secondaryLabelColor" catalog="System" colorSpace="catalog"/>
                         <color key="backgroundColor" name="controlColor" catalog="System" colorSpace="catalog"/>
                     </textFieldCell>
+                    <accessibility description="remaining time"/>
                 </textField>
                 <textField verticalHuggingPriority="750" id="uYx-2C-8et">
                     <rect key="frame" x="10" y="20" width="55" height="14"/>
@@ -80,11 +86,13 @@
                         <color key="textColor" name="secondaryLabelColor" catalog="System" colorSpace="catalog"/>
                         <color key="backgroundColor" name="controlColor" catalog="System" colorSpace="catalog"/>
                     </textFieldCell>
+                    <accessibility description="current time"/>
                 </textField>
                 <slider verticalHuggingPriority="750" id="gzA-no-jdv">
                     <rect key="frame" x="34" y="55" width="65" height="17"/>
                     <autoresizingMask key="autoresizingMask" flexibleMaxX="YES" flexibleMinY="YES"/>
                     <sliderCell key="cell" controlSize="small" continuous="YES" state="on" alignment="left" maxValue="1" doubleValue="1" tickMarkPosition="above" sliderType="linear" id="aQz-j6-bQi"/>
+                    <accessibility description="volume"/>
                     <connections>
                         <action selector="volumeSliderChanged:" target="-2" id="rLr-Ug-i8q"/>
                     </connections>
@@ -96,28 +104,31 @@
                         <behavior key="behavior" pushIn="YES" lightByBackground="YES" lightByGray="YES"/>
                         <font key="font" metaFont="system"/>
                     </buttonCell>
+                    <accessibility description="end live preview"/>
                     <connections>
                         <action selector="showPicturesPreview:" target="-2" id="kCs-UI-Epb"/>
                     </connections>
                 </button>
-                <button toolTip="Go To Beginning" id="euL-Tg-dmT">
+                <button toolTip="Mute" id="euL-Tg-dmT">
                     <rect key="frame" x="12" y="55" width="18" height="18"/>
                     <autoresizingMask key="autoresizingMask" flexibleMinX="YES" flexibleMaxX="YES" flexibleMinY="YES"/>
                     <buttonCell key="cell" type="square" bezelStyle="shadowlessSquare" image="volLowTemplate" imagePosition="overlaps" alignment="center" imageScaling="proportionallyDown" id="njB-22-thn">
                         <behavior key="behavior" pushIn="YES" lightByBackground="YES" lightByGray="YES"/>
                         <font key="font" metaFont="system"/>
                     </buttonCell>
+                    <accessibility description="mute"/>
                     <connections>
                         <action selector="mute:" target="-2" id="CIm-ce-Lnf"/>
                     </connections>
                 </button>
-                <button id="Kiv-cL-wYI">
+                <button toolTip="Full Volume" id="Kiv-cL-wYI">
                     <rect key="frame" x="103" y="55" width="18" height="18"/>
                     <autoresizingMask key="autoresizingMask" flexibleMinX="YES" flexibleMaxX="YES" flexibleMinY="YES"/>
                     <buttonCell key="cell" type="square" bezelStyle="shadowlessSquare" image="volHighTemplate" imagePosition="overlaps" alignment="center" imageScaling="proportionallyDown" id="rgx-mB-Rva">
                         <behavior key="behavior" pushIn="YES" lightByBackground="YES" lightByGray="YES"/>
                         <font key="font" metaFont="system"/>
                     </buttonCell>
+                    <accessibility description="full volume"/>
                     <connections>
                         <action selector="maxVolume:" target="-2" id="89B-BN-U8S"/>
                     </connections>
@@ -136,9 +147,10 @@
                             </items>
                         </menu>
                     </popUpButtonCell>
+                    <accessibility description="tracks selection"/>
                 </popUpButton>
             </subviews>
-            <point key="canvasLocation" x="466" y="429"/>
+            <point key="canvasLocation" x="73" y="25"/>
         </customView>
     </objects>
     <resources>
-- 
2.40.0