From c9d0717e6a9958d7699ddb86bc18bb4d9482033b Mon Sep 17 00:00:00 2001
From: Bleak Grey <bleakgrey@gmail.com>
Date: Sun, 2 Aug 2020 00:47:22 +0300
Subject: [PATCH] Reimplement Conversations

---
 data/ui/widgets/status.ui       | 70 ++++++++++++++++++++++-----------
 meson.build                     |  1 +
 src/API/Conversation.vala       | 26 +++++++++++-
 src/Views/Conversations.vala    | 11 +++---
 src/Widgets/Attachment/Box.vala | 31 ---------------
 src/Widgets/Conversation.vala   | 53 +++++++++++++++++++++++++
 src/Widgets/Status.vala         | 37 +++++++++++------
 7 files changed, 156 insertions(+), 73 deletions(-)
 create mode 100644 src/Widgets/Conversation.vala

diff --git a/data/ui/widgets/status.ui b/data/ui/widgets/status.ui
index 22c7512..d677d58 100644
--- a/data/ui/widgets/status.ui
+++ b/data/ui/widgets/status.ui
@@ -24,7 +24,7 @@
   </object>
   <template class="TootleWidgetsStatus" parent="GtkListBoxRow">
     <property name="visible">True</property>
-    <property name="activatable">True</property>
+    <property name="can_focus">False</property>
     <child>
       <object class="GtkGrid" id="grid">
         <property name="visible">True</property>
@@ -212,7 +212,6 @@
           <packing>
             <property name="left_attach">0</property>
             <property name="top_attach">1</property>
-            <property name="height">1</property>
           </packing>
         </child>
         <child>
@@ -252,7 +251,6 @@
             <property name="visible">True</property>
             <property name="can_focus">False</property>
             <property name="vexpand">True</property>
-            <property name="valign">fill</property>
             <property name="row_homogeneous">True</property>
             <child>
               <object class="TootleWidgetsRichLabel" id="name_label">
@@ -265,7 +263,7 @@
               <packing>
                 <property name="left_attach">0</property>
                 <property name="top_attach">0</property>
-                <property name="width">2</property>
+                <property name="width">3</property>
               </packing>
             </child>
             <child>
@@ -283,33 +281,57 @@
               </packing>
             </child>
             <child>
-              <object class="GtkImage" id="pin_indicator">
+              <object class="GtkBox" id="indicators">
                 <property name="visible">True</property>
                 <property name="can_focus">False</property>
-                <property name="margin_left">8</property>
+                <property name="halign">end</property>
+                <property name="valign">start</property>
                 <property name="margin_start">8</property>
-                <property name="icon_name">view-pin-symbolic</property>
-                <property name="icon_size">1</property>
+                <property name="spacing">8</property>
+                <child>
+                  <object class="GtkImage" id="pin_indicator">
+                    <property name="can_focus">False</property>
+                    <property name="icon_name">view-pin-symbolic</property>
+                    <property name="icon_size">1</property>
+                    <property name="opacity">0.5</property>
+                  </object>
+                  <packing>
+                    <property name="expand">False</property>
+                    <property name="fill">True</property>
+                    <property name="position">0</property>
+                  </packing>
+                </child>
+                <child>
+                  <object class="GtkImage" id="indicator">
+                    <property name="can_focus">False</property>
+                    <property name="icon_name">unknown</property>
+                    <property name="icon_size">1</property>
+                    <property name="opacity">0.5</property>
+                  </object>
+                  <packing>
+                    <property name="expand">False</property>
+                    <property name="fill">True</property>
+                    <property name="position">1</property>
+                  </packing>
+                </child>
+                <child>
+                  <object class="TootleWidgetsRichLabel" id="date_label">
+                    <property name="visible">True</property>
+                    <property name="can_focus">False</property>
+                    <property name="label" translatable="no">Yesterday</property>
+                    <property name="opacity">0.5</property>
+                  </object>
+                  <packing>
+                    <property name="expand">False</property>
+                    <property name="fill">True</property>
+                    <property name="position">2</property>
+                  </packing>
+                </child>
               </object>
               <packing>
                 <property name="left_attach">2</property>
                 <property name="top_attach">0</property>
-              </packing>
-            </child>
-            <child>
-              <object class="TootleWidgetsRichLabel" id="date_label">
-                <property name="visible">True</property>
-                <property name="can_focus">False</property>
-                <property name="opacity">0.5</property>
-                <property name="margin_left">8</property>
-                <property name="margin_start">8</property>
-                <property name="label" translatable="no">Yesterday</property>
-                <property name="halign">end</property>
-                <property name="valign">start</property>
-              </object>
-              <packing>
-                <property name="left_attach">3</property>
-                <property name="top_attach">0</property>
+                <property name="width">2</property>
               </packing>
             </child>
           </object>
diff --git a/meson.build b/meson.build
index 13462b8..e471d06 100644
--- a/meson.build
+++ b/meson.build
@@ -79,6 +79,7 @@ executable(
     'src/Widgets/RichLabel.vala',
     'src/Widgets/Status.vala',
     'src/Widgets/Notification.vala',
+    'src/Widgets/Conversation.vala',
     'src/Widgets/VisibilityPopover.vala',
     'src/Widgets/Attachment/Box.vala',
     'src/Widgets/Attachment/Slot.vala',
diff --git a/src/API/Conversation.vala b/src/API/Conversation.vala
index 8230560..713de5e 100644
--- a/src/API/Conversation.vala
+++ b/src/API/Conversation.vala
@@ -1,6 +1,30 @@
 public class Tootle.API.Conversation : Entity, Widgetizable {
 
-	public string id { get; construct set; }
+	public string id { get; set; }
+	public Gee.ArrayList<API.Account> accounts { get; set; }
 	public bool unread { get; set; default = false; }
+	public API.Status? last_status { get; set; default = null; }
+
+    public override Gtk.Widget to_widget () {
+        return new Widgets.Conversation (this);
+    }
+
+	public override void open () {
+		var view = new Views.ExpandedStatus (last_status.formal);
+		window.open_view (view);
+
+		if (unread)
+			mark_read ();
+	}
+
+	public void mark_read () {
+		new Request.POST (@"/api/v1/conversations/$id/read")
+			.with_account (Tootle.accounts.active)
+			.then (() => {
+				unread = false;
+			})
+			.on_error (() => {})
+			.exec ();
+	}
 
 }
diff --git a/src/Views/Conversations.vala b/src/Views/Conversations.vala
index e14247e..c81aab8 100644
--- a/src/Views/Conversations.vala
+++ b/src/Views/Conversations.vala
@@ -4,13 +4,14 @@ public class Tootle.Views.Conversations : Views.Timeline {
         Object (
             url: "/api/v1/conversations",
             label: _("Conversations"),
-            icon: API.Visibility.DIRECT.get_icon (),
-            accepts: typeof (API.Conversation)
+            icon: API.Visibility.DIRECT.get_icon ()
         );
+        accepts = typeof (API.Conversation);
     }
 
-    public override string? get_stream_url () {
-        return @"/api/v1/streaming/?stream=direct&access_token=$(account.access_token)";
-    }
+	// TODO: Reload when an update is received
+    // public override string? get_stream_url () {
+    //     return @"/api/v1/streaming/?stream=direct&access_token=$(account.access_token)";
+    // }
 
 }
diff --git a/src/Widgets/Attachment/Box.vala b/src/Widgets/Attachment/Box.vala
index 2bee445..8665dd6 100644
--- a/src/Widgets/Attachment/Box.vala
+++ b/src/Widgets/Attachment/Box.vala
@@ -17,37 +17,6 @@ public class Tootle.Widgets.Attachment.Box : FlowBox {
 		Object (editing: editing);
 	}
 
