fix: Fix ui issues in new article view layout - EXO-73043 - Meeds-io/…

…MIPs#129 Fix ui issues in new article view layout
Meeds-io · Oct 7, 2024 · 25e3984 · 25e3984
1 parent 25c20f4
commit 25e3984
Show file tree

Hide file tree

Showing 3 changed files with 16 additions and 10 deletions.
diff --git a/content-webapp/src/main/webapp/skin/less/newsDetails.less b/content-webapp/src/main/webapp/skin/less/newsDetails.less
@@ -182,7 +182,8 @@
   .newsDetails {
     .news-top-information {
       .articleTitle {
-        font-size: 34px!important;
+        font-size: 34px !important;
+        line-height: 1.1;
       }
 
       .newsTitle {

diff --git a/content-webapp/src/main/webapp/vue-app/news-details/components/ExoNewsDetailsBody.vue b/content-webapp/src/main/webapp/vue-app/news-details/components/ExoNewsDetailsBody.vue
@@ -31,17 +31,17 @@
           :alt="featuredImageAltText"
           :src="`${illustrationURL}&size=0x400`"
           contain
-          class="mb-5"
+          class="mt-5 mb-5"
           width="100%"
           max-height="400" />
       </div>
       <div class="newsDetails">
         <div class="news-top-information d-flex">
-          <p class="text-color font-weight-bold articleTitle text-break mb-0">
-            <span class="me-5">
+          <p class="text-color font-weight-bold articleTitle text-break mt-5 mb-0">
+            <span>
               {{ newsTitle }}
             </span>
-            <span>
+            <span class="ms-3">
               <content-translation-menu
                 :translations="translations"
                 :selected-translation="selectedTranslation"
@@ -54,9 +54,9 @@
           class="article-summary text-break text-sub-title mt-4 mb-0">
           {{ newsSummary }}
         </p>
-        <div class="mt-5">
+        <div class="mt-4">
           <div
-            v-if="!hiddenSpace"
+            v-if="!hiddenSpace && !isPublicAccess"
             class="d-flex">
             <exo-space-avatar
               :space-id="spaceId"
@@ -96,7 +96,7 @@
             </div>
             <div
               v-else
-              :class="{'no-updater-info': !hiddenSpace}"
+              :class="{'no-updater-info': !hiddenSpace && !isPublicAccess}"
               class="text-sub-title">
               <date-format
                 :value="updatedDate"
@@ -279,7 +279,6 @@ export default {
     newsTitleContent: null,
     newsSummaryContent: null,
     newsBodyContent: null,
-    showUpdaterInfo: true
   }),
   created() {
     this.setNewsTitle(this.news?.title);
@@ -293,6 +292,12 @@ export default {
     articleNewLayoutEnabled() {
       return eXo?.env?.portal?.articleNewLayoutEnabled;
     },
+    showUpdaterInfo() {
+      return !this.isPublicAccess;
+    },
+    isPublicAccess() {
+      return !eXo?.env?.portal?.userIdentityId;
+    },
     params() {
       return {
         news: this.news,

diff --git a/content-webapp/src/main/webapp/vue-app/news-details/components/ExoNewsDetailsToolBar.vue b/content-webapp/src/main/webapp/vue-app/news-details/components/ExoNewsDetailsToolBar.vue
@@ -21,7 +21,7 @@
 <template>
   <div
     :class="{'me-5': !showEditButton}"
-    class="newsDetailsTopBar mb-5">
+    class="newsDetailsTopBar">
     <a
       v-if="!articleNewLayoutEnabled"
       class="backBtn"