ios/Mobile/Base.lproj/Main.storyboard          |   53 ++++++++++++++++---------
 ios/Mobile/TemplateCollectionViewController.mm |   12 +++--
 2 files changed, 42 insertions(+), 23 deletions(-)

New commits:
commit da45975a5aea927bb8bdcaf4a58b0deced862b5c
Author:     Tor Lillqvist <t...@collabora.com>
AuthorDate: Thu Dec 6 17:14:47 2018 +0200
Commit:     Tor Lillqvist <t...@collabora.com>
CommitDate: Fri Dec 7 00:09:13 2018 +0200

    Make the template collection view look much better
    
    Now it finally looks like I want, but oh boy was that a pain. I am not
    sure at all I understand what I am doing in Xcode's Interface Builder.
    
    I tried hard at first to use the cell size 200x220 for the cell size
    of the UICollectionView, consisting of a 200x200 UIImageView and a
    200x20 UILabel below. But that did not seem to work, it still used a
    (default?) size of 150x150. Weird. Anyway, let's commit this state now
    that seems to work.
    
    Change-Id: I4021133619fbf62cd633392d93f19c2bbc81311a

diff --git a/ios/Mobile/Base.lproj/Main.storyboard 
b/ios/Mobile/Base.lproj/Main.storyboard
index 64685855d..062c453af 100644
--- a/ios/Mobile/Base.lproj/Main.storyboard
+++ b/ios/Mobile/Base.lproj/Main.storyboard
@@ -1,6 +1,6 @@
 <?xml version="1.0" encoding="UTF-8"?>
 <document type="com.apple.InterfaceBuilder3.CocoaTouch.Storyboard.XIB" 
version="3.0" toolsVersion="14460.31" targetRuntime="iOS.CocoaTouch" 
propertyAccessControl="none" useAutolayout="YES" useTraitCollections="YES" 
useSafeAreas="YES" colorMatched="YES" initialViewController="BYZ-38-t0r">
-    <device id="retina4_7" orientation="portrait">
+    <device id="ipad9_7" orientation="portrait">
         <adaptation id="fullscreen"/>
     </device>
     <dependencies>
@@ -15,7 +15,7 @@
             <objects>
                 <viewController id="BYZ-38-t0r" 
customClass="DocumentBrowserViewController" sceneMemberID="viewController">
                     <view key="view" contentMode="scaleToFill" id="8bC-Xf-vdC">
-                        <rect key="frame" x="0.0" y="0.0" width="375" 
height="667"/>
+                        <rect key="frame" x="0.0" y="0.0" width="768" 
height="1024"/>
                         <autoresizingMask key="autoresizingMask" 
widthSizable="YES" heightSizable="YES"/>
                         <color key="backgroundColor" red="1" green="1" 
blue="1" alpha="1" colorSpace="custom" customColorSpace="sRGB"/>
                         <viewLayoutGuide key="safeArea" id="0fo-ii-Aj5"/>
@@ -30,7 +30,7 @@
             <objects>
                 <viewController storyboardIdentifier="DocumentViewController" 
id="M2k-L8-Olo" customClass="DocumentViewController" 
sceneMemberID="viewController">
                     <view key="view" contentMode="scaleToFill" id="SIE-Uh-Zny">
-                        <rect key="frame" x="0.0" y="0.0" width="375" 
height="667"/>
+                        <rect key="frame" x="0.0" y="0.0" width="768" 
height="1024"/>
                         <autoresizingMask key="autoresizingMask" 
widthSizable="YES" heightSizable="YES"/>
                         <color key="backgroundColor" white="1" alpha="1" 
colorSpace="calibratedWhite"/>
                         <viewLayoutGuide key="safeArea" id="BoZ-jc-2Y8"/>
@@ -43,49 +43,66 @@
         <!--Template Collection View Controller-->
         <scene sceneID="juG-xb-Xrw">
             <objects>
