[gnome-shell] magnifier: Small cleanup
- From: Marge Bot <marge-bot src gnome org>
- To: commits-list gnome org
- Cc:
- Subject: [gnome-shell] magnifier: Small cleanup
- Date: Sat, 31 Jul 2021 17:22:04 +0000 (UTC)
commit 8f42053b5fde050dc98e459009eb476f86a86380
Author: Florian Müllner <fmuellner gnome org>
Date: Thu Jul 29 22:17:54 2021 +0200
magnifier: Small cleanup
Use an early return instead of a big if() block to cut down on
indentation.
Part-of: <https://gitlab.gnome.org/GNOME/gnome-shell/-/merge_requests/1932>
js/ui/magnifier.js | 29 +++++++++++++++--------------
1 file changed, 15 insertions(+), 14 deletions(-)
---
diff --git a/js/ui/magnifier.js b/js/ui/magnifier.js
index 57b32cb057..22581278df 100644
--- a/js/ui/magnifier.js
+++ b/js/ui/magnifier.js
@@ -232,20 +232,21 @@ var Magnifier = class Magnifier {
scrollToMousePos(...args) {
const [xMouse, yMouse] = args.length ? args : global.get_pointer();
- if (xMouse != this.xMouse || yMouse != this.yMouse) {
- this.xMouse = xMouse;
- this.yMouse = yMouse;
-
- let sysMouseOverAny = false;
- this._zoomRegions.forEach(zoomRegion => {
- if (zoomRegion.scrollToMousePos())
- sysMouseOverAny = true;
- });
- if (sysMouseOverAny)
- this.hideSystemCursor();
- else
- this.showSystemCursor();
- }
+ if (xMouse === this.xMouse && yMouse === this.yMouse)
+ return;
+
+ this.xMouse = xMouse;
+ this.yMouse = yMouse;
+
+ let sysMouseOverAny = false;
+ this._zoomRegions.forEach(zoomRegion => {
+ if (zoomRegion.scrollToMousePos())
+ sysMouseOverAny = true;
+ });
+ if (sysMouseOverAny)
+ this.hideSystemCursor();
+ else
+ this.showSystemCursor();
}
/**
[
Date Prev][
Date Next] [
Thread Prev][
Thread Next]
[
Thread Index]
[
Date Index]
[
Author Index]