"Applications" launcher is present in Wingpanel in Ubiquity session

[Sergey "Shnatsel" Davidoff]

"Applications" launcher is present in Wingpanel in the LiveCD greeter with the language selection try/install buttons; it's not supposed to be there.

Ubiquity now unsets the dconf key correctly. A screenshot of dconf-editor showing the key being unset yet Wingpanel still showing the launcher is attached. Killing Wingpanel and launching it again fixes the issue.

There seems to be a race condition in Wingpanel in setting up dconf watches.

[Andrea Basso (voluntatefaber)]

I can't confirm whether it shows or not during installing, but I can confirm show_launcher is not updated on the fly.

[Andrea Basso (voluntatefaber)]

Uhm, this is strange... I've hacked a bit, then reverted changes and now it works...

[Andrea Basso (voluntatefaber)]

Ok, my bad, I musta done something wrong, I re-branched wingpanel and now it is updated on the fly.

[Sergey "Shnatsel" Davidoff (shnatsel)]

It does update on the fly when I manually toggle it. However, it does not when Ubiquity unsets the key on startup. The only reasonable explaination I came up with is a race condition in setting up or using the dconf watches.

[Rico Tzschichholz (ricotz)]

Could be a race, but cleaning up the responsible code *might* work too.

[Rico Tzschichholz (ricotz)]

I pushed this proposed *maybe* fix.

[Sergey "Shnatsel" Davidoff (shnatsel)]

Unfortunately, that didn't fix it.

[Danielle Foré (danrabbit)]

One thing we should consider is maybe this is actually a feature? I've heard users actually applauding that we allow them to access all apps during install.

[Victor Martinez (victored)]

The gsettings key works perfectly here. It's also applied accordingly when the value is changed.

[Sergey "Shnatsel" Davidoff (shnatsel)]

Awaiting design decision - should we get rid of it or not?

[Danielle Foré (danrabbit)]

Let's just decide this is a feature since we don't know wtf is going wrong :P