Commit Graph

441 Commits

Author SHA1 Message Date
Florian Müllner acb44de469 cleanup: Port GObject classes to ES6 classes
GJS added API for defining GObject classes with ES6 class syntax
last cycle, use it to port the remaining Lang.Class classes to
the new syntax.

Fixes https://gitlab.gnome.org/GNOME/gnome-shell-extensions/issues/30
2017-12-01 20:50:52 +01:00
Florian Müllner 48399d46c0 cleanup: Port non-GObject classes to ES6 classes
ES6 finally adds standard class syntax to the language, so we can
replace our custom Lang.Class framework with the new syntax. Any
classes that inherit from GObject will need special treatment,
so limit the port to regular javascript classes for now.

Fixes https://gitlab.gnome.org/GNOME/gnome-shell-extensions/issues/30
2017-12-01 20:50:46 +01:00
Florian Müllner 30e7ff5a8f cleanup: Use method syntax
Modern javascript has a short-hand for function properties, embrace
it for better readability and to prepare for porting to ES6 classes.

Fixes https://gitlab.gnome.org/GNOME/gnome-shell-extensions/issues/30
2017-12-01 20:50:28 +01:00
Florian Müllner 968ae427f1 cleanup: Use arrow-notation for anonymous functions
Arrow notation is great, but as we only started using it recently,
we currently have a wild mix of Lang.bind(), function() and () => {}.
To make the style consistent again, change all anonymous functions
to arrow notation.

Fixes https://gitlab.gnome.org/GNOME/gnome-shell-extensions/issues/30
2017-12-01 20:40:32 +01:00
Florian Müllner 4ee30a5445 cleanup: Replace tabs with spaces
Fixes https://gitlab.gnome.org/GNOME/gnome-shell-extensions/issues/30
2017-12-01 20:38:38 +01:00
Florian Müllner d3bbeb1b0c cleanup: Fix some coding style issues
Fixes https://gitlab.gnome.org/GNOME/gnome-shell-extensions/issues/30
2017-12-01 20:38:38 +01:00
Florian Müllner b90c99b87c build: Optionally syntax-check sources
As a pure javascript project, building is really just a glorified
copy operation, so success doesn't even indicate that sources are
syntactically correct (a.k.a. "compile-tested"). We can at least
get some minimal testing by performing some basic syntax checking
when SpilderMonkey's JS shell is available.

Fixes https://gitlab.gnome.org/GNOME/gnome-shell-extensions/issues/32
2017-12-01 18:58:17 +01:00
Florian Müllner ba38019049 build: Remove autotools support
Complete the switch to meson and drop autotools support ...

Fixes https://gitlab.gnome.org/GNOME/gnome-shell-extensions/issues/31
2017-12-01 17:43:21 +01:00
Florian Müllner 406f4cd170 docs: Update instructions for extension additions for meson
The steps for adding a new extension are clearly different in meson,
so update the instructions accordingly. Don't bother with keeping
the existing autotools steps - supporting both build systems in
parallel is just temporary, autotools is on its way out.

Fixes https://gitlab.gnome.org/GNOME/gnome-shell-extensions/issues/31
2017-12-01 17:43:21 +01:00
Florian Müllner 3ab01aebec build: Support the meson build system
Meson is on track to replace autotools as the build system of choice,
so support it in addition to autotools.

Fixes https://gitlab.gnome.org/GNOME/gnome-shell-extensions/issues/31
2017-12-01 17:41:04 +01:00
Piotr Drąg a26f9e34c7 places-menu: Use Unicode quotation marks in a new string
See https://developer.gnome.org/hig/stable/typography.html

https://bugzilla.gnome.org/show_bug.cgi?id=772211
2017-11-25 16:21:35 +01:00
Florian Müllner 21315d74c3 apps-menu: Support separators
We currently only load entries and directories, and ignore any
separators defined by the user/admin. Make some people happy
by supporting them ...

Fixes https://gitlab.gnome.org/GNOME/gnome-shell-extensions/issues/27
2017-11-25 06:46:29 +01:00
Florian Müllner d86044f383 apps-menu: Minor code cleanup
The parameter to _clearApplicationBox() has never been used, so
remove it. In fact, modern javascript makes the function so compact
that we can just move the code inline.

