From 88c3945ae1f4988be8dabfd3b3c4f5d99075a82b Mon Sep 17 00:00:00 2001 From: Georges Basile Stavracas Neto Date: Tue, 9 Aug 2022 14:26:24 -0300 Subject: [PATCH] appDisplay: Center-align arrows As per feedback, center align the navigation arrows since they are big enough of click targets. Part-of: --- js/ui/appDisplay.js | 12 ++++++++---- 1 file changed, 8 insertions(+), 4 deletions(-) diff --git a/js/ui/appDisplay.js b/js/ui/appDisplay.js index 1b21f14dc..b4c00f824 100644 --- a/js/ui/appDisplay.js +++ b/js/ui/appDisplay.js @@ -426,9 +426,11 @@ class BaseAppViewGridLayout extends Clutter.BinLayout { rightBox.x1 = rightBox.x2 - indicatorsWidth; this._previousPageIndicator.allocate(ltr ? leftBox : rightBox); - this._previousPageArrow.allocate(ltr ? leftBox : rightBox); + this._previousPageArrow.allocate_align_fill(ltr ? leftBox : rightBox, + 0.5, 0.5, false, false); this._nextPageIndicator.allocate(ltr ? rightBox : leftBox); - this._nextPageArrow.allocate(ltr ? rightBox : leftBox); + this._nextPageArrow.allocate_align_fill(ltr ? rightBox : leftBox, + 0.5, 0.5, false, false); this._pageWidth = box.get_width(); } @@ -565,7 +567,8 @@ var BaseAppView = GObject.registerClass({ icon_name: rtl ? 'carousel-arrow-previous-symbolic' : 'carousel-arrow-next-symbolic', - x_expand: true, + x_align: Clutter.ActorAlign.CENTER, + y_align: Clutter.ActorAlign.CENTER, }); this._nextPageArrow.connect('clicked', () => this.goToPage(this._grid.currentPage + 1)); @@ -577,7 +580,8 @@ var BaseAppView = GObject.registerClass({ : 'carousel-arrow-previous-symbolic', opacity: 0, visible: false, - x_expand: true, + x_align: Clutter.ActorAlign.CENTER, + y_align: Clutter.ActorAlign.CENTER, }); this._prevPageArrow.connect('clicked', () => this.goToPage(this._grid.currentPage - 1));