[gnome-shell] [Overview] Don't treat 1024x600 as widescreen



commit 4bab511fa5f00d9172b4f3db4f9113a3a2036379
Author: Colin Walters <walters verbum org>
Date:   Fri Sep 25 16:59:38 2009 -0400

    [Overview] Don't treat 1024x600 as widescreen
    
    This works significantly better on my netbook.
    
    https://bugzilla.gnome.org/show_bug.cgi?id=596375

 js/ui/overview.js |    7 ++++++-
 1 files changed, 6 insertions(+), 1 deletions(-)
---
diff --git a/js/ui/overview.js b/js/ui/overview.js
index 0600414..f3f402d 100644
--- a/js/ui/overview.js
+++ b/js/ui/overview.js
@@ -39,6 +39,10 @@ const ANIMATION_TIME = 0.25;
 // will take up the remaining sections of the display.
 
 const WIDE_SCREEN_CUT_OFF_RATIO = 1.4;
+// A common netbook resolution is 1024x600, which trips the widescreen
+// ratio.  However that leaves way too few pixels for the dash.  So
+// just treat this as a regular screen.
+const WIDE_SCREEN_MINIMUM_HEIGHT = 768;
 
 const COLUMNS_REGULAR_SCREEN = 4;
 const ROWS_REGULAR_SCREEN = 8;
@@ -138,7 +142,8 @@ Overview.prototype = {
     },
 
     _recalculateGridSizes: function () {
-        wideScreen = (global.screen_width/global.screen_height > WIDE_SCREEN_CUT_OFF_RATIO);
+        wideScreen = (global.screen_width/global.screen_height > WIDE_SCREEN_CUT_OFF_RATIO) &&
+                     (global.screen_height >= WIDE_SCREEN_MINIMUM_HEIGHT);
 
         // We divide the screen into an imaginary grid which helps us determine the layout of
         // different visual components.



[Date Prev][Date Next]   [Thread Prev][Thread Next]   [Thread Index] [Date Index] [Author Index]