Commit 467c2eff authored by António Fernandes's avatar António Fernandes 🐚 Committed by Carlos Soriano Sánchez

window: don't always make sidebar visible on new windows

We are setting the sidebar as visible in the ui definition and binding
this visibility to the disabled-chrome property (inversely) to prevent
the sidebar from being visible in the desktop. But this also makes
the sidebar visible in all new windows, ignoring both the state of the
"Sidebar" toggle in the app menu and the start-with-sidebar setting.

Fix this by setting the sidebar as not visible in the ui definition and
removing the binding. The sidebar will be made visible if appropriate
in the nautilus-window-initialize-actions function.

Fixes: https://bugzilla.gnome.org/show_bug.cgi?id=746217
parent 40599696
......@@ -2493,10 +2493,6 @@ nautilus_window_init (NautilusWindow *window)
g_signal_connect_object (window->priv->notification_delete_undo, "clicked",
G_CALLBACK (nautilus_window_on_notification_delete_undo_clicked), window, 0);
g_object_bind_property (window, "disable-chrome",
window->priv->sidebar, "visible",
G_BINDING_INVERT_BOOLEAN);
window->priv->slots = NULL;
window->priv->active_slot = NULL;
......
......@@ -26,7 +26,7 @@
<property name="hexpand">True</property>
<child>
<object class="GtkBox" id="sidebar">
<property name="visible">True</property>
<property name="visible">False</property>
<property name="orientation">vertical</property>
<property name="spacing">6</property>
<child>
......
Markdown is supported
0% or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment