From 2fbd9b95a74bf9dc12ef9c0fe566301246379b9c Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?Florian=20M=C3=BCllner?= Date: Fri, 15 Feb 2013 16:47:39 +0100 Subject: [PATCH] appDisplay: Set a maximum column number In particular on widescreen monitors, the number of items in a row can be overwhelming, so limit the number of columns and center the view. https://bugzilla.gnome.org/show_bug.cgi?id=694192 --- js/ui/appDisplay.js | 7 +++++-- 1 file changed, 5 insertions(+), 2 deletions(-) diff --git a/js/ui/appDisplay.js b/js/ui/appDisplay.js index 456feb563..da876401b 100644 --- a/js/ui/appDisplay.js +++ b/js/ui/appDisplay.js @@ -27,6 +27,7 @@ const Util = imports.misc.util; const MAX_APPLICATION_WORK_MILLIS = 75; const MENU_POPUP_TIMEOUT = 600; const SCROLL_TIME = 0.1; +const MAX_COLUMNS = 6; // Recursively load a GMenuTreeDirectory; we could put this in ShellAppSystem too function _loadCategory(dir, view) { @@ -51,7 +52,8 @@ const AlphabeticalView = new Lang.Class({ Name: 'AlphabeticalView', _init: function() { - this._grid = new IconGrid.IconGrid({ xAlign: St.Align.START }); + this._grid = new IconGrid.IconGrid({ xAlign: St.Align.MIDDLE, + columnLimit: MAX_COLUMNS }); this._appIcons = {}; // desktop file id this._allApps = []; @@ -62,6 +64,7 @@ const AlphabeticalView = new Lang.Class({ this.actor = new St.ScrollView({ x_fill: true, y_fill: false, y_align: St.Align.START, + x_expand: true, style_class: 'vfade' }); this.actor.add_actor(box); this.actor.set_policy(Gtk.PolicyType.NEVER, Gtk.PolicyType.AUTOMATIC); @@ -140,7 +143,7 @@ const AppDisplay = new Lang.Class({ x_fill: true, y_fill: true }); this._view = new AlphabeticalView(); - box.add(this._view.actor); + box.add(this._view.actor, { expand: true }); // We need a dummy actor to catch the keyboard focus if the // user Ctrl-Alt-Tabs here before the deferred work creates