Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

multiple panels on undo #46

Closed
fossfreedom opened this issue Oct 26, 2016 · 7 comments
Closed

multiple panels on undo #46

fossfreedom opened this issue Oct 26, 2016 · 7 comments
Assignees
Labels

Comments

@fossfreedom
Copy link
Member

https://bugs.launchpad.net/budgie-remix/+bug/1636800

ProblemType: Bug
DistroRelease:Budgie remix 16.10
Package: Budgie v10.2.8
Uname: Linux 4.8.0-26-generic
Architecture: 64 bit

When using the Budgie Welcome->Default Settings->Restore panels->Reset Budgie panel and then undo reset it will multiply the panel/s every time.
The single way to fix this is to restart the system.

Screenshots as per the bug report

@niyasc
Copy link
Contributor

niyasc commented Oct 26, 2016

Strange. Is it something associated with new budgie release?

@fossfreedom
Copy link
Member Author

not sure @niyasc - I haven't looked at the code ... but for the undo routine I presume we clear the old "file" before we save the new panel values before doing the reset?

@ikeydoherty
Copy link

Doing the dconf reset is hazardous as it messes with the currently running panel managers internal state. This is why the latest git version of Budgie has a new "--reset --replace&" option. I am going to go one further though, and have a single key to define the default panel layout, and allow the shipment of multiple panel layouts within Budgie Desktop, which will help resolve this issue.

@fossfreedom
Copy link
Member Author

The new budgie-panel --reset method in yakkety and later is broken due to the later glib version.

Going to have to code around this - use the reset method for 16.04 and a dconf method for 16.10 and later

@fossfreedom fossfreedom self-assigned this Jan 13, 2017
@niyasc
Copy link
Contributor

niyasc commented Jan 14, 2017

But the initial approach (dconf reset) was causing the problem of having multiple panels. Right?

@fossfreedom
Copy link
Member Author

fossfreedom commented Jan 14, 2017 via email

@fossfreedom
Copy link
Member Author

k - have removed the undo - it is causing too many issues with applets being duplicated on panels and flicking around.

Put pack in the dconf reset method for 16.10 and later.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

3 participants