-    //TODO: Upload attachments in Compose dialog
-    public void select () {
-        var filter = new Gtk.FileFilter ();
-        filter.add_mime_type ("image/jpeg");
-        filter.add_mime_type ("image/png");
-        filter.add_mime_type ("image/gif");
-        filter.add_mime_type ("video/webm");
-        filter.add_mime_type ("video/mp4");
-
-        var chooser = new Gtk.FileChooserDialog (
-            _("Select media files to add"),
-            null,
-            Gtk.FileChooserAction.OPEN,
-            _("_Cancel"),
-            Gtk.ResponseType.CANCEL,
-            _("_Open"),
-            Gtk.ResponseType.ACCEPT);
-
-        chooser.select_multiple = true;
-        chooser.set_filter (filter);
-
-        if (chooser.run () == ResponseType.ACCEPT) {
-            show ();
-            foreach (unowned string uri in chooser.get_uris ()) {
-                //var widget = new ImageAttachment.upload (uri);
-                //append_widget (widget);
-            }
-        }
-        chooser.close ();
-    }
-
     public bool populate (ArrayList<API.Attachment>? list) {
         if (list == null)
             return false;
diff --git a/src/Widgets/Conversation.vala b/src/Widgets/Conversation.vala
new file mode 100644
index 0000000..76ffb45
--- /dev/null
+++ b/src/Widgets/Conversation.vala
@@ -0,0 +1,53 @@
+using Gtk;
+
+public class Tootle.Widgets.Conversation : Widgets.Status {
+
+	public API.Conversation conversation { get; construct set; }
+
+	public Conversation (API.Conversation entity) {
+		Object (conversation: entity, status: entity.last_status);
+		conversation.bind_property ("unread", this.indicator, "visible", BindingFlags.SYNC_CREATE);
+		this.indicators.child_set_property (this.indicator, "position", 2);
+		this.indicator.opacity = 1;
+		this.indicator.icon_name = Desktop.fallback_icon (
+			"software-update-urgent-symbolic",
+			"dialog-warning-symbolic");
+		this.actions.destroy ();
+	}
+
+	public new string title_text {
+		owned get {
+			var label = "";
+			foreach (API.Account account in conversation.accounts) {
+				label += "<b>" + Html.simplify (account.display_name) + "</b>";
+				if (conversation.accounts.last () != account)
+					label += ", ";
+			}
+			return @"$label";
+		}
+	}
+
+	public new string subtitle_text {
+		owned get {
+			var label = "";
+			foreach (API.Account account in conversation.accounts) {
+				label += account.handle + " ";
+			}
+			return @"<small>$label</small>";
+		}
+	}
+
+	public new string? avatar_url {
+		owned get {
+			if (conversation.accounts.size > 1)
+				return null;
+			else
+				return conversation.accounts.get (0).avatar;
+		}
+	}
+
+	public override void on_open () {
+		conversation.open ();
+	}
+
+}
diff --git a/src/Widgets/Status.vala b/src/Widgets/Status.vala
index 208850d..7b55dbc 100644
--- a/src/Widgets/Status.vala
+++ b/src/Widgets/Status.vala
@@ -22,10 +22,14 @@ public class Tootle.Widgets.Status : ListBoxRow {
 	[GtkChild]
 	protected Widgets.RichLabel handle_label;
 	[GtkChild]
+	protected Box indicators;
+	[GtkChild]
 	protected Widgets.RichLabel date_label;
 	[GtkChild]
 	protected Image pin_indicator;
 	[GtkChild]
+	protected Image indicator;
+	[GtkChild]
 	public Revealer revealer;
 	[GtkChild]
 	protected Widgets.RichLabel content;
@@ -68,13 +72,6 @@ public class Tootle.Widgets.Status : ListBoxRow {
 		}
 	}
 
-	protected string display_name {
-		owned get {
-			var name = Html.simplify (status.formal.account.display_name);
-			return @"<b>$name</b>";
-		}
-	}
-
 	protected string date {
 		owned get {
 			var date = new GLib.DateTime.from_iso8601 (status.formal.created_at, null);
@@ -83,13 +80,28 @@ public class Tootle.Widgets.Status : ListBoxRow {
 		}
 	}
 
-	protected string handle {
+	public string title_text {
+		owned get {
+			var name = Html.simplify (status.formal.account.display_name);
+			return @"<b>$name</b>";
+		}
+	}
+
+	public string subtitle_text {
 		owned get {
 			return @"<small>$(status.formal.account.handle)</small>";
 		}
 	}
 
-	public virtual signal void open () {
+	public string? avatar_url {
+		owned get {
+			return status.formal.account.avatar;
+		}
+	}
+
+	public signal void open ();
+
+	public virtual void on_open () {
 		if (status.id == "")
 			on_avatar_clicked ();
 		else
@@ -99,6 +111,7 @@ public class Tootle.Widgets.Status : ListBoxRow {
 	construct {
 		content.activate_link.connect (on_toggle_spoiler);
 		notify["kind"].connect (on_kind_changed);
+		open.connect (on_open);
 
 		if (kind == null) {
 			if (status.reblog != null)
@@ -124,11 +137,11 @@ public class Tootle.Widgets.Status : ListBoxRow {
 
 		bind_property ("escaped-spoiler", content, "text", BindingFlags.SYNC_CREATE);
 		bind_property ("escaped-content", revealer_content, "text", BindingFlags.SYNC_CREATE);
-		status.formal.account.bind_property ("avatar", avatar, "url", BindingFlags.SYNC_CREATE);
-		bind_property ("handle", handle_label, "label", BindingFlags.SYNC_CREATE);
-		bind_property ("display_name", name_label, "text", BindingFlags.SYNC_CREATE);
+		bind_property ("title_text", name_label, "text", BindingFlags.SYNC_CREATE);
+		bind_property ("subtitle_text", handle_label, "text", BindingFlags.SYNC_CREATE);
 		bind_property ("date", date_label, "label", BindingFlags.SYNC_CREATE);
 		status.formal.bind_property ("pinned", pin_indicator, "visible", BindingFlags.SYNC_CREATE);
+		bind_property ("avatar_url", avatar, "url", BindingFlags.SYNC_CREATE);
 
 		status.formal.bind_property ("has_spoiler", revealer_content, "visible", BindingFlags.SYNC_CREATE);
 		revealer.reveal_child = !status.formal.has_spoiler;