Simplify talking to yourself (notes) on the UI

- clicking on the menu will multiple accounts will bring up a modified
  "start chat" windows

3 files changed, 35 insertions, 21 deletions

diff --git a/main/src/ui/add_conversation/roster_list.vala b/main/src/ui/add_conversation/roster_list.vala
index bb338ce5..ebbe55e7 100644
--- a/main/src/ui/add_conversation/roster_list.vala
+++ b/main/src/ui/add_conversation/roster_list.vala
@@ -16,37 +16,37 @@ protected class RosterList {
     private ListBox list_box = new ListBox();
     private HashMap<Account, HashMap<Jid, ListBoxRow>> rows = new HashMap<Account, HashMap<Jid, ListBoxRow>>(Account.hash_func, Account.equals_func);
 
-    public RosterList(StreamInteractor stream_interactor, Gee.List<Account> accounts) {
+    public RosterList(StreamInteractor stream_interactor, Gee.List<Account> accounts, bool notes_mode = false) {
         this.stream_interactor = stream_interactor;
         this.accounts = accounts;
 
-        handler_ids += stream_interactor.get_module(RosterManager.IDENTITY).removed_roster_item.connect( (account, jid, roster_item) => {
+        handler_ids += stream_interactor.get_module(RosterManager.IDENTITY).removed_roster_item.connect( (account, jid) => {
             if (accounts.contains(account)) {
-                on_removed_roster_item(account, jid, roster_item);
+                remove_row(account, jid);
             }
         });
-        handler_ids += stream_interactor.get_module(RosterManager.IDENTITY).updated_roster_item.connect( (account, jid, roster_item) => {
+        handler_ids += stream_interactor.get_module(RosterManager.IDENTITY).updated_roster_item.connect( (account, jid) => {
             if (accounts.contains(account)) {
-                on_updated_roster_item(account, jid, roster_item);
+                update_row(account, jid);
             }
         });
         list_box.destroy.connect(() => {
             foreach (ulong handler_id in handler_ids) stream_interactor.get_module(RosterManager.IDENTITY).disconnect(handler_id);
         });
 
-        foreach (Account a in accounts) fetch_roster_items(a);
+        foreach (Account a in accounts) fetch_roster_items(a, notes_mode);
     }
 
-    private void on_removed_roster_item(Account account, Jid jid, Roster.Item roster_item) {
+    private void remove_row(Account account, Jid jid) {
         if (rows.has_key(account) && rows[account].has_key(jid)) {
             list_box.remove(rows[account][jid]);
             rows[account].unset(jid);
         }
     }
 
-    private void on_updated_roster_item(Account account, Jid jid, Roster.Item roster_item) {
-        on_removed_roster_item(account, jid, roster_item);
-        ListRow row = new ListRow.from_jid(stream_interactor, roster_item.jid, account, accounts.size > 1);
+    private void update_row(Account account, Jid jid) {
+        remove_row(account, jid);
+        ListRow row = new ListRow.from_jid(stream_interactor, jid, account, accounts.size > 1);
         ListBoxRow list_box_row = new ListBoxRow() { child=row };
         rows[account][jid] = list_box_row;
         list_box.append(list_box_row);
@@ -54,10 +54,17 @@ protected class RosterList {
         list_box.invalidate_filter();
     }
 
-    private void fetch_roster_items(Account account) {
-        rows[account] = new HashMap<Jid, ListBoxRow>(Jid.hash_func, Jid.equals_func);
-        foreach (Roster.Item roster_item in stream_interactor.get_module(RosterManager.IDENTITY).get_roster(account)) {
-            on_updated_roster_item(account, roster_item.jid, roster_item);
+    private void fetch_roster_items(Account account, bool notes_mode = false) {
+        if (!notes_mode){
+          rows[account] = new HashMap<Jid, ListBoxRow>(Jid.hash_func, Jid.equals_func);
+          foreach (Roster.Item roster_item in stream_interactor.get_module(RosterManager.IDENTITY).get_roster(account)) {
+            update_row(account, roster_item.jid);
+          }
+        }
+        else {
+          ListRow own_account_row = new ListRow.from_jid(stream_interactor, account.bare_jid, account, accounts.size > 1);
+          ListBoxRow own_account_lbrow = new ListBoxRow() { child=own_account_row };
+          list_box.append(own_account_lbrow);
         }
     }
 
diff --git a/main/src/ui/add_conversation/select_contact_dialog.vala b/main/src/ui/add_conversation/select_contact_dialog.vala
index fb69257e..d02a366f 100644
--- a/main/src/ui/add_conversation/select_contact_dialog.vala
+++ b/main/src/ui/add_conversation/select_contact_dialog.vala
@@ -18,8 +18,9 @@ public class SelectContactDialog : Gtk.Dialog {
     private SelectJidFragment select_jid_fragment;
     private StreamInteractor stream_interactor;
     private Gee.List<Account> accounts;
+    private bool notes_mode;
 
-    public SelectContactDialog(StreamInteractor stream_interactor, Gee.List<Account> accounts) {
+    public SelectContactDialog(StreamInteractor stream_interactor, Gee.List<Account> accounts, bool notes_mode = false) {
         Object(use_header_bar : Util.use_csd() ? 1 : 0);
         modal = true;
         this.default_width = 460;
@@ -27,6 +28,7 @@ public class SelectContactDialog : Gtk.Dialog {
 
         this.stream_interactor = stream_interactor;
         this.accounts = accounts;
+        this.notes_mode = notes_mode;
 
         setup_view();
         setup_headerbar();
@@ -72,10 +74,10 @@ public class SelectContactDialog : Gtk.Dialog {
     }
 
     private void setup_view() {
-        roster_list = new RosterList(stream_interactor, accounts);
+        roster_list = new RosterList(stream_interactor, accounts, notes_mode);
         roster_list_box = roster_list.get_list_box();
         roster_list_box.row_activated.connect(() => { ok_button.clicked(); });
-        select_jid_fragment = new SelectJidFragment(stream_interactor, roster_list_box, accounts);
+        select_jid_fragment = new SelectJidFragment(stream_interactor, roster_list_box, accounts, notes_mode);
         select_jid_fragment.add_jid.connect((row) => {
             AddContactDialog add_contact_dialog = new AddContactDialog(stream_interactor);
             add_contact_dialog.set_transient_for(this);
@@ -96,12 +98,13 @@ public class AddChatDialog : SelectContactDialog {
 
     public signal void added(Conversation conversation);
 
-    public AddChatDialog(StreamInteractor stream_interactor, Gee.List<Account> accounts) {
-        base(stream_interactor, accounts);
-        title = _("Start Conversation");
+    public AddChatDialog(StreamInteractor stream_interactor, Gee.List<Account> accounts, bool notes_mode = false) {
+        base(stream_interactor, accounts, notes_mode);
+        title = notes_mode ? _("Open notes") : _("Start Conversation");
         ok_button.label = _("Start");
         selected.connect((account, jid) => {
             Conversation conversation = stream_interactor.get_module(ConversationManager.IDENTITY).create_conversation(jid, account, Conversation.Type.CHAT);
+            if(notes_mode) conversation.pinned = 1;
             stream_interactor.get_module(ConversationManager.IDENTITY).start_conversation(conversation);
             added(conversation);
         });
diff --git a/main/src/ui/add_conversation/select_jid_fragment.vala b/main/src/ui/add_conversation/select_jid_fragment.vala
index e0682e29..7a1dc747 100644
--- a/main/src/ui/add_conversation/select_jid_fragment.vala
+++ b/main/src/ui/add_conversation/select_jid_fragment.vala
@@ -28,11 +28,15 @@ public class SelectJidFragment : Gtk.Box {
     private ListBox list;
     private string[]? filter_values;
 
-    public SelectJidFragment(StreamInteractor stream_interactor, ListBox list, Gee.List<Account> accounts) {
+    public SelectJidFragment(StreamInteractor stream_interactor, ListBox list, Gee.List<Account> accounts, bool notes_mode = false) {
         this.stream_interactor = stream_interactor;
         this.list = list;
         this.accounts = accounts;
 
+        if (notes_mode) {
+            add_button.visible = false;
+            remove_button.visible = false;
+        }
         list.activate_on_single_click = false;
         list.vexpand = true;
         box.append(list);