[gnome-shell/wip/fmuellner/overview-tweaks: 6/9] workspace: Don't restrict title width to window preview
- From: Florian Müllner <fmuellner src gnome org>
- To: commits-list gnome org
- Cc:
- Subject: [gnome-shell/wip/fmuellner/overview-tweaks: 6/9] workspace: Don't restrict title width to window preview
- Date: Wed, 19 Jul 2017 12:09:13 +0000 (UTC)
commit 520628b1e7fea825daed3c7bd7c9a9a2ded3e788
Author: Florian Müllner <fmuellner gnome org>
Date: Thu Jun 1 16:36:10 2017 +0200
workspace: Don't restrict title width to window preview
Now that only one window title is visible at any time, it no longer
matters if a title extends into other window previews, so we can
always show the full title.
https://bugzilla.gnome.org/show_bug.cgi?id=783953
js/ui/workspace.js | 21 +++++----------------
1 files changed, 5 insertions(+), 16 deletions(-)
---
diff --git a/js/ui/workspace.js b/js/ui/workspace.js
index d2a34a2..3a406cc 100644
--- a/js/ui/workspace.js
+++ b/js/ui/workspace.js
@@ -538,24 +538,13 @@ var WindowOverlay = new Lang.Class({
else
button.set_position(Math.floor(buttonX), Math.floor(buttonY));
- // Clutter.Actor.get_preferred_width() will return the fixed width if one
- // is set, so we need to reset the width by calling set_width(-1), to forward
- // the call down to StLabel.
- // We also need to save and restore the current width, otherwise the animation
- // starts from the wrong point.
- let prevTitleWidth = title.width;
- title.set_width(-1);
- let [titleMinWidth, titleNatWidth] = title.get_preferred_width(-1);
- let titleWidth = Math.max(titleMinWidth, Math.min(titleNatWidth, cloneWidth));
- title.width = prevTitleWidth;
-
- let titleX = cloneX + (cloneWidth - titleWidth) / 2;
+ let titleX = cloneX + (cloneWidth - title.width) / 2;
let titleY = cloneY + cloneHeight - (title.height - this.borderSize) / 2;
- if (animate)
- this._animateOverlayActor(title, Math.floor(titleX), Math.floor(titleY), titleWidth);
- else {
- title.width = titleWidth;
+ if (animate) {
+ this._animateOverlayActor(title, Math.floor(titleX), Math.floor(titleY), title.width);
+ } else {
+ title.width = title.width;
title.set_position(Math.floor(titleX), Math.floor(titleY));
}
[
Date Prev][
Date Next] [
Thread Prev][
Thread Next]
[
Thread Index]
[
Date Index]
[
Author Index]