[gnome-shell] cleanup: Switch some method calls to new indentation style
- From: Florian Müllner <fmuellner src gnome org>
- To: commits-list gnome org
- Cc:
- Subject: [gnome-shell] cleanup: Switch some method calls to new indentation style
- Date: Sun, 15 Sep 2019 13:34:57 +0000 (UTC)
commit 3a9eaa39ea50c909a3409d236c74259db4a2f079
Author: Florian Müllner <fmuellner gnome org>
Date: Fri Sep 13 00:41:47 2019 +0200
cleanup: Switch some method calls to new indentation style
The legacy indent rule currently ignores arrow functions in parameters
to allow callbacks to not align with the other arguments:
this._someFunctionWithFairlyLongishName(arg1, arg2, arg3,
() => {
this._someOtherFunctionWithLongName(arg1);
});
But as ignoring entire nodes means we can end up with arbitrary
indentation, we should drop the exception. While this would make
the above "illegal" under the legacy config, it conforms with the
non-legacy style, so everything should be fine ...
... except that eslint starts to complain about some function args
that should be fine under the legacy config. Maybe it's thrown off
by the function-arg-in-arrow-function-in-function-arg structure, but
rather than figuring it out, let's just move those to the new style.
https://gitlab.gnome.org/GNOME/gnome-shell/merge_requests/725
js/ui/screenshot.js | 12 ++++++------
js/ui/status/accessibility.js | 4 ++--
2 files changed, 8 insertions(+), 8 deletions(-)
---
diff --git a/js/ui/screenshot.js b/js/ui/screenshot.js
index c9ede2fc1d..33233db90d 100644
--- a/js/ui/screenshot.js
+++ b/js/ui/screenshot.js
@@ -116,8 +116,8 @@ var ScreenshotService = class {
try {
let [result, area, filenameUsed] =
screenshot.screenshot_area_finish(res);
- this._onScreenshotComplete(result, area, filenameUsed,
- flash, invocation);
+ this._onScreenshotComplete(
+ result, area, filenameUsed, flash, invocation);
} catch (e) {
invocation.return_gerror (e);
}
@@ -134,8 +134,8 @@ var ScreenshotService = class {
try {
let [result, area, filenameUsed] =
screenshot.screenshot_window_finish(res);
- this._onScreenshotComplete(result, area, filenameUsed,
- flash, invocation);
+ this._onScreenshotComplete(
+ result, area, filenameUsed, flash, invocation);
} catch (e) {
invocation.return_gerror (e);
}
@@ -152,8 +152,8 @@ var ScreenshotService = class {
try {
let [result, area, filenameUsed] =
screenshot.screenshot_finish(res);
- this._onScreenshotComplete(result, area, filenameUsed,
- flash, invocation);
+ this._onScreenshotComplete(
+ result, area, filenameUsed, flash, invocation);
} catch (e) {
invocation.return_gerror (e);
}
diff --git a/js/ui/status/accessibility.js b/js/ui/status/accessibility.js
index 56d7802b3f..0423fd510b 100644
--- a/js/ui/status/accessibility.js
+++ b/js/ui/status/accessibility.js
@@ -179,8 +179,8 @@ class ATIndicator extends PanelMenu.Button {
settings.is_writable(KEY_TEXT_SCALING_FACTOR),
enabled => {
if (enabled)
- settings.set_double(KEY_TEXT_SCALING_FACTOR,
- DPI_FACTOR_LARGE);
+ settings.set_double(
+ KEY_TEXT_SCALING_FACTOR, DPI_FACTOR_LARGE);
else
settings.reset(KEY_TEXT_SCALING_FACTOR);
});
[
Date Prev][
Date Next] [
Thread Prev][
Thread Next]
[
Thread Index]
[
Date Index]
[
Author Index]