From fa12e0f1d66eaca31f3cb9d647d89b6452c48a4a Mon Sep 17 00:00:00 2001
From: Faris Ansari <netchamp.faris@gmail.com>
Date: Thu, 11 Oct 2018 00:21:03 +0530
Subject: [PATCH] Initialize new theme and pages from scratch

- Add ListView and FormView
---
 models/doctype/Invoice/Invoice.js  | 20 +++++++++
 src/App.vue                        | 40 ++++-------------
 src/components/PageHeader.vue      | 10 +++++
 src/components/SearchInput.vue     |  3 ++
 src/components/Sidebar.vue         | 70 ++++++++++++++++++++++++++++++
 src/pages/FormView/index.vue       | 50 +++++++++++++++++++++
 src/pages/ListView/List.vue        | 67 ++++++++++++++++++++++++++++
 src/pages/ListView/ListCell.vue    | 10 +++++
 src/pages/ListView/ListRow.vue     | 21 +++++++++
 src/pages/ListView/ListToolbar.vue | 28 ++++++++++++
 src/pages/ListView/index.vue       | 42 ++++++++++++++++++
 src/router/index.js                | 34 +++++----------
 src/styles/index.scss              |  6 +++
 src/styles/variables.scss          |  5 ++-
 14 files changed, 352 insertions(+), 54 deletions(-)
 create mode 100644 src/components/PageHeader.vue
 create mode 100644 src/components/SearchInput.vue
 create mode 100644 src/components/Sidebar.vue
 create mode 100644 src/pages/FormView/index.vue
 create mode 100644 src/pages/ListView/List.vue
 create mode 100644 src/pages/ListView/ListCell.vue
 create mode 100644 src/pages/ListView/ListRow.vue
 create mode 100644 src/pages/ListView/ListToolbar.vue
 create mode 100644 src/pages/ListView/index.vue
 create mode 100644 src/styles/index.scss

