After an evaluation, GNOME has moved from Bugzilla to GitLab. Learn more about GitLab.
No new issues can be reported in GNOME Bugzilla anymore.
To report an issue in a GNOME project, go to GNOME GitLab.
Do not go to GNOME Gitlab for: Bluefish, Doxygen, GnuCash, GStreamer, java-gnome, LDTP, NetworkManager, Tomboy.
Bug 783563 - extensions: Make placeholder actually show up
extensions: Make placeholder actually show up
Status: RESOLVED FIXED
Product: gnome-tweak-tool
Classification: Applications
Component: general
unspecified
Other All
: Normal normal
: ---
Assigned To: GNOME Tweak Tool maintainer(s)
GNOME Tweak Tool maintainer(s)
Depends on:
Blocks:
 
 
Reported: 2017-06-08 17:55 UTC by Florian Müllner
Modified: 2017-06-08 20:53 UTC
See Also:
GNOME target: ---
GNOME version: ---


Attachments
extensions: Make placeholder actually show up (1.95 KB, patch)
2017-06-08 17:55 UTC, Florian Müllner
committed Details | Review
Screenshot: Tweak Tool on Kubuntu after patch 353418 applied (115.73 KB, image/png)
2017-06-08 18:16 UTC, Jeremy Bicha
  Details
extensions: Bail out early when not running under GNOME (1.62 KB, patch)
2017-06-08 18:29 UTC, Florian Müllner
committed Details | Review

Description Florian Müllner 2017-06-08 17:55:44 UTC
See patch.
Comment 1 Florian Müllner 2017-06-08 17:55:50 UTC
Created attachment 353418 [details] [review]
extensions: Make placeholder actually show up

Tweak tool automatically filters out empty tweak groups, which means the
extension tweak is hidden altogether when no extensions are installed
rather than showing the placeholder as intended.
Work around this by making the placeholder itself a tweak, so the group
is never considered empty.
Comment 2 Jeremy Bicha 2017-06-08 18:16:37 UTC
Created attachment 353420 [details]
Screenshot: Tweak Tool on Kubuntu after patch 353418 applied

Could you make it so that the Extensions panel does not show up if GNOME Shell is not running? This worked before the patch you posted here.

I'm attaching a screenshot of what it looks like in KDE Plasma after your patch. I tested with Kubuntu 17.10 Alpha daily:
http://cdimage.ubuntu.com/kubuntu/daily-live/current/

sudo apt install gnome-tweak-tool gnome-shell-extension-dashtodock

Tweak Tool's modules are currently installed to /usr/lib/python3/dist-packages/gtweak/

While I doubt GNOME Tweak Tool is popular with KDE Plasma users, it is used in other environments like MATE or Budgie.
Comment 3 Florian Müllner 2017-06-08 18:29:06 UTC
Created attachment 353424 [details] [review]
extensions: Bail out early when not running under GNOME

GNOME Shell extensions aren't useful in non-GNOME environments,
so return right after initializing the empty tweak group to make
sure it is kept hidden in that case.
Comment 4 Jeremy Bicha 2017-06-08 19:00:13 UTC
Review of attachment 353418 [details] [review]:

Thanks!

By the way, many users will already have some extensions installed (perhaps the distro installs GNOME Classic or specific extensions by default like is being proposed for Ubuntu 17.10). When you updated the Extensions page, you also removed the Get more extensions link from the bottom of the page. Maybe it's worth bringing it back, except pointed at GNOME Software instead of the website.
Comment 5 Jeremy Bicha 2017-06-08 19:00:24 UTC
Review of attachment 353424 [details] [review]:

Thanks!
Comment 6 Florian Müllner 2017-06-08 20:51:43 UTC
(In reply to Jeremy Bicha from comment #4)
> When you updated the Extensions page, you also removed the Get more extensions
> link from the bottom of the page. Maybe it's worth bringing it back

That's really a question for the design team, my patches were simply following https://raw.githubusercontent.com/gnome-design-team/gnome-mockups/master/tweak-tool/tweak-tool-wires.png
Comment 7 Florian Müllner 2017-06-08 20:53:26 UTC
Attachment 353418 [details] pushed as 3e95c35 - extensions: Make placeholder actually show up
Attachment 353424 [details] pushed as 1e9b9ef - extensions: Bail out early when not running under GNOME