From 75296498c4aa5dd701dd385a3834747c798eb237 Mon Sep 17 00:00:00 2001
From: Thomas Citharel <tcit@tcit.fr>
Date: Wed, 2 Nov 2022 17:12:35 +0100
Subject: [PATCH 1/9] Do not list drafts in upcoming / old events event if
 moderator

Closes #1086

Signed-off-by: Thomas Citharel <tcit@tcit.fr>
---
 lib/graphql/resolvers/group.ex | 3 +--
 1 file changed, 1 insertion(+), 2 deletions(-)

diff --git a/lib/graphql/resolvers/group.ex b/lib/graphql/resolvers/group.ex
index 9bc26198c..f2d6de213 100644
--- a/lib/graphql/resolvers/group.ex
+++ b/lib/graphql/resolvers/group.ex
@@ -446,8 +446,7 @@ defmodule Mobilizon.GraphQL.Resolvers.Group do
           }
         }
       ) do
-    if Actors.is_member?(actor_id, group_id) or is_moderator(user_role) do
-      # TODO : Handle public / restricted to group members events
+    if Actors.is_member?(actor_id, group_id) do
       {:ok,
        Events.list_organized_events_for_group(
          group,

From 4be5cc787ef860bd60fbfc0ad982b36cc35b61ae Mon Sep 17 00:00:00 2001
From: Thomas Citharel <tcit@tcit.fr>
Date: Wed, 2 Nov 2022 17:12:59 +0100
Subject: [PATCH 2/9] Make sure group is refreshed after action

Signed-off-by: Thomas Citharel <tcit@tcit.fr>
---
 js/src/composition/apollo/group.ts | 5 ++++-
 1 file changed, 4 insertions(+), 1 deletion(-)

diff --git a/js/src/composition/apollo/group.ts b/js/src/composition/apollo/group.ts
index 63c238ebe..0483d5872 100644
--- a/js/src/composition/apollo/group.ts
+++ b/js/src/composition/apollo/group.ts
@@ -55,7 +55,10 @@ export function useGroup(
       name: unref(name),
       ...options,
     }),
-    () => ({ enabled: unref(name) !== undefined && unref(name) !== "" })
+    () => ({
+      enabled: unref(name) !== undefined && unref(name) !== "",
+      fetchPolicy: "cache-and-network",
+    })
   );
   const group = computed(() => result.value?.group);
   return { group, error, loading, onResult, onError, refetch };

From 02381f2820f9255453df94427e6fdb83e435daf0 Mon Sep 17 00:00:00 2001
From: Thomas Citharel <tcit@tcit.fr>
Date: Wed, 2 Nov 2022 17:13:16 +0100
Subject: [PATCH 3/9] Fix deleting person detached from user

Signed-off-by: Thomas Citharel <tcit@tcit.fr>
---
 lib/mix/tasks/mobilizon/actors/delete.ex | 3 ++-
 1 file changed, 2 insertions(+), 1 deletion(-)

diff --git a/lib/mix/tasks/mobilizon/actors/delete.ex b/lib/mix/tasks/mobilizon/actors/delete.ex
index ffe6a686f..be55b0c24 100644
--- a/lib/mix/tasks/mobilizon/actors/delete.ex
+++ b/lib/mix/tasks/mobilizon/actors/delete.ex
@@ -93,7 +93,8 @@ defmodule Mix.Tasks.Mobilizon.Actors.Delete do
     end
   end
 
-  defp check_actor(%Actor{type: :Person, domain: nil} = profile, assume_yes?) do
+  defp check_actor(%Actor{type: :Person, domain: nil, user_id: user_id} = profile, assume_yes?)
+       when not is_nil(user_id) do
     %User{actors: actors, email: email} = Users.get_user_with_actors!(profile.user_id)
 
     if length(actors) == 1 do

From dd3e99bc5ef048815f7e89431d914e879865895c Mon Sep 17 00:00:00 2001
From: Thomas Citharel <tcit@tcit.fr>
Date: Wed, 2 Nov 2022 17:45:27 +0100
Subject: [PATCH 4/9] Fix pagination number text color in dark theme

