Add a sidebar with places

It happens to fix an issue with scroll offset resets in the browser.

Otherwise, it's very much WIP.
This commit is contained in:
Přemysl Eric Janouch 2021-11-18 10:35:58 +01:00
parent e045a35437
commit 0433c1a027
Signed by: p
GPG Key ID: A0420B94F92B9493
2 changed files with 48 additions and 6 deletions

View File

@ -198,6 +198,7 @@ fastiv_view_realize(GtkWidget *widget)
#ifdef GDK_WINDOWING_X11 #ifdef GDK_WINDOWING_X11
// FIXME: This causes some flicker while scrolling, because it disables // FIXME: This causes some flicker while scrolling, because it disables
// double buffering, see: https://gitlab.gnome.org/GNOME/gtk/-/issues/2560 // double buffering, see: https://gitlab.gnome.org/GNOME/gtk/-/issues/2560
// FIXME: It also breaks Tab-switching at the start of program.
// //
// If GTK+'s OpenGL integration fails to deliver, we need to use the window // If GTK+'s OpenGL integration fails to deliver, we need to use the window
// directly, sidestepping the toolkit entirely. // directly, sidestepping the toolkit entirely.

View File

@ -67,6 +67,8 @@ struct {
GtkWidget *view_scroller; GtkWidget *view_scroller;
GtkWidget *browser; GtkWidget *browser;
GtkWidget *browser_scroller; GtkWidget *browser_scroller;
GtkWidget *browser_paned;
GtkWidget *browser_sidebar;
} g; } g;
static gboolean static gboolean
@ -109,6 +111,11 @@ load_directory(const gchar *dirname)
g_ptr_array_set_size(g.files, 0); g_ptr_array_set_size(g.files, 0);
g.files_index = -1; g.files_index = -1;
GFile *file = g_file_new_for_path(dirname);
gtk_places_sidebar_set_location(
GTK_PLACES_SIDEBAR(g.browser_sidebar), file);
g_object_unref(file);
fastiv_browser_load(FASTIV_BROWSER(g.browser), dirname); fastiv_browser_load(FASTIV_BROWSER(g.browser), dirname);
GError *error = NULL; GError *error = NULL;
@ -244,6 +251,15 @@ on_item_activated(G_GNUC_UNUSED FastivBrowser *browser, const char *path,
open(path); open(path);
} }
static void
on_open_location(G_GNUC_UNUSED GtkPlacesSidebar *sidebar, GFile *location,
G_GNUC_UNUSED GtkPlacesOpenFlags flags, G_GNUC_UNUSED gpointer user_data)
{
gchar *path = g_file_get_path(location);
load_directory(path);
g_free(path);
}
// Cursor keys, e.g., simply cannot be bound through accelerators // Cursor keys, e.g., simply cannot be bound through accelerators
// (and GtkWidget::keynav-failed would arguably be an awful solution). // (and GtkWidget::keynav-failed would arguably be an awful solution).
// //
@ -286,12 +302,19 @@ on_key_press(G_GNUC_UNUSED GtkWidget *widget, GdkEventKey *event,
on_open(); on_open();
return TRUE; return TRUE;
case GDK_KEY_F9:
if (gtk_widget_is_visible(g.browser_sidebar))
gtk_widget_hide(g.browser_sidebar);
else
gtk_widget_show(g.browser_sidebar);
return TRUE;
case GDK_KEY_Tab: case GDK_KEY_Tab:
case GDK_KEY_Return: case GDK_KEY_Return:
gtk_stack_set_visible_child(GTK_STACK(g.stack), gtk_stack_set_visible_child(GTK_STACK(g.stack),
gtk_stack_get_visible_child(GTK_STACK(g.stack)) == gtk_stack_get_visible_child(GTK_STACK(g.stack)) ==
g.view_scroller g.view_scroller
? g.browser_scroller ? g.browser_paned
: g.view_scroller); : g.view_scroller);
return TRUE; return TRUE;
} }
@ -328,7 +351,7 @@ on_button_press_view(G_GNUC_UNUSED GtkWidget *widget, GdkEventButton *event)
{ {
if (!(event->state & gtk_accelerator_get_default_mod_mask()) && if (!(event->state & gtk_accelerator_get_default_mod_mask()) &&
event->button == 8 /* back */) { event->button == 8 /* back */) {
gtk_stack_set_visible_child(GTK_STACK(g.stack), g.browser_scroller); gtk_stack_set_visible_child(GTK_STACK(g.stack), g.browser_paned);
return TRUE; return TRUE;
} }
return FALSE; return FALSE;
@ -412,14 +435,30 @@ main(int argc, char *argv[])
g_signal_connect(g.browser, "item-activated", g_signal_connect(g.browser, "item-activated",
G_CALLBACK(on_item_activated), NULL); G_CALLBACK(on_item_activated), NULL);
gtk_container_add(GTK_CONTAINER(g.browser_scroller), g.browser); gtk_container_add(GTK_CONTAINER(g.browser_scroller), g.browser);
// TODO(p): Put a GtkListBox underneath, but with subdirectories.
// - Set the scrolled window's vertical policy to nope,
// and put it inside another scrolled window.
g.browser_sidebar = gtk_places_sidebar_new();
gtk_places_sidebar_set_show_recent(
GTK_PLACES_SIDEBAR(g.browser_sidebar), FALSE);
gtk_places_sidebar_set_show_trash(
GTK_PLACES_SIDEBAR(g.browser_sidebar), FALSE);
g_signal_connect(g.browser_sidebar, "open-location",
G_CALLBACK(on_open_location), NULL);
g.browser_paned = gtk_paned_new(GTK_ORIENTATION_HORIZONTAL);
gtk_paned_add1(GTK_PANED(g.browser_paned), g.browser_sidebar);
gtk_paned_add2(GTK_PANED(g.browser_paned), g.browser_scroller);
// TODO(p): Can we not do it here separately? // TODO(p): Can we not do it here separately?
gtk_widget_show_all(g.browser_scroller); gtk_widget_show_all(g.browser_paned);
g.stack = gtk_stack_new(); g.stack = gtk_stack_new();
gtk_stack_set_transition_type( gtk_stack_set_transition_type(
GTK_STACK(g.stack), GTK_STACK_TRANSITION_TYPE_NONE); GTK_STACK(g.stack), GTK_STACK_TRANSITION_TYPE_NONE);
gtk_container_add(GTK_CONTAINER(g.stack), g.view_scroller); gtk_container_add(GTK_CONTAINER(g.stack), g.view_scroller);
gtk_container_add(GTK_CONTAINER(g.stack), g.browser_scroller); gtk_container_add(GTK_CONTAINER(g.stack), g.browser_paned);
g.window = gtk_window_new(GTK_WINDOW_TOPLEVEL); g.window = gtk_window_new(GTK_WINDOW_TOPLEVEL);
gtk_window_set_default_size (GTK_WINDOW (g.window), 800, 600); gtk_window_set_default_size (GTK_WINDOW (g.window), 800, 600);
@ -452,8 +491,10 @@ main(int argc, char *argv[])
} }
g_free(cwd); g_free(cwd);
if (g.files_index < 0) if (g.files_index < 0) {
gtk_stack_set_visible_child(GTK_STACK(g.stack), g.browser_scroller); gtk_stack_set_visible_child(GTK_STACK(g.stack), g.browser_paned);
gtk_widget_grab_focus(g.browser_scroller);
}
gtk_widget_show_all(g.window); gtk_widget_show_all(g.window);
gtk_main(); gtk_main();