[Bug 1769383] Re: Ubuntu dock/launcher is shown on the lock screen

2018-11-14 Thread pdrd
@azzar1 Did you try out the suggested fix by @mchaberski? The enable()
function sets dockManager intially to null and then calls
conditionallyenabledock(). But in the disable() function always
dockManager.destroy() gets called without a type-check raising a
TypeError. I think the null-check should be implemented in any way.

This would explain why the suggested fix by @didrocks is working by
disabling mod extensions, because disable() is not getting called
anymore.

Maybe someone can try out if the code change fixes the bug for the
normal dock, cause i am using dash-to-dock.

-- 
You received this bug notification because you are a member of Ubuntu
Desktop Bugs, which is subscribed to gnome-shell in Ubuntu.
https://bugs.launchpad.net/bugs/1769383

Title:
  Ubuntu dock/launcher is shown on the lock screen

To manage notifications about this bug go to:
https://bugs.launchpad.net/ubuntu/+source/gnome-shell/+bug/1769383/+subscriptions

-- 
desktop-bugs mailing list
desktop-bugs@lists.ubuntu.com
https://lists.ubuntu.com/mailman/listinfo/desktop-bugs

[Bug 1769383] Re: Ubuntu dock/launcher is shown on the lock screen

2018-11-15 Thread pdrd
Sorry, i missed comment #29, but there are a lot of bug reports out
there, some of them over a year old, which describe the same issue.

My reproduction steps, tested on a virtualbox machine:

1) Fresh-Install Ubuntu 18.0.4.1 LTS
2) Install package 'chrome-gnome-shell'
3) Navigate to https://extensions.gnome.org/extension/307/dash-to-dock, install 
firefox/chrome extension and install dash-to-dock extension.
4) Restart
5) On next lock the ubuntu-dock reappears and the following journalctl 
/usr/bin/gnome-shell output is generated: 

Nov 15 12:53:15 ubuntu gnome-shell[1143]: Extension "ubuntu-
d...@ubuntu.com" had error: TypeError: dockManager is null


The suggested fix is resolving this and a quick look into 
https://git.launchpad.net/ubuntu/+source/gnome-shell-extension-ubuntu-dock/tree/extension.js?h=ubuntu/bionic
 exposes, that the issue is a simple missing null-check in the disable() 
function where dockManager.destroy() is called on a null-instance, which should 
be added in either way.

-- 
You received this bug notification because you are a member of Ubuntu
Desktop Bugs, which is subscribed to gnome-shell in Ubuntu.
https://bugs.launchpad.net/bugs/1769383

Title:
  Ubuntu dock/launcher is shown on the lock screen

To manage notifications about this bug go to:
https://bugs.launchpad.net/ubuntu/+source/gnome-shell/+bug/1769383/+subscriptions

-- 
desktop-bugs mailing list
desktop-bugs@lists.ubuntu.com
https://lists.ubuntu.com/mailman/listinfo/desktop-bugs

[Bug 1769383] Re: Ubuntu dock/launcher is shown on the lock screen

2018-11-16 Thread pdrd
@vanvugt: You're absolutely right, but the dockManager-instance is null
because of an explicit exception in
https://git.launchpad.net/ubuntu/+source/gnome-shell-extension-ubuntu-
dock/tree/extension.js?h=ubuntu/bionic to disable ubuntu-dock if dash-
to-dock is installed:

let to_enable = true;
let runningExtensions = ExtensionSystem.extensionOrder;
for (let i = 0; i < runningExtensions.length; i++) {
if (runningExtensions[i] === "dash-to-d...@micxgx.gmail.com") {
to_enable = false;
}
}

So i'm not able to fix this issue in the dash-to-dock repo. May we
create a seperate bug report?

-- 
You received this bug notification because you are a member of Ubuntu
Desktop Bugs, which is subscribed to gnome-shell in Ubuntu.
https://bugs.launchpad.net/bugs/1769383

Title:
  Ubuntu dock/launcher is shown on the lock screen

To manage notifications about this bug go to:
https://bugs.launchpad.net/ubuntu/+source/gnome-shell/+bug/1769383/+subscriptions

-- 
desktop-bugs mailing list
desktop-bugs@lists.ubuntu.com
https://lists.ubuntu.com/mailman/listinfo/desktop-bugs