Skip to content

Commit

Permalink
Add support for indicators in Ubuntu Gnome Session
Browse files Browse the repository at this point in the history
Ubuntu is switching to Gnome in 17.10 but it's not going to be the stock Gnome session. It's going to support the existing Unity APIs. Support has already landed for application indicators and likely other APIs (Launcher,etc) will be supported soon. The Ubuntu Gnome session will have `XDG_CURRENT_DESKTOP=ubuntu:GNOME` environment variable.

Ref: https://didrocks.fr/2017/08/23/ubuntu-gnome-shell-in-artful-day-7/
  • Loading branch information
owais committed Aug 25, 2017
1 parent 6a93052 commit dc43dc2
Showing 1 changed file with 1 addition and 1 deletion.
2 changes: 1 addition & 1 deletion lib/browser/init.js
Original file line number Diff line number Diff line change
Expand Up @@ -163,7 +163,7 @@ require('./api/protocol')
const mainStartupScript = packageJson.main || 'index.js'

// Workaround for electron/electron#5050 and electron/electron#9046
if (process.platform === 'linux' && ['Pantheon', 'Unity:Unity7'].includes(process.env.XDG_CURRENT_DESKTOP)) {
if (process.platform === 'linux' && ['Pantheon', 'Unity:Unity7', 'ubuntu:GNOME'].includes(process.env.XDG_CURRENT_DESKTOP)) {
process.env.XDG_CURRENT_DESKTOP = 'Unity'
}

Expand Down

1 comment on commit dc43dc2

@3v1n0
Copy link
Contributor

@3v1n0 3v1n0 commented on dc43dc2 Oct 12, 2017

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Proper fix to this sould be done in libchromiumcontent though, and even better it should be check for dbus interfaces more than this.

Please sign in to comment.