-                <collectionViewController 
storyboardIdentifier="TemplateCollectionViewController" id="Cwb-6T-NJd" 
customClass="TemplateCollectionViewController" sceneMemberID="viewController">
-                    <collectionView key="view" clipsSubviews="YES" 
contentMode="scaleToFill" dataMode="prototypes" id="cHi-RU-96O">
-                        <rect key="frame" x="0.0" y="0.0" width="375" 
height="667"/>
+                <collectionViewController 
storyboardIdentifier="TemplateCollectionViewController" 
autoresizesArchivedViewToFullSize="NO" id="Cwb-6T-NJd" 
customClass="TemplateCollectionViewController" sceneMemberID="viewController">
+                    <collectionView key="view" clipsSubviews="YES" 
contentMode="scaleAspectFit" dataMode="prototypes" id="cHi-RU-96O">
+                        <rect key="frame" x="0.0" y="0.0" width="768" 
height="1024"/>
                         <autoresizingMask key="autoresizingMask" 
widthSizable="YES" heightSizable="YES"/>
                         <color key="backgroundColor" 
white="0.33333333333333331" alpha="1" colorSpace="custom" 
customColorSpace="genericGamma22GrayColorSpace"/>
                         <collectionViewFlowLayout key="collectionViewLayout" 
minimumLineSpacing="10" minimumInteritemSpacing="10" id="w0h-io-kZq">
-                            <size key="itemSize" width="50" height="50"/>
+                            <size key="itemSize" width="150" height="150"/>
                             <size key="headerReferenceSize" width="50" 
height="50"/>
                             <size key="footerReferenceSize" width="0.0" 
height="0.0"/>
                             <inset key="sectionInset" minX="50" minY="0.0" 
maxX="0.0" maxY="0.0"/>
                         </collectionViewFlowLayout>
                         <cells>
-                            <collectionViewCell opaque="NO" 
clipsSubviews="YES" multipleTouchEnabled="YES" contentMode="center" 
reuseIdentifier="Cell" id="ppW-Sx-T3H">
+                            <collectionViewCell opaque="NO" 
clipsSubviews="YES" multipleTouchEnabled="YES" contentMode="center" 
insetsLayoutMarginsFromSafeArea="NO" reuseIdentifier="Cell" id="ppW-Sx-T3H">
                                 <rect key="frame" x="50" y="50" width="150" 
height="150"/>
                                 <autoresizingMask key="autoresizingMask"/>
                                 <view key="contentView" opaque="NO" 
clipsSubviews="YES" multipleTouchEnabled="YES" contentMode="center" 
insetsLayoutMarginsFromSafeArea="NO">
                                     <rect key="frame" x="0.0" y="0.0" 
width="150" height="150"/>
                                     <autoresizingMask key="autoresizingMask"/>
                                     <subviews>
