[ post editing ] editing kinda works.

Author: soapdog

lib/depject/message/html/actions.js

@@ -23,15 +23,17 @@ exports.gives = nest("message.html.actions");
 exports.create = (api) => {
   const i18n = api.intl.sync.i18n;
 
-  return nest("message.html.actions", function like(msg, opts) {
+  return nest("message.html.actions", function like(msg, opts = {}) {
     const liked = api.message.obs.doesLike(msg.key);
 
     const customActions = [];
 
-    if (typeof opts === undefined || !opts?.onEdited) {
-      onEdited = (msg) => {
-        console.log(`message edited`, msg);
-      };
+    let onEdited = (msg) => {
+      console.log(`message edited`, msg);
+    };
+
+    if (Object.hasOwn(opts, "onEdited")) {
+      onEdited = opts.onEdited;
     }
 
     if (isFeatureEnabled("custom-scripts")) {

lib/depject/message/html/layout/default.js

@@ -3,6 +3,7 @@ const nest = require("depnest");
 const ref = require("ssb-ref");
 const ExpanderHook = require("../../../../expander-hook");
 const timestamp = require("../../../../message/html/timestamp");
+const crypto = require("crypto");
 
 exports.needs = nest({
   "profile.html.person": "first",
@@ -57,19 +58,20 @@ exports.create = function (api) {
     const needsExpand = Value(false);
     const expanded = Value(false);
     const contentToShow = Value(content);
+    const hasEdits = Value(false);
 
-    let edits = Value(false);
+    let edits = MutantArray([]);
+    let showing = Value(-1);
 
     if (msg?.value?.content?.type === "post" && msg?.key) {
-      edits = api.message.obs.edits(msg.key);
+      const newEdits = api.message.obs.edits(msg.key);
+      newEdits((es) => {
+        edits.set(es);
+        hasEdits.set(true);
+      });
     }
 
-    onEdited = (msg) => {
-      // edits = api.message.obs.edits(msg.key);
-      const md = api.message.html.markdown(msg.value.content.text);
-      contentToShow.set(md);
-      console.log("edited", md);
-    };
+    let uuid = crypto.randomUUID();
 
     // new message previews shouldn't contract
     if (!msg.key) expanded.set(true);
@@ -133,6 +135,7 @@ exports.create = function (api) {
       h("section.content", {
         classList: [when(expanded, "-expanded")],
         hooks: [ExpanderHook(needsExpand)],
+        id: uuid,
       }, contentToShow),
       computed(msg.key, (key) => {
         if (ref.isMsg(key)) {
@@ -149,7 +152,16 @@ exports.create = function (api) {
               ]),
             ),
             h("div.actions", [
-              api.message.html.actions(msg, { onEdited }),
+              api.message.html.actions(msg, {
+                onEdited: (m) => {
+                  // edits = api.message.obs.edits(msg.key);
+                  console.log(edits());
+                  edits.push(m);
+                  showing.set(edits.getLength() - 1);
+                  console.log(edits());
+                  console.log(showing());
+                },
+              }),
             ]),
           ]);
         }
@@ -161,7 +173,6 @@ exports.create = function (api) {
     // scoped
 
     function editControl() {
-      let showing = Value(-1);
       edits((es) => {
         showing.set(es.length - 1);
       });
@@ -252,7 +263,7 @@ exports.create = function (api) {
         h("div.meta", [
           additionalMeta,
           api.message.html.metas(msg),
-          when(edits, editControl()),
+          when(hasEdits, editControl()),
         ]),
       ]);
     }

lib/depject/message/obs/edits.js

@@ -10,7 +10,7 @@ exports.gives = nest("message.obs.edits");
 exports.create = function (api) {
   return nest("message.obs.edits", function (key, hintMessage = null) {
     if (!ref.isMsg(key)) throw new Error("a msg id must be specified");
-    const edited = Value(false);
+    const edited = MutantArray([]);
 
     onceTrue(api.sbot.obs.connection, (sbot) => {
       pull(

lib/depject/sheet/editPost.js

@@ -118,8 +118,6 @@ exports.create = function (api) {
           revisionBranch: revisionBranch(),
         };
 
-        console.log("post-edit obj", obj);
-
         api.sbot.async.publish(obj, done);
 
         function done(err, msg) {
@@ -134,7 +132,6 @@ exports.create = function (api) {
               detail: err.message,
             });
           } else {
-            console.log(msg);
             callback(msg);
           }
         }