Fixes https://gitlab.gnome.org/GNOME/gnome-shell-extensions/issues/27
2017-11-25 06:46:29 +01:00
Florian Müllner 077a292952 apps-menu: Reload on tree changes
Now that we respect the sort order defined in the .menu file, make
sure to reload the menu on those changes as well, not just when
the installed apps themselves change.

Fixes https://gitlab.gnome.org/GNOME/gnome-shell-extensions/issues/27
2017-11-25 06:46:29 +01:00
Florian Müllner 8bd35eeb09 apps-menu: Don't override sort order
Keep the order in which GMenu returns loaded apps, so users can
reorder entries in Alacarte.

Fixes https://gitlab.gnome.org/GNOME/gnome-shell-extensions/issues/27
2017-11-25 06:46:29 +01:00
Rémy Lefevre eb425ac8a2 places-menu: Support unmounting ejectable places
Being able to unmount places that can be ejected directly from the
menu is convenient and consistent with Nautilus, so add an eject
button to items that are removable.

Fixes https://gitlab.gnome.org/GNOME/gnome-shell-extensions/issues/17
2017-11-25 06:22:16 +01:00
Florian Müllner 2460ac9df8 auto-move: Remove unused imports 2017-10-26 20:03:12 +02:00
Florian Müllner 8c551ba8ee windowsNavigator: Use regular for() loops
"for each ... in" has been deprecated for a long time and won't be
supported in upcoming SpiderMonkey versions, so replace it with
"for ... of" instead.
2017-10-26 20:03:12 +02:00
Florian Müllner ef99394ffb nativeWindowPlacement: Use custom strategy to hook into layout
Instead of copying a long function for a single changed line, wrap the
layout algorithm in a LayoutStrategy so the workspace code picks it
up without modifications.

https://bugzilla.gnome.org/show_bug.cgi?id=787934
2017-10-13 16:08:07 +02:00
Florian Müllner 778eebc421 nativeWindowPlacement: Handle captions-on-top settings changes
Currently the injection to move title captions to the top depends on
the value of the setting at the time the extension is enabled.
Instead, do the injections unconditionally and query the setting
inside the function to pick up settings changes.

https://bugzilla.gnome.org/show_bug.cgi?id=787934
2017-10-13 16:08:07 +02:00
Florian Müllner 97d1cbca19 naturalWindowPlacement: Don't copy relayout() function
To move title captions to the top, we can simply apply a translation
before calling the unmodified original function.

https://bugzilla.gnome.org/show_bug.cgi?id=787934
2017-10-13 16:08:07 +02:00
Florian Müllner a0cae3d130 naturalWindowPlacement: Remove unused stuff
https://bugzilla.gnome.org/show_bug.cgi?id=787934
2017-10-13 16:08:07 +02:00
Florian Müllner 01893ca048 apps-menu: Fix whitespace errors 2017-10-13 16:08:07 +02:00
Florian Müllner 530b165ddf nativeWindowPlacement: Adjust to overview changes
title._spacing is no longer defined, so we end up with bogus positions
when window-captions-on-top is set to true. Adjust the positioning to
do without that for now, though the whole extension could use a rewrite
to not copy everything-and-the-kitching-sink, or be killed off as yet
another extension from the original random collection that turned out
too expensive to keep dragging along ...

https://bugzilla.gnome.org/show_bug.cgi?id=787604
2017-10-04 01:06:42 +02:00
Piotr Drąg 824e79d83e gschema: Fix gettext-domain 2017-10-01 19:10:42 +02:00
Alexander Rüedlinger c1143fa716 apps-menu: Use "0o" prefix for octal literals
In short, gjs complains that octal escape sequences are deprecated
and advises to use the "0o" prefix for octal literals. Do that to
fix the warning.

https://bugzilla.gnome.org/show_bug.cgi?id=787294
2017-09-17 23:18:52 +02:00
Alexander Rüedlinger 37b95a8498 apps-menu: Remove excess argument
This was an oversight in commit 63653406a3 when porting from a
removed Popover API to ClutterActor.add_child().