Signed-off-by: Thomas Citharel <tcit@tcit.fr>
---
 js/src/assets/oruga-tailwindcss.css | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/js/src/assets/oruga-tailwindcss.css b/js/src/assets/oruga-tailwindcss.css
index 4310f3579..5924b0965 100644
--- a/js/src/assets/oruga-tailwindcss.css
+++ b/js/src/assets/oruga-tailwindcss.css
@@ -266,7 +266,7 @@ button.menubar__button {
   @apply px-3 dark:text-black;
 }
 .pagination-link-current {
-  @apply bg-primary cursor-not-allowed pointer-events-none border-primary text-white;
+  @apply bg-primary cursor-not-allowed pointer-events-none border-primary text-white dark:text-zinc-900;
 }
 .pagination-ellipsis {
   @apply text-center m-1 text-gray-300;

From 7ee79a00e270728f7879cdc14d58aa2d353357b1 Mon Sep 17 00:00:00 2001
From: Thomas Citharel <tcit@tcit.fr>
Date: Wed, 2 Nov 2022 17:45:49 +0100
Subject: [PATCH 5/9] Fix post sharing URL

Signed-off-by: Thomas Citharel <tcit@tcit.fr>
---
 js/src/components/Post/SharePostModal.vue | 21 +++------------------
 1 file changed, 3 insertions(+), 18 deletions(-)

diff --git a/js/src/components/Post/SharePostModal.vue b/js/src/components/Post/SharePostModal.vue
index d784646d6..16b1813e3 100644
--- a/js/src/components/Post/SharePostModal.vue
+++ b/js/src/components/Post/SharePostModal.vue
@@ -2,7 +2,7 @@
   <share-modal
     :title="t('Share this post')"
     :text="post.title"
-    :url="postURL"
+    :url="post.url ?? ''"
     :input-label="t('Post URL')"
   >
     <o-notification
@@ -11,7 +11,7 @@
       :closable="false"
     >
       {{
-        $t(
+        t(
           "This post is accessible only through it's link. Be careful where you post this link."
         )
       }}
@@ -22,29 +22,14 @@
 <script lang="ts" setup>
 import { PostVisibility } from "@/types/enums";
 import { IPost } from "../../types/post.model";
-import RouteName from "@/router/name";
-import { computed } from "vue";
-import { useRouter } from "vue-router";
 import { useI18n } from "vue-i18n";
 import ShareModal from "@/components/Share/ShareModal.vue";
 
-const props = defineProps<{
+defineProps<{
   post: IPost;
 }>();
 
 const { t } = useI18n({ useScope: "global" });
-
-const router = useRouter();
-
-const postURL = computed((): string => {
-  if (props.post.id) {
-    return router.resolve({
-      name: RouteName.POST,
-      params: { id: props.post.id },
-    }).href;
-  }
-  return props.post.url ?? "";
-});
 </script>
 <style lang="scss" scoped>
 .diaspora,

From 30eb37e31bf1d6b7b6695272e814c38609648cdd Mon Sep 17 00:00:00 2001
From: Thomas Citharel <tcit@tcit.fr>
Date: Wed, 2 Nov 2022 17:45:58 +0100
Subject: [PATCH 6/9] Post view improvements

Signed-off-by: Thomas Citharel <tcit@tcit.fr>
---
 js/src/views/Posts/PostView.vue | 72 ++++++++++++++++++---------------
 1 file changed, 39 insertions(+), 33 deletions(-)

diff --git a/js/src/views/Posts/PostView.vue b/js/src/views/Posts/PostView.vue
index 8ec1b0fb6..40bff220e 100644
--- a/js/src/views/Posts/PostView.vue
+++ b/js/src/views/Posts/PostView.vue
@@ -6,17 +6,17 @@
       </div>
       <div class="relative flex flex-col">
         <div
-          class="px-2 py-3 flex flex-wrap justify-center items-center"
+          class="px-2 py-3 flex flex-wrap gap-4 justify-center items-center"
           dir="auto"
         >
-          <div class="flex-1">
+          <div class="flex-auto min-w-[300px] max-w-screen-lg">
             <div class="inline">
               <tag
                 class="mr-2"
                 variant="warning"
                 size="medium"
                 v-if="post.draft"
-                >{{ $t("Draft") }}</tag
+                >{{ t("Draft") }}</tag
               >
               <h1 class="inline" :lang="post.language">
                 {{ post.title }}
@@ -52,7 +52,7 @@
               >
                 <Clock :size="16" />
                 {{
-                  $t("Edited {relative_time} ago", {
+                  t("Edited {relative_time} ago", {
                     relative_time: formatDistanceToNowStrict(
                       new Date(post.updatedAt),
                       {
@@ -64,7 +64,7 @@
               </span>
               <span v-if="post.visibility === PostVisibility.UNLISTED" class="">
                 <o-icon icon="link" size="small" />
-                {{ $t("Accessible only by link") }}
+                {{ t("Accessible only by link") }}
               </span>
               <span
                 v-else-if="post.visibility === PostVisibility.PRIVATE"
@@ -72,7 +72,7 @@
               >
                 <Lock :size="16" />
                 {{
-                  $t("Accessible only to members", {
+                  t("Accessible only to members", {
                     group: post.attributedTo?.name,
                   })
                 }}
@@ -82,24 +82,27 @@
           <o-dropdown position="bottom-left" aria-role="list">
             <template #trigger>
               <o-button role="button" icon-right="dots-horizontal">
-                {{ $t("Actions") }}
+                {{ t("Actions") }}
               </o-button>
             </template>
             <o-dropdown-item
               aria-role="listitem"
               has-link
+              tabIndex="-1"
               v-if="
                 currentActor?.id === post?.author?.id ||
                 isCurrentActorAGroupModerator
               "
             >
-              <router-link
+              <router-link class="flex gap-1 whitespace-nowrap"
                 :to="{
                   name: RouteName.POST_EDIT,
                   params: { slug: post.slug },
                 }"
-                >{{ $t("Edit") }} <o-icon icon="pencil"
-              /></router-link>
+                >
+                <Pencil />
+                {{ t("Edit") }}
+              </router-link>
             </o-dropdown-item>
             <o-dropdown-item
               aria-role="listitem"
@@ -107,11 +110,12 @@
                 currentActor?.id === post?.author?.id ||
                 isCurrentActorAGroupModerator
               "
-              @click="openDeletePostModal"
-              @keyup.enter="openDeletePostModal"
+              tabIndex="-1"
             >
-              {{ $t("Delete") }}
-              <o-icon icon="delete" />
+            <button @click="openDeletePostModal" class="flex gap-1 whitespace-nowrap">
+              <Delete />
+              {{ t("Delete") }}
+            </button>
             </o-dropdown-item>
 
             <hr
@@ -126,32 +130,30 @@
             <o-dropdown-item
               aria-role="listitem"
               v-if="!post.draft"
-              @click="triggerShare()"
-              @keyup.enter="triggerShare()"
+              tabIndex="-1"
             >
-              <span>
-                {{ $t("Share this event") }}
-                <o-icon icon="share" />
-              </span>
+            <button @click="triggerShare()" class="flex gap-1 whitespace-nowrap">
+                <Share />
+                {{ t("Share this event") }}
+              </button>
             </o-dropdown-item>
 
             <o-dropdown-item
               aria-role="listitem"
               v-if="ableToReport"
-              @click="isReportModalActive = true"
-              @keyup.enter="isReportModalActive = true"
+              tabIndex="-1"
             >
-              <span>
-                {{ $t("Report") }}
-                <o-icon icon="flag" />
-              </span>
+            <button @click="isReportModalActive = true" class="flex gap-1 whitespace-nowrap">
+                <Flag />
+                {{ t("Report") }}
+              </button>
             </o-dropdown-item>
           </o-dropdown>
         </div>
       </div>
     </header>
     <o-notification
-      :title="$t('Members-only post')"
+      :title="t('Members-only post')"
       class="mx-4"
       variant="warning"
       :closable="false"
@@ -164,7 +166,7 @@
       "
     >
       {{
-        $t(
+        t(
           "This post is accessible only for members. You have access to it for moderation purposes only because you are an instance moderator."
         )
       }}
@@ -173,10 +175,10 @@
     <section
       v-html="post.body"
       dir="auto"
-      class="px-1 prose lg:prose-xl prose-p:mt-6 dark:prose-invert"
+      class="px-2 md:px-4 py-4 prose lg:prose-xl prose-p:mt-6 dark:prose-invert bg-white dark:bg-zinc-700 mx-auto"
       :lang="post.language"
     />
-    <section class="flex gap-2 my-6" dir="auto">
+    <section class="flex gap-2 my-6 justify-center" dir="auto">
       <router-link
         v-for="tag in post.tags"
         :key="tag.title"
@@ -186,14 +188,14 @@
       </router-link>
     </section>
     <o-modal
-      :close-button-aria-label="$t('Close')"
+      :close-button-aria-label="t('Close')"
       v-model:active="isReportModalActive"
       has-modal-card
       ref="reportModal"
     >
       <ReportModal
         :on-confirm="reportPost"
-        :title="$t('Report this post')"
+        :title="t('Report this post')"
         :outside-domain="groupDomain"
         @close="isReportModalActive = false"
       />
@@ -202,7 +204,7 @@
       v-model:active="isShareModalActive"
       has-modal-card
       ref="shareModal"
-      :close-button-aria-label="$t('Close')"
+      :close-button-aria-label="t('Close')"
     >
       <share-post-modal :post="post" />
     </o-modal>
@@ -241,6 +243,10 @@ import { useRouter } from "vue-router";
 import { useCreateReport } from "@/composition/apollo/report";
 import Clock from "vue-material-design-icons/Clock.vue";
 import Lock from "vue-material-design-icons/Lock.vue";
+import Pencil from "vue-material-design-icons/Pencil.vue";
+import Delete from "vue-material-design-icons/Delete.vue";
+import Share from "vue-material-design-icons/Share.vue";
+import Flag from "vue-material-design-icons/Flag.vue";
 import { Dialog } from "@/plugins/dialog";
 import { useI18n } from "vue-i18n";
 import { Notifier } from "@/plugins/notifier";

From 3ddf360e1109fd0db3f6cc9b1b0ec51906faf6f8 Mon Sep 17 00:00:00 2001
From: Thomas Citharel <tcit@tcit.fr>
Date: Wed, 2 Nov 2022 18:02:20 +0100
Subject: [PATCH 7/9] Add breadcrumb trail on Post view

And lint

Closes #466

Signed-off-by: Thomas Citharel <tcit@tcit.fr>
---
 js/src/views/Posts/PostView.vue | 42 ++++++++++++++++++++++++++-------
 1 file changed, 34 insertions(+), 8 deletions(-)

diff --git a/js/src/views/Posts/PostView.vue b/js/src/views/Posts/PostView.vue
index 40bff220e..3db903ff7 100644
--- a/js/src/views/Posts/PostView.vue
+++ b/js/src/views/Posts/PostView.vue
@@ -1,5 +1,21 @@
 <template>
   <article class="container mx-auto post" v-if="post">
+    <breadcrumbs-nav
+      v-if="post.attributedTo"
+      :links="[
+        { name: RouteName.MY_GROUPS, text: t('My groups') },
+        {
+          name: RouteName.GROUP,
+          params: { preferredUsername: usernameWithDomain(post.attributedTo) },
+          text: displayName(post.attributedTo),
+        },
+        {
+          name: RouteName.POST,
+          params: { slug: post.slug },
+          text: post.title,
+        },
+      ]"
+    />
     <header>
       <div class="flex justify-center">
         <lazy-image-wrapper :picture="post.picture" />
@@ -94,12 +110,13 @@
                 isCurrentActorAGroupModerator
               "
             >
-              <router-link class="flex gap-1 whitespace-nowrap"
+              <router-link
+                class="flex gap-1 whitespace-nowrap"
                 :to="{
                   name: RouteName.POST_EDIT,
                   params: { slug: post.slug },
                 }"
-                >
+              >
                 <Pencil />
                 {{ t("Edit") }}
               </router-link>
@@ -112,10 +129,13 @@
               "
               tabIndex="-1"
             >
-            <button @click="openDeletePostModal" class="flex gap-1 whitespace-nowrap">
-              <Delete />
-              {{ t("Delete") }}
-            </button>
+              <button
+                @click="openDeletePostModal"
+                class="flex gap-1 whitespace-nowrap"
+              >
+                <Delete />
+                {{ t("Delete") }}
+              </button>
             </o-dropdown-item>
 
             <hr
@@ -132,7 +152,10 @@
               v-if="!post.draft"
               tabIndex="-1"
             >
-            <button @click="triggerShare()" class="flex gap-1 whitespace-nowrap">
+              <button
+                @click="triggerShare()"
+                class="flex gap-1 whitespace-nowrap"
+              >
                 <Share />
                 {{ t("Share this event") }}
               </button>
@@ -143,7 +166,10 @@
               v-if="ableToReport"
               tabIndex="-1"
             >
-            <button @click="isReportModalActive = true" class="flex gap-1 whitespace-nowrap">
+              <button
+                @click="isReportModalActive = true"
+                class="flex gap-1 whitespace-nowrap"
+              >
                 <Flag />
                 {{ t("Report") }}
               </button>

From e69e8a5044d82f68fe2edabe2b0445609a0b8843 Mon Sep 17 00:00:00 2001
From: Thomas Citharel <tcit@tcit.fr>
Date: Wed, 2 Nov 2022 18:09:26 +0100
Subject: [PATCH 8/9] Allow to configure editor heading levels and set level 2
 as start for posts

Signed-off-by: Thomas Citharel <tcit@tcit.fr>
---
 js/src/components/TextEditor.vue | 48 ++++++++++++++++++++++++++------
 js/src/views/Posts/EditView.vue  |  1 +
 js/src/views/Posts/PostView.vue  |  2 +-
 3 files changed, 42 insertions(+), 9 deletions(-)

diff --git a/js/src/components/TextEditor.vue b/js/src/components/TextEditor.vue
index a532a19b0..953d88740 100644
--- a/js/src/components/TextEditor.vue
+++ b/js/src/components/TextEditor.vue
@@ -44,8 +44,18 @@
         <button
           v-if="!isBasicMode"
           class="menubar__button"
-          :class="{ 'is-active': editor?.isActive('heading', { level: 3 }) }"
-          @click="editor?.chain().focus().toggleHeading({ level: 3 }).run()"
+          :class="{
+            'is-active': editor?.isActive('heading', {
+              level: props.headingLevel[0],
+            }),
+          }"
+          @click="
+            editor
+              ?.chain()
+              .focus()
+              .toggleHeading({ level: props.headingLevel[0] })
+              .run()
+          "
           type="button"
           :title="t('Heading Level 1')"
         >
@@ -55,8 +65,18 @@
         <button
           v-if="!isBasicMode"
           class="menubar__button"
-          :class="{ 'is-active': editor?.isActive('heading', { level: 4 }) }"
-          @click="editor?.chain().focus().toggleHeading({ level: 4 }).run()"
+          :class="{
+            'is-active': editor?.isActive('heading', {
+              level: props.headingLevel[1],
+            }),
+          }"
+          @click="
+            editor
+              ?.chain()
+              .focus()
+              .toggleHeading({ level: props.headingLevel[1] })
+              .run()
+          "
           type="button"
           :title="t('Heading Level 2')"
         >
@@ -66,8 +86,18 @@
         <button
           v-if="!isBasicMode"
           class="menubar__button"
-          :class="{ 'is-active': editor?.isActive('heading', { level: 5 }) }"
-          @click="editor?.chain().focus().toggleHeading({ level: 5 }).run()"
+          :class="{
+            'is-active': editor?.isActive('heading', {
+              level: props.headingLevel[2],
+            }),
+          }"
+          @click="
+            editor
+              ?.chain()
+              .focus()
+              .toggleHeading({ level: props.headingLevel[2] })
+              .run()
+          "
           type="button"
           :title="t('Heading Level 3')"
         >
@@ -196,7 +226,7 @@
 import { useEditor, EditorContent, BubbleMenu } from "@tiptap/vue-3";
 import Blockquote from "@tiptap/extension-blockquote";
 import BulletList from "@tiptap/extension-bullet-list";
-import Heading from "@tiptap/extension-heading";
+import Heading, { Level } from "@tiptap/extension-heading";
 import Document from "@tiptap/extension-document";
 import Paragraph from "@tiptap/extension-paragraph";
 import Bold from "@tiptap/extension-bold";
@@ -248,10 +278,12 @@ const props = withDefaults(
     ariaLabel?: string;
     currentActor: IPerson;
     placeholder?: string;
+    headingLevel?: Level[];
   }>(),
   {
     mode: "description",
     maxSize: 100_000_000,
+    headingLevel: () => [3, 4, 5],
   }
 );
 
@@ -310,7 +342,7 @@ const editor = useEditor({
     Blockquote,
     BulletList,
     Heading.configure({
-      levels: [3, 4, 5],
+      levels: props.headingLevel,
     }),
     Document,
     Paragraph,
diff --git a/js/src/views/Posts/EditView.vue b/js/src/views/Posts/EditView.vue
index 51877895b..d92a6df0b 100644
--- a/js/src/views/Posts/EditView.vue
+++ b/js/src/views/Posts/EditView.vue
@@ -43,6 +43,7 @@
             :aria-label="t('Post body')"
             :current-actor="currentActor"
             :placeholder="t('Write your post')"
+            :headingLevel="[2, 3, 4]"
           />
         </o-field>
         <h2 class="mt-2">{{ t("Who can view this post") }}</h2>
diff --git a/js/src/views/Posts/PostView.vue b/js/src/views/Posts/PostView.vue
index 3db903ff7..bcb2332e1 100644
--- a/js/src/views/Posts/PostView.vue
+++ b/js/src/views/Posts/PostView.vue
@@ -111,7 +111,7 @@
               "
             >
               <router-link
-                class="flex gap-1 whitespace-nowrap"
+                class="flex gap-1 whitespace-nowrap flex-1"
                 :to="{
                   name: RouteName.POST_EDIT,
                   params: { slug: post.slug },

From 84009a953088359a9af8991076eeb922fd4e8353 Mon Sep 17 00:00:00 2001
From: Thomas Citharel <tcit@tcit.fr>
Date: Wed, 2 Nov 2022 18:15:19 +0100
Subject: [PATCH 9/9] Fix current format status of text not displayed in text
 editor

Signed-off-by: Thomas Citharel <tcit@tcit.fr>
---
 js/src/components/TextEditor.vue | 9 +++++++++
 1 file changed, 9 insertions(+)

diff --git a/js/src/components/TextEditor.vue b/js/src/components/TextEditor.vue
index 953d88740..9a5b12abc 100644
--- a/js/src/components/TextEditor.vue
+++ b/js/src/components/TextEditor.vue
@@ -643,3 +643,12 @@ onBeforeUnmount(() => {
   height: 0;
 }
 </style>
+<style>
+.menubar__button {
+  @apply mx-0.5;
+}
+
+.menubar__button.is-active {
+  @apply bg-zinc-300 dark:bg-zinc-500;
+}
+</style>