Fixed duplicated bookmarks after overwrite

CHANGELOG.md

@@ -29,6 +29,7 @@ Released on FIXME: date
   - SCP file transfer: prevent infinite loops while performing `stat` on symbolic links pointing to themselves (e.g. `mylink -> mylink`)
   - Fixed a bug causing termscp to crash if removing a bookmark
   - Fixed file format cursor position in the GUI
+  - Fixed a bug causing termscp to show two equal bookmarks when overwriting one.
 - Dependencies:
   - Removed `unicode-width`
 - For developers:

src/ui/activities/auth_activity/bookmarks.rs

@@ -94,6 +94,8 @@ impl AuthActivity {
             bookmarks_cli.add_bookmark(name.clone(), address, port, protocol, username, password);
             // Save bookmarks
             self.write_bookmarks();
+            // Remove `name` from bookmarks if exists
+            self.bookmarks_list.retain(|b| b.as_str() != name.as_str());
             // Push bookmark to list and sort
             self.bookmarks_list.push(name);
             self.sort_bookmarks();