https://bugzilla.gnome.org/show_bug.cgi?id=787294
2017-09-17 23:18:52 +02:00
Alexander Rüedlinger cccef0e857 places-menu: Fix JS warning
Define class PlacesManager with 'var' instead of 'const', to shut
up a gjs warning when accessing it from the extension.

https://bugzilla.gnome.org/show_bug.cgi?id=787291
2017-09-17 23:18:52 +02:00
Piotr Drąg 71ba8b04d7 drive-menu: Add a translator comment explaining a variable
https://bugzilla.gnome.org/show_bug.cgi?id=687590
2017-08-03 14:06:26 +02:00
Alexander Rüedlinger 69700dc29b apps-menu: Remove more dead code
Commit 0a91893b02 removed dead code, namely a ::panel-box-changed signal
handler. Unfortunately, it forgot to remove the associated code that
disconnects the signal.

Signed-off-by: Alexander Rüedlinger <a.rueedlinger@gmail.com>

https://bugzilla.gnome.org/show_bug.cgi?id=785202
2017-07-21 10:09:13 +02:00
Alexander Rüedlinger 0a91893b02 apps-menu: Remove dead code
This fell through the cracks in commit 2d1d81247 when adjusting to
the gnome-shell changes that also removed the ::panel-box-changed
signal ...

Signed-off-by: Alexander Rüedlinger <a.rueedlinger@gmail.com>

https://bugzilla.gnome.org/show_bug.cgi?id=785200
2017-07-21 01:34:38 +02:00
Florian Müllner 74643a62d1 screenshot-window-sizer: Add explicit keybinding for backward cycling
The REVERSES flag was removed from Meta.KeyBindingFlags a while ago, as
gnome-control-center doesn't recognize it and the corresponding "magic"
shift handling. That is, nowadays reversible keybindings need to
provide an explicit reversed binding.

https://bugzilla.gnome.org/show_bug.cgi?id=784079
2017-07-05 17:06:37 +02:00
Florian Müllner eafdd1d78d window-list: Respect MWM hints
The window context menu contains minimize, maximize and close items
that are currently enabled unconditionally, regardless of whether
the window indicates support. Respect those hints by updating the
items' sensitivity every time the popup is shown.

https://bugzilla.gnome.org/show_bug.cgi?id=783601
2017-06-09 17:54:24 +02:00
Florian Müllner 61a2e5ab9e window-list: Do not pick up transparency from top bar
The top bar now uses a translucent style when no windows are in its
proximity. As translucency looks odd in some situations (in particular
with maximized windows), we don't want to pick it up unconditionally.
If someone fancies to integrate with the top bar's proximity tracking,
they are welcome to have a go, but for now we just restore the former
solid style unconditionally.
2017-06-09 17:54:24 +02:00
Christian Kellner 0824847ac1 places-menu: can now ask user for info on mounting
When we try to launch an application for an uri where the enclosing
mount is not yet mounted we might need information from the user
such as passwwords. Using a MountOperation makes this possible.

https://bugzilla.gnome.org/show_bug.cgi?id=781788
2017-06-09 16:35:48 +02:00
Florian Müllner c5da5cf9b5 drive-menu: Fix launch-nautilus label
The user-visible name of nautilus is "Files", not "File" ...

https://bugzilla.gnome.org/show_bug.cgi?id=781235
2017-05-09 19:30:52 +02:00
Christian Kellner c82f8a995b places-menu: report errors for mounting of volumes
When launching an application for an uri we detect the case that
the volume is not mounted and try to mount it. If this fails we
don't report any error, so there is no feedback for the user.
2017-04-28 15:14:59 +02:00
Christian Kellner cfbb2a459b places-menu: make launching asynchronous
Use the async version of Gio.AppInfo.launch_default_for_uri so
we don't hang or block if the uri we are trying to launch the
application for is on slow or dead network connections.

https://bugzilla.gnome.org/show_bug.cgi?id=781831
2017-04-28 15:14:14 +02:00
Christian Kellner 8a4b245812 places-menu: make the icon query asynchronous
Don't use the sync version of g_file_query_info because that might
potentially block (especially when the connection is slow or drops).

https://bugzilla.gnome.org/show_bug.cgi?id=781765
2017-04-27 14:19:26 +02:00
Florian Müllner 8453cf05a7 apps-menu: 'Touch' copied .desktop file after updating attributes
At least for now, nautilus' file monitor is blind to certain metadata
updates:
https://git.gnome.org/browse/nautilus/tree/src/nautilus-mime-actions.c#n1537