-                                        <label opaque="NO" 
userInteractionEnabled="NO" tag="2" contentMode="left" 
horizontalHuggingPriority="251" verticalHuggingPriority="251" ambiguous="YES" 
text="Label" textAlignment="center" lineBreakMode="tailTruncation" 
baselineAdjustment="alignBaselines" adjustsFontSizeToFit="NO" 
translatesAutoresizingMaskIntoConstraints="NO" id="fjg-QH-lLg">
-                                            <rect key="frame" x="0.0" 
y="129.5" width="150" height="20.5"/>
+                                        <view opaque="NO" contentMode="center" 
insetsLayoutMarginsFromSafeArea="NO" 
translatesAutoresizingMaskIntoConstraints="NO" id="riJ-1i-vBs">
+                                            <rect key="frame" x="0.0" y="0.0" 
width="150" height="130"/>
+                                            <subviews>
+                                                <imageView 
verifyAmbiguity="off" opaque="NO" userInteractionEnabled="NO" tag="1" 
contentMode="scaleAspectFit" horizontalHuggingPriority="251" 
verticalHuggingPriority="251" insetsLayoutMarginsFromSafeArea="NO" 
translatesAutoresizingMaskIntoConstraints="NO" id="79y-w8-uiT">
+                                                    <rect key="frame" x="0.0" 
y="0.0" width="150" height="130"/>
+                                                    <color 
key="backgroundColor" white="0.0" alpha="0.0" colorSpace="custom" 
customColorSpace="genericGamma22GrayColorSpace"/>
+                                                </imageView>
+                                            </subviews>
+                                            <color key="backgroundColor" 
white="0.0" alpha="0.0" colorSpace="custom" 
customColorSpace="genericGamma22GrayColorSpace"/>
+                                            <constraints>
+                                                <constraint 
firstAttribute="bottom" secondItem="79y-w8-uiT" secondAttribute="bottom" 
id="0I0-7w-Dde"/>
+                                                <constraint 
firstAttribute="trailing" secondItem="79y-w8-uiT" secondAttribute="trailing" 
id="UjO-rr-ITf"/>
+                                                <constraint 
firstItem="79y-w8-uiT" firstAttribute="leading" secondItem="riJ-1i-vBs" 
secondAttribute="leading" id="bBb-Cf-c4a"/>
+                                                <constraint 
firstItem="79y-w8-uiT" firstAttribute="top" secondItem="riJ-1i-vBs" 
secondAttribute="top" id="giz-fe-CV5"/>
+                                            </constraints>
+                                        </view>
+                                        <label verifyAmbiguity="off" 
opaque="NO" userInteractionEnabled="NO" tag="2" contentMode="left" 
horizontalHuggingPriority="251" verticalHuggingPriority="251" 
insetsLayoutMarginsFromSafeArea="NO" text="Label" textAlignment="center" 
lineBreakMode="tailTruncation" baselineAdjustment="alignBaselines" 
adjustsFontSizeToFit="NO" translatesAutoresizingMaskIntoConstraints="NO" 
id="fjg-QH-lLg">
+                                            <rect key="frame" x="0.0" y="130" 
width="150" height="20"/>
                                             <fontDescription 
key="fontDescription" type="system" pointSize="17"/>
                                             <color key="textColor" white="1" 
alpha="1" colorSpace="custom" customColorSpace="genericGamma22GrayColorSpace"/>
                                             <nil key="highlightedColor"/>
                                         </label>
-                                        <imageView userInteractionEnabled="NO" 
tag="1" contentMode="scaleAspectFit" horizontalHuggingPriority="251" 
verticalHuggingPriority="251" ambiguous="YES" 
translatesAutoresizingMaskIntoConstraints="NO" id="79y-w8-uiT">
-                                            <rect key="frame" x="0.0" y="0.0" 
width="150" height="121.5"/>
-                                        </imageView>
                                     </subviews>
                                 </view>
                                 <constraints>
-                                    <constraint firstAttribute="bottom" 
secondItem="fjg-QH-lLg" secondAttribute="bottom" id="AT2-XZ-lxx"/>
-                                    <constraint firstItem="79y-w8-uiT" 
firstAttribute="top" secondItem="ppW-Sx-T3H" secondAttribute="top" 
id="Qao-Rw-z5w"/>
+                                    <constraint firstItem="fjg-QH-lLg" 
firstAttribute="trailing" secondItem="riJ-1i-vBs" secondAttribute="trailing" 
id="5X8-PS-XYq"/>
+                                    <constraint firstItem="fjg-QH-lLg" 
firstAttribute="leading" secondItem="riJ-1i-vBs" secondAttribute="leading" 
id="8fo-lW-iy0"/>
+                                    <constraint firstAttribute="trailing" 
secondItem="riJ-1i-vBs" secondAttribute="trailing" id="9OL-6w-5ui"/>
+                                    <constraint firstAttribute="bottom" 
secondItem="riJ-1i-vBs" secondAttribute="bottom" constant="20" id="FmN-mf-0fQ"/>
+                                    <constraint firstItem="riJ-1i-vBs" 
firstAttribute="leading" secondItem="ppW-Sx-T3H" secondAttribute="leading" 
id="fc4-fE-xoO"/>
+                                    <constraint firstItem="fjg-QH-lLg" 
firstAttribute="top" secondItem="riJ-1i-vBs" secondAttribute="bottom" 
id="ise-Cp-lOO"/>
+                                    <constraint firstItem="riJ-1i-vBs" 
firstAttribute="top" secondItem="ppW-Sx-T3H" secondAttribute="top" 
id="udK-op-sng"/>
                                 </constraints>