diff --git a/models/doctype/Invoice/Invoice.js b/models/doctype/Invoice/Invoice.js
index a2252381..f47a7833 100644
--- a/models/doctype/Invoice/Invoice.js
+++ b/models/doctype/Invoice/Invoice.js
@@ -151,5 +151,25 @@ module.exports = {
                     <div class='col-4 text-muted'>${data.customer}</div>
                     <div class='col-4 text-muted text-right'>${frappe.format(data.grandTotal, 'Currency')}</div>`;
     }
+  },
+
+  listView: {
+    columns: [
+      'customer',
+      {
+        label: 'Status',
+        getValue(doc) {
+          return doc.submitted ? 'Paid' : 'Pending';
+        }
+      },
+      'grandTotal',
+      'date',
+      {
+        label: 'INV #',
+        getValue(doc) {
+          return doc.name;
+        }
+      }
+    ]
   }
 };
diff --git a/src/App.vue b/src/App.vue
index 4a4d9c67..a72b4a57 100644
--- a/src/App.vue
+++ b/src/App.vue
@@ -1,46 +1,24 @@
 <template>
   <div id="app">
-    <frappe-desk v-if="showDesk" :sidebarConfig="sidebarConfig">
-      <router-view />
-    </frappe-desk>
-    <router-view v-else name="setup" />
+    <div class="row no-gutters">
+      <sidebar class="col-2" />
+      <div class="page-container col-10 bg-light">
+        <router-view />
+      </div>
+    </div>
   </div>
 </template>
 
 <script>
-import Vue from 'vue';
-import Observable from 'frappejs/utils/observable';
-import Desk from 'frappejs/ui/components/Desk';
-import sidebarConfig from './sidebarConfig';
+import Sidebar from './components/Sidebar';
 
 export default {
   name: 'App',
-  data() {
-    return {
-      showDesk: true,
-      sidebarConfig
-    }
-  },
   components: {
-    FrappeDesk: Desk,
-  },
-  async beforeRouteUpdate(to, from, next) {
-    const accountingSettings = await frappe.getSingle('AccountingSettings');
-    if (accountingSettings.companyName) {
-      this.showDesk = true;
-    } else {
-      this.showDesk = true;
-    }
+    Sidebar
   }
 }
 </script>
-
 <style lang="scss">
-@import "~bootstrap/scss/bootstrap";
-@import '~frappe-datatable/dist/frappe-datatable';
-
-html {
-  font-size: 14px;
-}
-
+@import "styles/index.scss";
 </style>
diff --git a/src/components/PageHeader.vue b/src/components/PageHeader.vue
new file mode 100644
index 00000000..53020995
--- /dev/null
+++ b/src/components/PageHeader.vue
@@ -0,0 +1,10 @@
+<template>
+  <div class="page-header px-4 py-3 border-bottom bg-white">
+    <h5 class="m-0">{{ title }}</h5>
+  </div>
+</template>
+<script>
+export default {
+  props: ['title']
+}
+</script>
diff --git a/src/components/SearchInput.vue b/src/components/SearchInput.vue
new file mode 100644
index 00000000..bfb52050
--- /dev/null
+++ b/src/components/SearchInput.vue
@@ -0,0 +1,3 @@
+<template>
+  <input type="search" class="form-control" placeholder="Search..." autocomplete="off" spellcheck="false">
+</template>
diff --git a/src/components/Sidebar.vue b/src/components/Sidebar.vue
new file mode 100644
index 00000000..8d5868a4
--- /dev/null
+++ b/src/components/Sidebar.vue
@@ -0,0 +1,70 @@
+<template>
+  <div class="page-sidebar bg-dark p-2 text-light">
+    <div class="company-name px-3 py-2 my-2">
+      <h6 class="m-0">{{ companyName }}</h6>
+    </div>
+    <div :class="['sidebar-item px-3 py-2 ', isCurrentRoute(item.route) ? 'active' : '']" @click="routeTo(item.route)" v-for="item in items" :key="item.label">
+      {{ item.label }}
+    </div>
+  </div>
+</template>
+<script>
+export default {
+  data() {
+    return {
+      companyName: '',
+      items: [
+        {
+          label: 'Invoices',
+          route: '/list/Invoice'
+        },
+        {
+          label: 'Customers',
+          route: '/list/Party'
+        },
+        {
+          label: 'Items',
+          route: '/list/Item'
+        },
+        {
+          label: 'Reports',
+          route: '/reports'
+        },
+        {
+          label: 'Settings',
+          route: '/settings'
+        },
+      ]
+    }
+  },
+  async mounted() {
+    const accountingSettings = await frappe.getDoc('AccountingSettings');
+    this.companyName = accountingSettings.companyName;
+  },
+  methods: {
+    isCurrentRoute(route) {
+      return this.$route.path === route;
+    },
+    routeTo(route) {
+      this.$router.push(route);
+    }
+  }
+}
+</script>
+
+<style lang="scss">
+@import "../styles/variables.scss";
+
+.page-sidebar {
+  height: 100vh;
+}
+
+.sidebar-item {
+  cursor: pointer;
+  border-radius: $border-radius;
+
+  &:hover, &.active {
+    background-color: $primary;
+  }
+}
+</style>
diff --git a/src/pages/FormView/index.vue b/src/pages/FormView/index.vue
new file mode 100644
index 00000000..9ceecfb8
--- /dev/null
+++ b/src/pages/FormView/index.vue
@@ -0,0 +1,50 @@
+<template>
+  <div class="bg-light">
+    <page-header :title="name" />
+    <div class="form-container col-8 bg-white mt-4 ml-auto mr-auto border px-4 py-3">
+      <form-layout
+        class="p-3"
+        v-if="shouldRenderForm"
+        :doc="doc"
+        :fields="meta.fields"
+        :layout="meta.layout"
+        :invalid="invalid"
+      />
+    </div>
+  </div>
+</template>
+<script>
+import frappe from 'frappejs';
+import FormLayout from 'frappejs/ui/components/Form/FormLayout';
+import PageHeader from '@/components/PageHeader';
+
+export default {
+  name: 'FormView',
+  props: ['doctype', 'name'],
+  components: {
+    PageHeader,
+    FormLayout
+  },
+  data() {
+    return {
+      doc: null
+    }
+  },
+  computed: {
+    shouldRenderForm() {
+      return this.name && this.doc;
+    },
+    meta() {
+      return frappe.getMeta(this.doctype);
+    }
+  },
+  async mounted() {
+    this.doc = await frappe.getDoc(this.doctype, this.name);
+  }
+}
+</script>
+<style>
+.form-container {
+
+}
+</style>
diff --git a/src/pages/ListView/List.vue b/src/pages/ListView/List.vue
new file mode 100644
index 00000000..ef770925
--- /dev/null
+++ b/src/pages/ListView/List.vue
@@ -0,0 +1,67 @@
+<template>
+  <div class="list-container">
+    <list-row class="text-muted rounded-top bg-light">
+      <list-cell v-for="column in columns" :key="column.label">
+        {{ column.label }}
+      </list-cell>
+    </list-row>
+    <list-row v-for="doc in data" :key="doc.name" @click.native="openForm(doc.name)">
+      <list-cell v-for="column in columns" :key="column.label">
+        {{ column.getValue(doc) }}
+      </list-cell>
+    </list-row>
+  </div>
+</template>
+<script>
+import ListRow from './ListRow';
+import ListCell from './ListCell';
+
+export default {
+  name: 'List',
+  props: ['doctype'],
+  components: {
+    ListRow,
+    ListCell
+  },
+  data() {
+    return {
+      columns: [],
+      data: []
+    };
+  },
+  async mounted() {
+    this.prepareColumns();
+
+    this.data = await frappe.db.getAll({
+      doctype: this.doctype,
+      fields: ['*']
+    });
+  },
+  methods: {
+    openForm(name) {
+      console.log(name);
+      this.$router.push(`/edit/${this.doctype}/${name}`);
+    },
+    prepareColumns() {
+      this.columns = this.meta.listView.columns.map(col => {
+        if (typeof col === 'string') {
+          const field = this.meta.getField(col);
+          if (!field) return null;
+          return {
+            label: field.label,
+            getValue(doc) {
+              return doc[col];
+            }
+          };
+        }
+        return col;
+      }).filter(Boolean);
+    }
+  },
+  computed: {
+    meta() {
+      return frappe.getMeta(this.doctype);
+    }
+  }
+};
+</script>
diff --git a/src/pages/ListView/ListCell.vue b/src/pages/ListView/ListCell.vue
new file mode 100644
index 00000000..2a42a91c
--- /dev/null
+++ b/src/pages/ListView/ListCell.vue
@@ -0,0 +1,10 @@
+<template>
+  <div class="col">
+    <slot></slot>
+  </div>
+</template>
+<script>
+export default {
+  props: ['col']
+}
+</script>
diff --git a/src/pages/ListView/ListRow.vue b/src/pages/ListView/ListRow.vue
new file mode 100644
index 00000000..b037a10c
--- /dev/null
+++ b/src/pages/ListView/ListRow.vue
@@ -0,0 +1,21 @@
+<template>
+  <div class="list-row row no-gutters py-2 px-3">
+    <slot></slot>
+  </div>
+</template>
+<style lang="scss">
+@import "../../styles/variables";
+.list-row {
+  cursor: pointer;
+  border: 1px solid $border-color;
+  background-color: var(--white);
+
+  &:hover {
+    background-color: var(--light);
+  }
+}
+
+.list-row:not(:first-child) {
+  border-top: none;
+}
+</style>
diff --git a/src/pages/ListView/ListToolbar.vue b/src/pages/ListView/ListToolbar.vue
new file mode 100644
index 00000000..50b0c830
--- /dev/null
+++ b/src/pages/ListView/ListToolbar.vue
@@ -0,0 +1,28 @@
+<template>
+  <div class="row">
+    <div class="col-6 d-flex">
+      <search-input class="mr-2" />
+      <f-button secondary>{{ _('Add Filter') }}</f-button>
+    </div>
+    <div class="col-6 d-flex flex-row-reverse">
+      <f-button primary @click="$emit('newClick')">{{ _('New {0}', doctype) }}</f-button>
+    </div>
+  </div>
+</template>
+<script>
+import SearchInput from '@/components/SearchInput';
+
+export default {
+  name: 'ListToolbar',
+  props: ['doctype'],
+  components: {
+    SearchInput
+  },
+  methods: {
+    async newInvoice() {
+      const doc = await frappe.getNewDoc('Invoice');
+      this.$formModal.open(doc);
+    }
+  }
+}
+</script>
diff --git a/src/pages/ListView/index.vue b/src/pages/ListView/index.vue
new file mode 100644
index 00000000..159aa0b6
--- /dev/null
+++ b/src/pages/ListView/index.vue
@@ -0,0 +1,42 @@
+<template>
+  <div class="bg-light">
+    <page-header :title="meta.label || meta.name" />
+    <div class="px-4 py-3">
+      <list-toolbar
+        :doctype="doctype"
+        @newClick="openNewForm"
+        class="mb-4"
+      />
+      <list
+        :doctype="doctype"
+      />
+    </div>
+  </div>
+</template>
+<script>
+import frappe from 'frappejs';
+import PageHeader from '@/components/PageHeader';
+import ListToolbar from './ListToolbar';
+import List from './List';
+
+export default {
+  name: 'ListView',
+  props: ['doctype'],
+  components: {
+    PageHeader,
+    ListToolbar,
+    List
+  },
+  computed: {
+    meta() {
+      return frappe.getMeta(this.doctype);
+    }
+  },
+  methods: {
+    async openNewForm() {
+      const doc = await frappe.getNewDoc(this.doctype);
+      this.$router.push(`/edit/${this.doctype}/${doc.name}`);
+    }
+  }
+}
+</script>
diff --git a/src/router/index.js b/src/router/index.js
index f190e2c0..8b2f0787 100644
--- a/src/router/index.js
+++ b/src/router/index.js
@@ -1,34 +1,24 @@
 import Vue from 'vue';
 import Router from 'vue-router';
-import coreRoutes from 'frappejs/ui/routes';
 
-import SetupWizard from '../pages/SetupWizard';
-import Report from 'frappejs/ui/pages/Report';
-import reportViewConfig from '../../reports/view';
+import ListView from '../pages/ListView';
+import FormView from '../pages/FormView';
 
 Vue.use(Router);
 
-const routes = [].concat(coreRoutes, [
+const routes = [
   {
-    path: '/setup-wizard',
-    name: 'SetupWizard',
-    components: {
-      setup: SetupWizard
-    }
+    path: '/list/:doctype',
+    name: 'ListView',
+    component: ListView,
+    props: true
   },
   {
-    path: '/report/:reportName',
-    name: 'Report',
-    component: Report,
-    props: (route) => {
-      const { reportName } = route.params;
-      return {
-        reportName,
-        reportConfig: reportViewConfig[reportName] || null,
-        filters: route.query
-      };
-    }
+    path: '/edit/:doctype/:name',
+    name: 'FormView',
+    component: FormView,
+    props: true
   }
-]);
+];
 
 export default new Router({ routes });
diff --git a/src/styles/index.scss b/src/styles/index.scss
new file mode 100644
index 00000000..48ec2515
--- /dev/null
+++ b/src/styles/index.scss
@@ -0,0 +1,6 @@
+@import "variables.scss";
+@import "~bootstrap/scss/bootstrap";
+
+html {
+  font-size: 14px;
+}
\ No newline at end of file
diff --git a/src/styles/variables.scss b/src/styles/variables.scss
index 367009b9..f9c673a5 100644
--- a/src/styles/variables.scss
+++ b/src/styles/variables.scss
@@ -1,2 +1,5 @@
 @import "~bootstrap/scss/functions";
-@import "~bootstrap/scss/variables";
\ No newline at end of file
+@import "~bootstrap/scss/variables";
+
+$dark: #252529;
+$blue: #1665D8;
\ No newline at end of file