From d83d6e857a2b644b3d9b49e41faa808672807859 Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?Florian=20M=C3=BCllner?= Date: Thu, 28 Feb 2019 19:18:53 +0100 Subject: [PATCH] style: Use single-quotes for translated strings The coding style of using double quotes for translatable strings and single quotes otherwise is unnecessarily complex and cannot be enforced with an eslint rule. Simply use single quotes consistently for all strings. https://gitlab.gnome.org/GNOME/gnome-shell-extensions/merge_requests/57 --- extensions/apps-menu/extension.js | 6 +++--- extensions/auto-move-windows/prefs.js | 20 ++++++++++---------- extensions/drive-menu/extension.js | 6 +++--- extensions/places-menu/extension.js | 4 ++-- extensions/places-menu/placeDisplay.js | 14 +++++++------- extensions/window-list/extension.js | 20 ++++++++++---------- extensions/window-list/prefs.js | 10 +++++----- extensions/workspace-indicator/extension.js | 2 +- extensions/workspace-indicator/prefs.js | 6 +++--- lint/eslintrc-legacy.json | 3 +-- 10 files changed, 45 insertions(+), 46 deletions(-) diff --git a/extensions/apps-menu/extension.js b/extensions/apps-menu/extension.js index fb5d6fd3..0c991231 100644 --- a/extensions/apps-menu/extension.js +++ b/extensions/apps-menu/extension.js @@ -26,7 +26,7 @@ class ActivitiesMenuItem extends PopupMenu.PopupBaseMenuItem { constructor(button) { super(); this._button = button; - this.actor.add_child(new St.Label({ text: _("Activities Overview") })); + this.actor.add_child(new St.Label({ text: _('Activities Overview') })); } activate(event) { @@ -115,7 +115,7 @@ class CategoryMenuItem extends PopupMenu.PopupBaseMenuItem { if (this._category) name = this._category.get_name(); else - name = _("Favorites"); + name = _('Favorites'); this.actor.add_child(new St.Label({ text: name })); this.actor.connect('motion-event', this._onMotionEvent.bind(this)); @@ -404,7 +404,7 @@ class ApplicationsButton extends PanelMenu.Button { let hbox = new St.BoxLayout({ style_class: 'panel-status-menu-box' }); this._label = new St.Label({ - text: _("Applications"), + text: _('Applications'), y_expand: true, y_align: Clutter.ActorAlign.CENTER }); diff --git a/extensions/auto-move-windows/prefs.js b/extensions/auto-move-windows/prefs.js index 3b3a70e6..f808347c 100644 --- a/extensions/auto-move-windows/prefs.js +++ b/extensions/auto-move-windows/prefs.js @@ -57,25 +57,25 @@ const Widget = GObject.registerClass({ let appColumn = new Gtk.TreeViewColumn({ expand: true, sort_column_id: Columns.DISPLAY_NAME, - title: _("Application") + title: _('Application') }); let iconRenderer = new Gtk.CellRendererPixbuf; appColumn.pack_start(iconRenderer, false); - appColumn.add_attribute(iconRenderer, "gicon", Columns.ICON); + appColumn.add_attribute(iconRenderer, 'gicon', Columns.ICON); let nameRenderer = new Gtk.CellRendererText; appColumn.pack_start(nameRenderer, true); - appColumn.add_attribute(nameRenderer, "text", Columns.DISPLAY_NAME); + appColumn.add_attribute(nameRenderer, 'text', Columns.DISPLAY_NAME); this._treeView.append_column(appColumn); let workspaceColumn = new Gtk.TreeViewColumn({ - title: _("Workspace"), + title: _('Workspace'), sort_column_id: Columns.WORKSPACE }); let workspaceRenderer = new Gtk.CellRendererSpin({ editable: true }); workspaceRenderer.connect('edited', this._workspaceEdited.bind(this)); workspaceColumn.pack_start(workspaceRenderer, true); - workspaceColumn.add_attribute(workspaceRenderer, "adjustment", Columns.ADJUSTMENT); - workspaceColumn.add_attribute(workspaceRenderer, "text", Columns.WORKSPACE); + workspaceColumn.add_attribute(workspaceRenderer, 'adjustment', Columns.ADJUSTMENT); + workspaceColumn.add_attribute(workspaceRenderer, 'text', Columns.WORKSPACE); this._treeView.append_column(workspaceColumn); scrolled.add(this._treeView); @@ -86,7 +86,7 @@ const Widget = GObject.registerClass({ let newButton = new Gtk.ToolButton({ icon_name: 'bookmark-new-symbolic', - label: _("Add Rule"), + label: _('Add Rule'), is_important: true }); newButton.connect('clicked', this._createNew.bind(this)); @@ -108,13 +108,13 @@ const Widget = GObject.registerClass({ _createNew() { let dialog = new Gtk.Dialog({ - title: _("Create new matching rule"), + title: _('Create new matching rule'), transient_for: this.get_toplevel(), use_header_bar: true, modal: true }); dialog.add_button(Gtk.STOCK_CANCEL, Gtk.ResponseType.CANCEL); - let addButton = dialog.add_button(_("Add"), Gtk.ResponseType.OK); + let addButton = dialog.add_button(_('Add'), Gtk.ResponseType.OK); dialog.set_default_response(Gtk.ResponseType.OK); let grid = new Gtk.Grid({ @@ -131,7 +131,7 @@ const Widget = GObject.registerClass({ grid.attach(dialog._appChooser, 0, 0, 2, 1); grid.attach(new Gtk.Label({ - label: _("Workspace"), + label: _('Workspace'), halign: Gtk.Align.END }), 0, 1, 1, 1); let adjustment = new Gtk.Adjustment({ diff --git a/extensions/drive-menu/extension.js b/extensions/drive-menu/extension.js index 47537f5e..51b4aa67 100644 --- a/extensions/drive-menu/extension.js +++ b/extensions/drive-menu/extension.js @@ -97,7 +97,7 @@ class MountMenuItem extends PopupMenu.PopupBaseMenuItem { _reportFailure(exception) { // TRANSLATORS: %s is the filesystem name - let msg = _("Ejecting drive “%s” failed:").format(this.mount.get_name()); + let msg = _('Ejecting drive “%s” failed:').format(this.mount.get_name()); Main.notifyError(msg, exception.message); } @@ -113,7 +113,7 @@ class MountMenuItem extends PopupMenu.PopupBaseMenuItem { let DriveMenu = GObject.registerClass( class DriveMenu extends PanelMenu.Button { _init() { - super._init(0.0, _("Removable devices")); + super._init(0.0, _('Removable devices')); let hbox = new St.BoxLayout({ style_class: 'panel-status-menu-box' }); let icon = new St.Icon({ @@ -140,7 +140,7 @@ class DriveMenu extends PanelMenu.Button { this._monitor.get_mounts().forEach(this._addMount.bind(this)); this.menu.addMenuItem(new PopupMenu.PopupSeparatorMenuItem()); - this.menu.addAction(_("Open Files"), event => { + this.menu.addAction(_('Open Files'), event => { let appSystem = Shell.AppSystem.get_default(); let app = appSystem.lookup_app('org.gnome.Nautilus.desktop'); app.activate_full(-1, event.get_time()); diff --git a/extensions/places-menu/extension.js b/extensions/places-menu/extension.js index 11090e18..c477a4ac 100644 --- a/extensions/places-menu/extension.js +++ b/extensions/places-menu/extension.js @@ -76,11 +76,11 @@ const SECTIONS = [ let PlacesMenu = GObject.registerClass( class PlacesMenu extends PanelMenu.Button { _init() { - super._init(0.0, _("Places")); + super._init(0.0, _('Places')); let hbox = new St.BoxLayout({ style_class: 'panel-status-menu-box' }); let label = new St.Label({ - text: _("Places"), + text: _('Places'), y_expand: true, y_align: Clutter.ActorAlign.CENTER }); diff --git a/extensions/places-menu/placeDisplay.js b/extensions/places-menu/placeDisplay.js index 79d401de..037a3394 100644 --- a/extensions/places-menu/placeDisplay.js +++ b/extensions/places-menu/placeDisplay.js @@ -56,7 +56,7 @@ class PlaceInfo { if (e.matches(Gio.IOErrorEnum, Gio.IOErrorEnum.FAILED_HANDLED)) // e.g. user canceled the password dialog return; - Main.notifyError(_("Failed to mount volume for “%s”").format(this.name), e.message); + Main.notifyError(_('Failed to mount volume for “%s”').format(this.name), e.message); return; } @@ -69,7 +69,7 @@ class PlaceInfo { } }); } else { - Main.notifyError(_("Failed to launch “%s”").format(this.name), e.message); + Main.notifyError(_('Failed to launch “%s”').format(this.name), e.message); } } }; @@ -130,7 +130,7 @@ Signals.addSignalMethods(PlaceInfo.prototype); class RootInfo extends PlaceInfo { _init() { - super._init('devices', Gio.File.new_for_path('/'), _("Computer")); + super._init('devices', Gio.File.new_for_path('/'), _('Computer')); let busName = 'org.freedesktop.hostname1'; let objPath = '/org/freedesktop/hostname1'; @@ -153,7 +153,7 @@ class RootInfo extends PlaceInfo { // GDBusProxy will emit a g-properties-changed when hostname1 goes down // ignore it if (proxy.g_name_owner) { - this.name = proxy.PrettyHostname || _("Computer"); + this.name = proxy.PrettyHostname || _('Computer'); this.emit('changed'); } } @@ -214,7 +214,7 @@ class PlaceDeviceInfo extends PlaceInfo { } _reportFailure(exception) { - let msg = _("Ejecting drive “%s” failed:").format(this._mount.get_name()); + let msg = _('Ejecting drive “%s” failed:').format(this._mount.get_name()); Main.notifyError(msg, exception.message); } } @@ -340,7 +340,7 @@ var PlacesManager = class { this._places.special.push(new PlaceInfo('special', Gio.File.new_for_path(homePath), - _("Home"))); + _('Home'))); let specials = []; let dirs = DEFAULT_DIRECTORIES.slice(); @@ -384,7 +384,7 @@ var PlacesManager = class { this._places.devices.push(new RootInfo()); this._places.network.push(new PlaceInfo('network', Gio.File.new_for_uri('network:///'), - _("Browse Network"), + _('Browse Network'), 'network-workgroup-symbolic')); /* first go through all connected drives */ diff --git a/extensions/window-list/extension.js b/extensions/window-list/extension.js index 24cd2334..9e2ee94c 100644 --- a/extensions/window-list/extension.js +++ b/extensions/window-list/extension.js @@ -97,7 +97,7 @@ class WindowContextMenu extends PopupMenu.PopupMenu { this._updateMaximizeItem.bind(this)); this._updateMaximizeItem(); - this._closeItem = new PopupMenu.PopupMenuItem(_("Close")); + this._closeItem = new PopupMenu.PopupMenuItem(_('Close')); this._closeItem.connect('activate', () => { this._metaWindow.delete(global.get_current_time()); }); @@ -117,14 +117,14 @@ class WindowContextMenu extends PopupMenu.PopupMenu { _updateMinimizeItem() { this._minimizeItem.label.text = this._metaWindow.minimized ? - _("Unminimize") : _("Minimize"); + _('Unminimize') : _('Minimize'); } _updateMaximizeItem() { let maximized = this._metaWindow.maximized_vertically && this._metaWindow.maximized_horizontally; this._maximizeItem.label.text = maximized ? - _("Unmaximize") : _("Maximize"); + _('Unmaximize') : _('Maximize'); } _onDestroy() { @@ -405,19 +405,19 @@ class AppContextMenu extends PopupMenu.PopupMenu { this._appButton = appButton; - this._minimizeItem = new PopupMenu.PopupMenuItem(_("Minimize all")); + this._minimizeItem = new PopupMenu.PopupMenuItem(_('Minimize all')); this._minimizeItem.connect('activate', () => { this._appButton.getWindowList().forEach(w => w.minimize()); }); this.addMenuItem(this._minimizeItem); - this._unminimizeItem = new PopupMenu.PopupMenuItem(_("Unminimize all")); + this._unminimizeItem = new PopupMenu.PopupMenuItem(_('Unminimize all')); this._unminimizeItem.connect('activate', () => { this._appButton.getWindowList().forEach(w => w.unminimize()); }); this.addMenuItem(this._unminimizeItem); - this._maximizeItem = new PopupMenu.PopupMenuItem(_("Maximize all")); + this._maximizeItem = new PopupMenu.PopupMenuItem(_('Maximize all')); this._maximizeItem.connect('activate', () => { this._appButton.getWindowList().forEach(w => { w.maximize(Meta.MaximizeFlags.HORIZONTAL | @@ -426,7 +426,7 @@ class AppContextMenu extends PopupMenu.PopupMenu { }); this.addMenuItem(this._maximizeItem); - this._unmaximizeItem = new PopupMenu.PopupMenuItem(_("Unmaximize all")); + this._unmaximizeItem = new PopupMenu.PopupMenuItem(_('Unmaximize all')); this._unmaximizeItem.connect('activate', () => { this._appButton.getWindowList().forEach(w => { w.unmaximize(Meta.MaximizeFlags.HORIZONTAL | @@ -435,7 +435,7 @@ class AppContextMenu extends PopupMenu.PopupMenu { }); this.addMenuItem(this._unmaximizeItem); - let item = new PopupMenu.PopupMenuItem(_("Close all")); + let item = new PopupMenu.PopupMenuItem(_('Close all')); item.connect('activate', () => { this._appButton.getWindowList().forEach(w => { w.delete(global.get_current_time()); @@ -652,7 +652,7 @@ class AppButton extends BaseButton { let WorkspaceIndicator = GObject.registerClass( class WorkspaceIndicator extends PanelMenu.Button { _init() { - super._init(0.0, _("Workspace Indicator"), true); + super._init(0.0, _('Workspace Indicator'), true); this.setMenu(new PopupMenu.PopupMenu(this, 0.0, St.Side.BOTTOM)); this.add_style_class_name('window-list-workspace-indicator'); this.menu.actor.remove_style_class_name('panel-menu'); @@ -833,7 +833,7 @@ class WindowList { trackFullscreen: true }); Main.uiGroup.set_child_above_sibling(this.actor, Main.layoutManager.panelBox); - Main.ctrlAltTabManager.addGroup(this.actor, _("Window List"), 'start-here-symbolic'); + Main.ctrlAltTabManager.addGroup(this.actor, _('Window List'), 'start-here-symbolic'); this.actor.width = this._monitor.width; this.actor.connect('notify::height', this._updatePosition.bind(this)); diff --git a/extensions/window-list/prefs.js b/extensions/window-list/prefs.js index 3c2960a6..78792b5a 100644 --- a/extensions/window-list/prefs.js +++ b/extensions/window-list/prefs.js @@ -22,7 +22,7 @@ class WindowListPrefsWidget extends Gtk.Grid { this.row_spacing = 6; this.orientation = Gtk.Orientation.VERTICAL; - let groupingLabel = '%s'.format(_("Window Grouping")); + let groupingLabel = '%s'.format(_('Window Grouping')); this.add(new Gtk.Label({ label: groupingLabel, use_markup: true, halign: Gtk.Align.START @@ -44,9 +44,9 @@ class WindowListPrefsWidget extends Gtk.Grid { let modes = range.deep_unpack()[1].deep_unpack(); let modeLabels = { - 'never': _("Never group windows"), - 'auto': _("Group windows when space is limited"), - 'always': _("Always group windows") + 'never': _('Never group windows'), + 'auto': _('Group windows when space is limited'), + 'always': _('Always group windows') }; let radio = null; @@ -72,7 +72,7 @@ class WindowListPrefsWidget extends Gtk.Grid { } let check = new Gtk.CheckButton({ - label: _("Show on all monitors"), + label: _('Show on all monitors'), margin_top: 6 }); this._settings.bind('show-on-all-monitors', check, 'active', Gio.SettingsBindFlags.DEFAULT); diff --git a/extensions/workspace-indicator/extension.js b/extensions/workspace-indicator/extension.js index 38ed8703..3be12686 100644 --- a/extensions/workspace-indicator/extension.js +++ b/extensions/workspace-indicator/extension.js @@ -18,7 +18,7 @@ const WORKSPACE_KEY = 'workspace-names'; let WorkspaceIndicator = GObject.registerClass( class WorkspaceIndicator extends PanelMenu.Button { _init() { - super._init(0.0, _("Workspace Indicator")); + super._init(0.0, _('Workspace Indicator')); let workspaceManager = global.workspace_manager; diff --git a/extensions/workspace-indicator/prefs.js b/extensions/workspace-indicator/prefs.js index ed5c4785..d8640559 100644 --- a/extensions/workspace-indicator/prefs.js +++ b/extensions/workspace-indicator/prefs.js @@ -128,7 +128,7 @@ class WorkspaceSettingsWidget extends Gtk.Grid { this.orientation = Gtk.Orientation.VERTICAL; this.add(new Gtk.Label({ - label: '%s'.format(_("Workspace Names")), + label: '%s'.format(_('Workspace Names')), use_markup: true, margin_bottom: 6, hexpand: true, @@ -148,7 +148,7 @@ class WorkspaceSettingsWidget extends Gtk.Grid { vexpand: true }); - let column = new Gtk.TreeViewColumn({ title: _("Name") }); + let column = new Gtk.TreeViewColumn({ title: _('Name') }); let renderer = new Gtk.CellRendererText({ editable: true }); renderer.connect('edited', this._cellEdited.bind(this)); column.pack_start(renderer, true); @@ -188,7 +188,7 @@ class WorkspaceSettingsWidget extends Gtk.Grid { let iter = this._store.append(); let index = this._store.get_path(iter).get_indices()[0]; - let label = _("Workspace %d").format(index + 1); + let label = _('Workspace %d').format(index + 1); this._store.set(iter, [this._store.Columns.LABEL], [label]); } diff --git a/lint/eslintrc-legacy.json b/lint/eslintrc-legacy.json index 1c57f080..aca79adf 100644 --- a/lint/eslintrc-legacy.json +++ b/lint/eslintrc-legacy.json @@ -11,7 +11,6 @@ "CallExpression": { "arguments": "first" }, "MemberExpression": "off" } - ], - "quotes": "off" + ] } }