From 8453cea3f0a6dc4babf6917ae868c429f873635d Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?Bahad=C4=B1r=20Sofuo=C4=9Flu?= <bahadirsofuoglu@gmail.com>
Date: Thu, 1 Jun 2023 15:14:49 +0300
Subject: [PATCH] Refactor #3965 - For Timeline

---
 components/lib/timeline/BaseTimeline.vue | 155 +++++++++++++++++++++++
 components/lib/timeline/Timeline.d.ts    |   5 +
 components/lib/timeline/Timeline.vue     | 144 +--------------------
 3 files changed, 167 insertions(+), 137 deletions(-)
 create mode 100644 components/lib/timeline/BaseTimeline.vue

diff --git a/components/lib/timeline/BaseTimeline.vue b/components/lib/timeline/BaseTimeline.vue
new file mode 100644
index 000000000..55cfc3045
--- /dev/null
+++ b/components/lib/timeline/BaseTimeline.vue
@@ -0,0 +1,155 @@
+<script>
+import BaseComponent from 'primevue/basecomponent';
+import { useStyle } from 'primevue/usestyle';
+
+const styles = `
+.p-timeline {
+    display: flex;
+    flex-grow: 1;
+    flex-direction: column;
+}
+
+.p-timeline-left .p-timeline-event-opposite {
+    text-align: right;
+}
+
+.p-timeline-left .p-timeline-event-content {
+    text-align: left;
+}
+
+.p-timeline-right .p-timeline-event {
+    flex-direction: row-reverse;
+}
+
+.p-timeline-right .p-timeline-event-opposite {
+    text-align: left;
+}
+
+.p-timeline-right .p-timeline-event-content {
+    text-align: right;
+}
+
+.p-timeline-vertical.p-timeline-alternate .p-timeline-event:nth-child(even) {
+    flex-direction: row-reverse;
+}
+
+.p-timeline-vertical.p-timeline-alternate .p-timeline-event:nth-child(odd) .p-timeline-event-opposite {
+    text-align: right;
+}
+
+.p-timeline-vertical.p-timeline-alternate .p-timeline-event:nth-child(odd) .p-timeline-event-content {
+    text-align: left;
+}
+
+.p-timeline-vertical.p-timeline-alternate .p-timeline-event:nth-child(even) .p-timeline-event-opposite {
+    text-align: left;
+}
+
+.p-timeline-vertical.p-timeline-alternate .p-timeline-event:nth-child(even) .p-timeline-event-content {
+    text-align: right;
+}
+
+.p-timeline-event {
+    display: flex;
+    position: relative;
+    min-height: 70px;
+}
+
+.p-timeline-event:last-child {
+    min-height: 0;
+}
+
+.p-timeline-event-opposite {
+    flex: 1;
+    padding: 0 1rem;
+}
+
+.p-timeline-event-content {
+    flex: 1;
+    padding: 0 1rem;
+}
+
+.p-timeline-event-separator {
+    flex: 0;
+    display: flex;
+    align-items: center;
+    flex-direction: column;
+}
+
+.p-timeline-event-marker {
+    display: flex;
+    align-self: baseline;
+}
+
+.p-timeline-event-connector {
+    flex-grow: 1;
+}
+
+.p-timeline-horizontal {
+    flex-direction: row;
+}
+
+.p-timeline-horizontal .p-timeline-event {
+    flex-direction: column;
+    flex: 1;
+}
+
+.p-timeline-horizontal .p-timeline-event:last-child {
+    flex: 0;
+}
+
+.p-timeline-horizontal .p-timeline-event-separator {
+    flex-direction: row;
+}
+
+.p-timeline-horizontal .p-timeline-event-connector {
+    width: 100%;
+}
+
+.p-timeline-bottom .p-timeline-event {
+    flex-direction: column-reverse;
+}
+
+.p-timeline-horizontal.p-timeline-alternate .p-timeline-event:nth-child(even) {
+    flex-direction: column-reverse;
+}
+`;
+
+const classes = {
+    root: ({ props }) => ['p-timeline p-component', 'p-timeline-' + props.align, 'p-timeline-' + props.layout],
+    event: 'p-timeline-event',
+    opposite: 'p-timeline-event-opposite',
+    separator: 'p-timeline-event-separator',
+    marker: 'p-timeline-event-marker',
+    connector: 'p-timeline-event-connector',
+    content: 'p-timeline-event-content'
+};
+
+const { load: loadStyle } = useStyle(styles, { id: 'primevue_timeline_style', manual: true });
+
+export default {
+    name: 'BaseTimeline',
+    extends: BaseComponent,
+    props: {
+        value: null,
+        align: {
+            mode: String,
+            default: 'left'
+        },
+        layout: {
+            mode: String,
+            default: 'vertical'
+        },
+        dataKey: null
+    },
+    css: {
+        classes,
+        loadStyle
+    },
+    provide() {
+        return {
+            $parentInstance: this
+        };
+    }
+};
+</script>
diff --git a/components/lib/timeline/Timeline.d.ts b/components/lib/timeline/Timeline.d.ts
index fc6996397..b5b8da24d 100644
--- a/components/lib/timeline/Timeline.d.ts
+++ b/components/lib/timeline/Timeline.d.ts
@@ -80,6 +80,11 @@ export interface TimelineProps {
      * @type {TimelinePassThroughOptions}
      */
     pt?: TimelinePassThroughOptions;
+    /**
+     * When enabled, it removes component related styles in the core.
+     * @defaultValue false
+     */
+    unstyled?: boolean;
 }
 
 /**
diff --git a/components/lib/timeline/Timeline.vue b/components/lib/timeline/Timeline.vue
index d43359fdc..0546c5216 100644
--- a/components/lib/timeline/Timeline.vue
+++ b/components/lib/timeline/Timeline.vue
@@ -1,18 +1,18 @@
 <template>
-    <div :class="containerClass" v-bind="ptm('root')">
-        <div v-for="(item, index) of value" :key="getKey(item, index)" class="p-timeline-event" v-bind="ptm('event')">
-            <div class="p-timeline-event-opposite" v-bind="ptm('opposite')">
+    <div :class="cx('root')" v-bind="ptm('root')">
+        <div v-for="(item, index) of value" :key="getKey(item, index)" :class="cx('event')" v-bind="ptm('event')">
+            <div :class="cx('opposite')" v-bind="ptm('opposite')">
                 <slot name="opposite" :item="item" :index="index"></slot>
             </div>
-            <div class="p-timeline-event-separator" v-bind="ptm('separator')">
+            <div :class="cx('separator')" v-bind="ptm('separator')">
                 <slot name="marker" :item="item" :index="index">
-                    <div class="p-timeline-event-marker" v-bind="ptm('marker')"></div>
+                    <div :class="cx('marker')" v-bind="ptm('marker')"></div>
                 </slot>
                 <slot v-if="index !== value.length - 1" name="connector" :item="item" :index="index">
-                    <div class="p-timeline-event-connector" v-bind="ptm('connector')"></div>
+                    <div :class="cx('connector')" v-bind="ptm('connector')"></div>
                 </slot>
             </div>
-            <div class="p-timeline-event-content" v-bind="ptm('content')">
+            <div :class="cx('content')" v-bind="ptm('content')">
                 <slot name="content" :item="item" :index="index"></slot>
             </div>
         </div>
@@ -26,140 +26,10 @@ import { ObjectUtils } from 'primevue/utils';
 export default {
     name: 'Timeline',
     extends: BaseComponent,
-    props: {
-        value: null,
-        align: {
-            mode: String,
-            default: 'left'
-        },
-        layout: {
-            mode: String,
-            default: 'vertical'
-        },
-        dataKey: null
-    },
     methods: {
         getKey(item, index) {
             return this.dataKey ? ObjectUtils.resolveFieldData(item, this.dataKey) : index;
         }
-    },
-    computed: {
-        containerClass() {
-            return ['p-timeline p-component', 'p-timeline-' + this.align, 'p-timeline-' + this.layout];
-        }
     }
 };
 </script>
-
-<style>
-.p-timeline {
-    display: flex;
-    flex-grow: 1;
-    flex-direction: column;
-}
-
-.p-timeline-left .p-timeline-event-opposite {
-    text-align: right;
-}
-
-.p-timeline-left .p-timeline-event-content {
-    text-align: left;
-}
-
-.p-timeline-right .p-timeline-event {
-    flex-direction: row-reverse;
-}
-
-.p-timeline-right .p-timeline-event-opposite {
-    text-align: left;
-}
-
-.p-timeline-right .p-timeline-event-content {
-    text-align: right;
-}
-
-.p-timeline-vertical.p-timeline-alternate .p-timeline-event:nth-child(even) {
-    flex-direction: row-reverse;
-}
-
-.p-timeline-vertical.p-timeline-alternate .p-timeline-event:nth-child(odd) .p-timeline-event-opposite {
-    text-align: right;
-}
-
-.p-timeline-vertical.p-timeline-alternate .p-timeline-event:nth-child(odd) .p-timeline-event-content {
-    text-align: left;
-}
-
-.p-timeline-vertical.p-timeline-alternate .p-timeline-event:nth-child(even) .p-timeline-event-opposite {
-    text-align: left;
-}
-
-.p-timeline-vertical.p-timeline-alternate .p-timeline-event:nth-child(even) .p-timeline-event-content {
-    text-align: right;
-}
-
-.p-timeline-event {
-    display: flex;
-    position: relative;
-    min-height: 70px;
-}
-
-.p-timeline-event:last-child {
-    min-height: 0;
-}
-
-.p-timeline-event-opposite {
-    flex: 1;
-    padding: 0 1rem;
-}
-
-.p-timeline-event-content {
-    flex: 1;
-    padding: 0 1rem;
-}
-
-.p-timeline-event-separator {
-    flex: 0;
-    display: flex;
-    align-items: center;
-    flex-direction: column;
-}
-
-.p-timeline-event-marker {
-    display: flex;
-    align-self: baseline;
-}
-
-.p-timeline-event-connector {
-    flex-grow: 1;
-}
-
-.p-timeline-horizontal {
-    flex-direction: row;
-}
-
-.p-timeline-horizontal .p-timeline-event {
-    flex-direction: column;
-    flex: 1;
-}
-
-.p-timeline-horizontal .p-timeline-event:last-child {
-    flex: 0;
-}
-
-.p-timeline-horizontal .p-timeline-event-separator {
-    flex-direction: row;
-}
-
-.p-timeline-horizontal .p-timeline-event-connector {
-    width: 100%;
-}
-
-.p-timeline-bottom .p-timeline-event {
-    flex-direction: column-reverse;
-}
-
-.p-timeline-horizontal.p-timeline-alternate .p-timeline-event:nth-child(even) {
-    flex-direction: column-reverse;
-}
-</style>