Revision: 10221
          http://sourceforge.net/p/skim-app/code/10221
Author:   hofman
Date:     2019-01-17 20:08:23 +0000 (Thu, 17 Jan 2019)
Log Message:
-----------
don't add 1 to auto sized column width, make sure to use ceil

Modified Paths:
--------------
    trunk/SKMainWindowController.m
    trunk/SKPresentationOptionsSheetController.m

Modified: trunk/SKMainWindowController.m
===================================================================
--- trunk/SKMainWindowController.m      2019-01-17 19:38:29 UTC (rev 10220)
+++ trunk/SKMainWindowController.m      2019-01-17 20:08:23 UTC (rev 10221)
@@ -693,10 +693,10 @@
     
     for (NSString *label in pageLabels) {
         [cell setStringValue:label];
-        labelWidth = fmax(labelWidth, [cell cellSize].width + 1.0);
+        labelWidth = fmax(labelWidth, [cell cellSize].width + 0.0);
     }
     
-    labelWidth = fmin(labelWidth, MAX_PAGE_COLUMN_WIDTH);
+    labelWidth = fmin(ceil(labelWidth), MAX_PAGE_COLUMN_WIDTH);
     [tableColumn setMinWidth:labelWidth];
     [tableColumn setMaxWidth:labelWidth];
     [tableColumn setWidth:labelWidth];

Modified: trunk/SKPresentationOptionsSheetController.m
===================================================================
--- trunk/SKPresentationOptionsSheetController.m        2019-01-17 19:38:29 UTC 
(rev 10220)
+++ trunk/SKPresentationOptionsSheetController.m        2019-01-17 20:08:23 UTC 
(rev 10221)
@@ -220,13 +220,13 @@
             [info setProperties:([ptEnum nextObject] ?: dictionary)];
             [array addObject:info];
             [cell setStringValue:[info label]];
-            labelWidth = fmax(labelWidth, ceil([cell cellSize].width + 1.0));
+            labelWidth = fmax(labelWidth, [cell cellSize].width);
             [info release];
         }
         tn = next;
     }
     
-    labelWidth = fmin(labelWidth, MAX_PAGE_COLUMN_WIDTH);
+    labelWidth = fmin(ceil(labelWidth), MAX_PAGE_COLUMN_WIDTH);
     [tableColumn setMinWidth:labelWidth];
     [tableColumn setMaxWidth:labelWidth];
     [tableColumn setWidth:labelWidth];

This was sent by the SourceForge.net collaborative development platform, the 
world's largest Open Source development site.



_______________________________________________
Skim-app-commit mailing list
[email protected]
https://lists.sourceforge.net/lists/listinfo/skim-app-commit

Reply via email to