So in order for the changes in the previous patch to take effect
immediately, we need to force nautilus to reload the file info.
Achieve this by updating the access time of the newly copied file
after marking it as trusted.

https://bugzilla.gnome.org/show_bug.cgi?id=781596
2017-04-24 17:22:26 +02:00
Florian Müllner 5cb2657df7 apps-menu: Mark copied .desktop files as trusted
The application can already be launched from the menu without further
confirmation from the user, so there is no security gain in asking the
user to trust it when launched from the desktop - just set the appropriate
attributes of the newly copied file to mark it as trusted to nautilus.

https://bugzilla.gnome.org/show_bug.cgi?id=781596
2017-04-24 17:22:26 +02:00
Florian Müllner 234cf96d39 apps-menu: Remove excess parameters
Someone mixed up add() and add_actor() - this has been present since the
the big rewrite based on the AxeMenu extension in commit 9211fa4409, so
there's little point in coming up with a replacement for something that
never had any effect to begin with ...
2017-04-22 00:59:00 +02:00
Florian Müllner 09a60a27ba apps-menu: Only enable DND when there's a desktop
It's not very useful to allow dragging when there's no drop target,
so tie the functionality added in the previous commit to the presence
of a DESKTOP window.

https://bugzilla.gnome.org/show_bug.cgi?id=780371
2017-03-22 19:15:28 +01:00
Florian Müllner 243f700fa2 apps-menu: Allow creating desktop launchers via DND
Back in the olden days, it used to be possible to drag items from
the application menu to the desktop to create a launcher shortcut.
Reimplement that "classic" functionality in the apps menu extension.

https://bugzilla.gnome.org/show_bug.cgi?id=780371
2017-03-22 19:15:28 +01:00
Florian Müllner 021037bfcd apps-menu: Use Map to keep track of app items
The use of Array to keep track of inserted items is extremely
confusing, as no elements are ever added to the array. What
the code actually does is monkey-patching properties into an
empty object (that happens to be of type "Array"). While the
direct idiomatic replacement would be {}, update the code to
use a proper map instead.

https://bugzilla.gnome.org/show_bug.cgi?id=780371
2017-03-21 21:01:04 +01:00
Piotr Drąg 5ad82d05f9 Use Unicode in translatable strings
See https://developer.gnome.org/hig/stable/typography.html

https://bugzilla.gnome.org/show_bug.cgi?id=772211
2017-02-16 02:20:03 +01:00
Felipe Borges 0f9ac6088f window-list: Hide workspace indicator when there's 1 workspace
There's no need to show the workspace indicator at the right
corner of the window-list if there's just a single workspace
AND the workspace creation is static. This saves us a bit more
of space.

https://bugzilla.gnome.org/show_bug.cgi?id=777504
2017-01-20 16:31:16 +01:00
Jakub Steiner 5e545d2991 theme: window-list minimized button
- when a window is minimized, it's unfocused, thus the button
in the window list should remain raised

https://bugzilla.gnome.org/show_bug.cgi?id=775282
2017-01-20 14:45:30 +01:00
Florian Müllner 0cdae2dae0 alternateTab: Don't take over 'switch-group' shortcuts
The original extension author really hated the app switcher with a
passion and took over all its uses, but there's really no reason
to replace the 'switch-group' shortcuts - not least because the
window switcher doesn't implement switching between windows of
a single application. So just keep the extension to making the
'switch-application' shortcuts behave as 'switch-windows' for the
"classic" session (and all users who rather install an extension
than change shortcut settings).

https://bugzilla.gnome.org/show_bug.cgi?id=771531
2016-11-14 12:30:12 +01:00
Florian Müllner f99b42e732 window-list: Update icon on app changes
We currently assume that the application associated with a particular
window is fixed. While this holds true for almost every application,
there are some cases of multi-app-packages like LibreOffice where
windows may change the properties used for application matching at
runtime. Catch those cases to make sure we display the correct icon
when the window shifts applications.

https://bugzilla.gnome.org/show_bug.cgi?id=771731
2016-09-20 18:06:08 +02:00