diff options
| author | Carlos Soriano <csoriano@gnome.org> | 2016-09-13 22:45:46 +0200 |
|---|---|---|
| committer | Carlos Soriano <csoriano@gnome.org> | 2016-09-13 23:08:52 +0200 |
| commit | 7fb5aa77c904905e9af7e34b5f3a1dbd4dc66b4f (patch) | |
| tree | d2753918526bf6debf3ceb3a234b699fff1ecc15 /src/resources/ui/nautilus-batch-rename-dialog.ui | |
| parent | 1d397ba03d6063989df5edbae23f8a568ac2e91a (diff) | |
| download | nautilus-7fb5aa77c904905e9af7e34b5f3a1dbd4dc66b4f.tar.gz | |
batch-rename-dialog: use positions for tag handling
When a tag is added, deleted or moved or a text is changed, inserted
or deleted, we need to do update the text and tags already present
based on what changed.
We were checking whether a tag was present or not matching with the
text rather than deterministically adding, removing or updating the
positions based on the direct user interaction.
Doing it in this way is a potential problem since two tags added
writing manually would confuse the underlying handling, apart of being
harder to have a consistent state of the tags like deleting multiple
tags at once, deleting tags with text on the sides when selected,
replacing tags with text, crashes when using overwrite mode instead of
insertion mode, or crashes when using modifiers. Apart of making the
handling more complex.
This patch refactors the tag handling for using positions and direct
manipulation of them instead of text.
https://bugzilla.gnome.org/show_bug.cgi?id=771069
Diffstat (limited to 'src/resources/ui/nautilus-batch-rename-dialog.ui')
| -rw-r--r-- | src/resources/ui/nautilus-batch-rename-dialog.ui | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/src/resources/ui/nautilus-batch-rename-dialog.ui b/src/resources/ui/nautilus-batch-rename-dialog.ui index 4d4a6ec14..d4bef6dc9 100644 --- a/src/resources/ui/nautilus-batch-rename-dialog.ui +++ b/src/resources/ui/nautilus-batch-rename-dialog.ui @@ -110,9 +110,9 @@ <property name="width_request">400</property> <property name="hexpand">True</property> <property name="activates-default">True</property> - <signal name="changed" handler="file_names_widget_entry_on_changed" swapped="yes" /> <signal name="activate" handler="file_names_widget_on_activate" swapped="yes" /> - <signal name="key-press-event" handler="on_key_press_event" swapped="no"/> + <signal name="insert-text" handler="on_insert_text" swapped="no" /> + <signal name="delete-text" handler="on_delete_text" swapped="no" /> </object> </child> <child> |