-                                <size key="customSize" width="150" 
height="150"/>
                             </collectionViewCell>
                         </cells>
                         <collectionReusableView key="sectionHeaderView" 
opaque="NO" clipsSubviews="YES" multipleTouchEnabled="YES" contentMode="center" 
insetsLayoutMarginsFromSafeArea="NO" reuseIdentifier="SectionHeaderView" 
id="Vbs-oR-qri" customClass="TemplateSectionHeaderView">
-                            <rect key="frame" x="0.0" y="0.0" width="375" 
height="50"/>
+                            <rect key="frame" x="0.0" y="0.0" width="768" 
height="50"/>
                             <autoresizingMask key="autoresizingMask"/>
                             <subviews>
                                 <label opaque="NO" userInteractionEnabled="NO" 
contentMode="left" horizontalHuggingPriority="251" 
verticalHuggingPriority="251" fixedFrame="YES" text="Label" 
lineBreakMode="tailTruncation" baselineAdjustment="alignBaselines" 
adjustsFontSizeToFit="NO" translatesAutoresizingMaskIntoConstraints="NO" 
id="DE4-jV-CVW">
-                                    <rect key="frame" x="8" y="0.0" 
width="359" height="50"/>
+                                    <rect key="frame" x="0.0" y="0.0" 
width="768" height="50"/>
                                     <autoresizingMask key="autoresizingMask" 
flexibleMinY="YES" flexibleMaxY="YES"/>
                                     <fontDescription key="fontDescription" 
type="system" pointSize="17"/>
                                     <color key="textColor" white="1" alpha="1" 
colorSpace="custom" customColorSpace="genericGamma22GrayColorSpace"/>
diff --git a/ios/Mobile/TemplateCollectionViewController.mm 
b/ios/Mobile/TemplateCollectionViewController.mm
index 91d5689d0..94a922825 100644
--- a/ios/Mobile/TemplateCollectionViewController.mm
+++ b/ios/Mobile/TemplateCollectionViewController.mm
@@ -12,8 +12,7 @@
 #import "TemplateCollectionViewController.h"
 #import "TemplateSectionHeaderView.h"
 
-static NSString *mapTemplateExtensionToActual(NSString *templateName)
-{
+static NSString *mapTemplateExtensionToActual(NSString *templateName) {
     NSString *baseName = [templateName stringByDeletingPathExtension];
     NSString *extension = [templateName substringFromIndex:baseName.length];
 
@@ -29,7 +28,7 @@ static NSString *mapTemplateExtensionToActual(NSString 
*templateName)
 
 @implementation TemplateCollectionViewController
 
--(void)viewDidLoad {
+- (void)viewDidLoad {
 
     // Here we should scan for available templates.
 
@@ -59,10 +58,13 @@ static NSString *mapTemplateExtensionToActual(NSString 
*templateName)
     UILabel *title = (UILabel *)[cell viewWithTag:2];
 
     NSString *templateThumbnail = [[templates[[indexPath 
indexAtPosition:0]][[indexPath indexAtPosition:1]] path] 
stringByAppendingString:@".png"];
+    UIImage *thumbnail;
     if ([NSFileManager.defaultManager fileExistsAtPath:templateThumbnail])
-          image.image = [UIImage imageWithContentsOfFile:templateThumbnail];
+        thumbnail = [UIImage imageWithContentsOfFile:templateThumbnail];
     else
-          image.image = [UIImage imageNamed:@"AppIcon"];
+        thumbnail = [UIImage imageNamed:@"AppIcon"];
+
+    image.image = thumbnail;
 
     NSString *fileName = [templates[[indexPath indexAtPosition:0]][[indexPath 
indexAtPosition:1]] lastPathComponent];
 
_______________________________________________
Libreoffice-commits mailing list
libreoffice-comm...@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/libreoffice-commits

Reply via email to