Changeset 13426

Timestamp:

Feb 28, 2012, 10:18:03 PM (12 years ago)

Author:

rvelices

Message:

• derivative fix floating point rounding errors for certain sizes
• increase derivative size for which metadata is kept
• imagick use 4:2:2 chroma subsampling (reduce file size by 20-30% with "almost" no human perception)
• dont show mobile theme links if there is no mobile theme installed
• mobile theme favicon removed because 404

Location:

trunk

Files:

• admin/include/image.class.php (modified) (1 diff)
• i.php (modified) (1 diff)
• include/derivative_params.inc.php (modified) (1 diff)
• include/page_tail.php (modified) (1 diff)
• themes/smartpocket/template/header.tpl (modified) (1 diff)

trunk/admin/include/image.class.php

@@ -459 +459 @@
   function write($destination_filepath)
   {
+    // use 4:2:2 chroma subsampling (reduce file size by 20-30% with "almost" no human perception)
+    $this->image->setSamplingFactors( array(2,1) );
     return $this->image->writeImage($destination_filepath);
   }

trunk/i.php

@@ -520 +520 @@
 }
 
-if ($d_size[0]*$d_size[1] < 100000)
+if ($d_size[0]*$d_size[1] < 256000)
 {// strip metadata for small images
   $image->strip();

trunk/include/derivative_params.inc.php

@@ -210 +210 @@
       {
         $scale_size[0] = $this->ideal_size[0];
-        $scale_size[1] = floor($scale_size[1] / $ratio_w);
+        $scale_size[1] = floor(1e-6 + $scale_size[1] / $ratio_w);
       }
       else
       {
-        $scale_size[0] = floor($scale_size[0] / $ratio_h);
+        $scale_size[0] = floor(1e-6 + $scale_size[0] / $ratio_h);
         $scale_size[1] = $this->ideal_size[1];
       }

trunk/include/page_tail.php

@@ -67 +67 @@
 
 //------------------------------------------------------------- mobile version
-if (get_device() != 'desktop')
+if ( !empty($conf['mobile_theme']) && get_device() != 'desktop')
 {
   $template->assign('TOGGLE_MOBILE_THEME_URL',

trunk/themes/smartpocket/template/header.tpl

@@ -23 +23 @@
 <title>{$GALLERY_TITLE} | {$PAGE_TITLE}</title>{else}
 <title>{$PAGE_TITLE} | {$GALLERY_TITLE}</title>{/if}
-<link rel="shortcut icon" type="image/x-icon" href="{$ROOT_URL}{$themeconf.icon_dir}/favicon.ico">
 <link rel="start" title="{'Home'|@translate}" href="{$U_HOME}" >