[nautilus/wip/alexpandelea/batchRename: 11/17] Update UI and add functionalities to new UI
- From: Alexandru-Ionut Pandelea <alexpandelea src gnome org>
- To: commits-list gnome org
- Cc:
- Subject: [nautilus/wip/alexpandelea/batchRename: 11/17] Update UI and add functionalities to new UI
- Date: Tue, 5 Jul 2016 16:49:25 +0000 (UTC)
commit ceefb843aca5a42b3d90c0d34049b85e907b78b7
Author: Alexandru Pandelea <alexandru pandelea gmail com>
Date: Thu Jun 23 21:57:11 2016 +0300
Update UI and add functionalities to new UI
Now the user can replace text (as before) and the more complex mode only
does prepending for now, because the options in the popovers are still not
added.
https://bugzilla.gnome.org/show_bug.cgi?id=768311
src/nautilus-batch-rename.c | 129 +++++++++++++++-------
src/resources/ui/nautilus-batch-rename-dialog.ui | 77 +++++++++++--
2 files changed, 155 insertions(+), 51 deletions(-)
---
diff --git a/src/nautilus-batch-rename.c b/src/nautilus-batch-rename.c
index fa1e086..264f7cf 100644
--- a/src/nautilus-batch-rename.c
+++ b/src/nautilus-batch-rename.c
@@ -47,6 +47,10 @@ struct _NautilusBatchRename
GtkWidget *replace_entry;
GtkWidget *format_mode_button;
GtkWidget *replace_mode_button;
+ GtkWidget *add_button;
+ GtkWidget *add_popover;
+ GtkWidget *numbering_order_popover;
+ GtkWidget *numbering_order_button;
GList *listbox_rows;
@@ -136,41 +140,69 @@ listbox_header_func (GtkListBoxRow *row,
}
static GtkWidget*
-create_row_for_label (const gchar *text,
+create_row_for_label (const gchar *new_text,
+ const gchar *old_text,
gboolean show_separator)
{
GtkWidget *row;
- GtkWidget *label;
+ GtkWidget *label_new;
+ GtkWidget *label_old;
+ GtkWidget *arrow;
+ GtkWidget *box;
row = gtk_list_box_row_new ();
g_object_set_data (G_OBJECT (row), "show-separator", GINT_TO_POINTER (show_separator));
- label = g_object_new (GTK_TYPE_LABEL,
- "label", text,
+ arrow = g_object_new (GTK_TYPE_ARROW,
"hexpand", TRUE,
"xalign", 0.0,
"margin-start", 6,
NULL);
+ box = g_object_new (GTK_TYPE_BOX,
+ "orientation",GTK_ORIENTATION_HORIZONTAL,
+ "hexpand", TRUE,
+ NULL);
+ label_new = g_object_new (GTK_TYPE_LABEL,
+ "label",new_text,
+ "hexpand", TRUE,
+ "xalign", 0.0,
+ "margin-start", 6,
+ NULL);
+
+ label_old = g_object_new (GTK_TYPE_LABEL,
+ "label",old_text,
+ "hexpand", TRUE,
+ "xalign", 0.0,
+ "margin-start", 6,
+ NULL);
+ gtk_box_pack_end (GTK_BOX (box), label_new, FALSE, FALSE, 0);
+ gtk_box_pack_end (GTK_BOX (box), arrow, FALSE, FALSE, 0);
+ gtk_box_pack_end (GTK_BOX (box), label_old, FALSE, FALSE, 0);
gtk_list_box_row_set_selectable (GTK_LIST_BOX_ROW (row), FALSE);
- gtk_container_add (GTK_CONTAINER (row), label);
+ gtk_container_add (GTK_CONTAINER (row), box);
gtk_widget_show_all (row);
return row;
}
static void
-fill_conflict_listbox (NautilusBatchRename *dialog,
- GList *conflict_list)
+fill_display_listbox (NautilusBatchRename *dialog,
+ GList *new_names)
{
GtkWidget *row;
+ GList *l1;
+ GList *l2;
GList *l;
+ gchar *tmp_name;
+ NautilusFile *file;
/* clear rows from listbox (if there are any) */
if (dialog->listbox_rows != NULL)
for (l = dialog->listbox_rows; l != NULL; l = l->next) {
+ /*Fix me: figure why this shows warning on run */
gtk_container_remove (GTK_CONTAINER (dialog->conflict_listbox),
GTK_WIDGET (l->data));
}
@@ -178,18 +210,15 @@ fill_conflict_listbox (NautilusBatchRename *dialog,
g_list_free (dialog->listbox_rows);
dialog->listbox_rows = NULL;
- row = create_row_for_label ("These files already exist:", FALSE);
-
- gtk_container_add (GTK_CONTAINER (dialog->conflict_listbox), row);
-
/* add rows to a list so that they can be removed when there appears
* a new conflict */
dialog->listbox_rows = g_list_prepend (dialog->listbox_rows,
(gpointer) row);
- for (l = conflict_list; l != NULL; l = l->next) {
+ for (l1 = new_names, l2 = dialog->selection; l1 != NULL, l2 != NULL; l1 = l1->next, l2 = l2->next) {
+ file = NAUTILUS_FILE (l2->data);
- row = create_row_for_label ((gchar*) l->data, l == conflict_list);
+ row = create_row_for_label (l1->data, nautilus_file_get_name (file), TRUE);
gtk_container_add (GTK_CONTAINER (dialog->conflict_listbox), row);
@@ -252,7 +281,7 @@ file_names_widget_entry_on_changed (NautilusBatchRename *dialog)
}
/* add name conflicts to the listbox */
- fill_conflict_listbox (dialog, duplicates);
+ //fill_conflict_listbox (dialog, duplicates);
g_free (file_name);
@@ -268,29 +297,8 @@ file_names_widget_entry_on_changed (NautilusBatchRename *dialog)
//gtk_stack_set_visible_child (GTK_STACK (dialog->label_stack), GTK_WIDGET
(dialog->error_label));
}
- /* Update label that shows an example of the renaming result */
- file = NAUTILUS_FILE (dialog->selection->data);
-
- file_name = g_strdup (nautilus_file_get_name (file));
- if (dialog->mode == NAUTILUS_BATCH_RENAME_REPLACE) {
- entry_text = g_strdup (gtk_entry_get_text (GTK_ENTRY (dialog->find_entry)));
- }
- else
- entry_text = g_strdup (gtk_entry_get_text (GTK_ENTRY (dialog->name_entry)));
-
- replace_text = g_strdup (gtk_entry_get_text (GTK_ENTRY (dialog->replace_entry)));
-
- if (entry_text == NULL) {
-
- gtk_label_set_label (GTK_LABEL (dialog->error_label), file_name);
- g_free (file_name);
-
- return;
- }
-
- display_text = get_new_display_name (dialog->mode, file_name, entry_text, replace_text);
-
- gtk_label_set_label (GTK_LABEL (dialog->error_label), display_text);
+ /* Update listbox that shows the result of the renaming for each file */
+ fill_display_listbox (dialog, new_names);
g_list_free (new_names);
g_list_free (duplicates);
@@ -328,11 +336,18 @@ batch_rename_mode_changed (NautilusBatchRename *dialog)
if (gtk_toggle_button_get_active (GTK_TOGGLE_BUTTON (dialog->format_mode_button))) {
gtk_stack_set_visible_child_name (dialog->mode_stack, "format");
- dialog->mode = NAUTILUS_BATCH_RENAME_FORMAT;
+ dialog->mode = NAUTILUS_BATCH_RENAME_PREPEND;
+
+ gtk_entry_set_text (GTK_ENTRY (dialog->name_entry),
+ gtk_entry_get_text (dialog->find_entry));
+
} else {
gtk_stack_set_visible_child_name (dialog->mode_stack, "replace");
dialog->mode = NAUTILUS_BATCH_RENAME_REPLACE;
+
+ gtk_entry_set_text (GTK_ENTRY (dialog->find_entry),
+ gtk_entry_get_text (dialog->name_entry));
}
/* update display text */
@@ -341,6 +356,36 @@ batch_rename_mode_changed (NautilusBatchRename *dialog)
}
static void
+add_button_clicked (NautilusBatchRename *dialog)
+{
+ if (gtk_widget_is_visible (dialog->add_popover))
+ gtk_widget_set_visible (dialog->add_popover, FALSE);
+ else
+ gtk_widget_set_visible (dialog->add_popover, TRUE);
+}
+
+static void
+numbering_order_button_clicked (NautilusBatchRename *dialog)
+{
+ if (gtk_widget_is_visible (dialog->numbering_order_popover))
+ gtk_widget_set_visible (dialog->numbering_order_popover, FALSE);
+ else
+ gtk_widget_set_visible (dialog->numbering_order_popover, TRUE);
+}
+
+static void
+add_popover_closed (NautilusBatchRename *dialog)
+{
+ gtk_toggle_button_set_active (GTK_TOGGLE_BUTTON (dialog->add_button), FALSE);
+}
+
+static void
+numbering_order_popover_closed (NautilusBatchRename *dialog)
+{
+ gtk_toggle_button_set_active (GTK_TOGGLE_BUTTON (dialog->numbering_order_button), FALSE);
+}
+
+static void
nautilus_batch_rename_class_init (NautilusBatchRenameClass *klass)
{
GtkDialogClass *dialog_class = GTK_DIALOG_CLASS (klass);
@@ -362,11 +407,19 @@ nautilus_batch_rename_class_init (NautilusBatchRenameClass *klass)
gtk_widget_class_bind_template_child (widget_class, NautilusBatchRename, mode_stack);
gtk_widget_class_bind_template_child (widget_class, NautilusBatchRename, replace_mode_button);
gtk_widget_class_bind_template_child (widget_class, NautilusBatchRename, format_mode_button);
+ gtk_widget_class_bind_template_child (widget_class, NautilusBatchRename, add_button);
+ gtk_widget_class_bind_template_child (widget_class, NautilusBatchRename, add_popover);
+ gtk_widget_class_bind_template_child (widget_class, NautilusBatchRename, numbering_order_popover);
+ gtk_widget_class_bind_template_child (widget_class, NautilusBatchRename, numbering_order_button);
gtk_widget_class_bind_template_callback (widget_class, file_names_widget_entry_on_changed);
gtk_widget_class_bind_template_callback (widget_class, batch_rename_dialog_on_closed);
gtk_widget_class_bind_template_callback (widget_class, file_names_widget_on_activate);
gtk_widget_class_bind_template_callback (widget_class, batch_rename_mode_changed);
+ gtk_widget_class_bind_template_callback (widget_class, add_button_clicked);
+ gtk_widget_class_bind_template_callback (widget_class, add_popover_closed);
+ gtk_widget_class_bind_template_callback (widget_class, numbering_order_popover_closed);
+ gtk_widget_class_bind_template_callback (widget_class, numbering_order_button_clicked);
}
GtkWidget*
diff --git a/src/resources/ui/nautilus-batch-rename-dialog.ui
b/src/resources/ui/nautilus-batch-rename-dialog.ui
index e497d50..8fd3c92 100644
--- a/src/resources/ui/nautilus-batch-rename-dialog.ui
+++ b/src/resources/ui/nautilus-batch-rename-dialog.ui
@@ -78,7 +78,7 @@
<packing>
<property name="left-attach">3</property>
<property name="top-attach">0</property>
- <property name="width">1</property>
+ <property name="width">1</property>
</packing>
</child>
<child>
@@ -86,7 +86,7 @@
<property name="visible">True</property>
<property name="can_focus">False</property>
<property name="vhomogeneous">False</property>
- <property name="hhomogeneous">False</property>
+ <property name="hhomogeneous">True</property>
<property name="transition_type">crossfade</property>
<property name="transition_duration">100</property>
<child>
@@ -96,7 +96,7 @@
<property name="margin-right">40</property>
<property name="margin-top">0</property>
<property name="margin-bottom">10</property>
- <property name="row-spacing">6</property>
+ <property name="row-spacing">15</property>
<property name="column-spacing">6</property>
<child>
<object class="GtkBox">
@@ -116,16 +116,10 @@
</object>
</child>
<child>
- <object class="GtkPopover" id="add_popover">
+ <object class="GtkToggleButton" id="add_button">
<property name="visible">True</property>
- <child>
- <object class="GtkLabel" id="_label">
- <property name="visible">True</property>
- <property name="label" translatable="yes">Replace With</property>
- <property name="can_focus">False</property>
- <property name="sensitive">False</property>
- </object>
- </child>
+ <property name="label" translatable="yes">+ Add</property>
+ <signal name="toggled" handler="add_button_clicked" swapped="yes" />
</object>
</child>
</object>
@@ -135,6 +129,37 @@
<property name="width">5</property>
</packing>
</child>
+ <child>
+ <object class="GtkLabel" id="numbering_order_label">
+ <property name="visible">True</property>
+ <property name="label" translatable="yes">Automatic Numbering Order</property>
+ <property name="can_focus">False</property>
+ <property name="sensitive">False</property>
+ </object>
+ <packing>
+ <property name="left-attach">0</property>
+ <property name="top-attach">1</property>
+ <property name="width">1</property>
+ </packing>
+ </child>
+ <child>
+ <object class="GtkToggleButton" id="numbering_order_button">
+ <property name="visible">True</property>
+ <signal name="toggled" handler="numbering_order_button_clicked" swapped="yes" />
+ <child>
+ <object class="GtkLabel" id="numering_order">
+ <property name="visible">True</property>
+ <property name="label" translatable="yes">Original name (Ascending)</property>
+ <property name="can_focus">False</property>
+ </object>
+ </child>
+ </object>
+ <packing>
+ <property name="left-attach">4</property>
+ <property name="top-attach">1</property>
+ <property name="width">1</property>
+ </packing>
+ </child>
</object>
<packing>
<property name="name">format</property>
@@ -148,7 +173,7 @@
<property name="margin-right">40</property>
<property name="margin-top">0</property>
<property name="margin-bottom">10</property>
- <property name="row-spacing">6</property>
+ <property name="row-spacing">15</property>
<property name="column-spacing">6</property>
<child>
<object class="GtkLabel" id="existing_text_label">
@@ -264,4 +289,30 @@
<property name="step_increment">1</property>
<property name="page_increment">10</property>
</object>
+ <object class="GtkPopover" id="add_popover">
+ <property name="relative-to">add_button</property>
+ <property name="position">bottom</property>
+ <signal name="closed" handler="add_popover_closed" swapped="yes" />
+ <child>
+ <object class="GtkLabel">
+ <property name="visible">True</property>
+ <property name="label" translatable="yes">label for testing purpose</property>
+ <property name="can_focus">False</property>
+ <property name="sensitive">True</property>
+ </object>
+ </child>
+ </object>
+ <object class="GtkPopover" id="numbering_order_popover">
+ <property name="relative-to">numbering_order_button</property>
+ <property name="position">bottom</property>
+ <signal name="closed" handler="numbering_order_popover_closed" swapped="yes" />
+ <child>
+ <object class="GtkLabel">
+ <property name="visible">True</property>
+ <property name="label" translatable="yes">some other ordering option</property>
+ <property name="can_focus">False</property>
+ <property name="sensitive">True</property>
+ </object>
+ </child>
+ </object>
</interface>
\ No newline at end of file
[
Date Prev][
Date Next] [
Thread Prev][
Thread Next]
[
Thread Index]
[
Date Index]
[
Author Index]
