From 3dc54be2d0fbc648407fafdcd72bd27bc5d19cc8 Mon Sep 17 00:00:00 2001
From: esikkala <esko.ikkala@aalto.fi>
Date: Wed, 28 Aug 2019 12:09:00 +0300
Subject: [PATCH] Manuscripts: update table columns

---
 src/client/reducers/manuscripts.js       | 190 +++++++++++++++++++----
 src/client/reducers/manuscriptsFacets.js |   5 +-
 2 files changed, 163 insertions(+), 32 deletions(-)

diff --git a/src/client/reducers/manuscripts.js b/src/client/reducers/manuscripts.js
index 1460803b..fe1d28ed 100644
--- a/src/client/reducers/manuscripts.js
+++ b/src/client/reducers/manuscripts.js
@@ -43,7 +43,7 @@ export const INITIAL_STATE = {
       id: 'prefLabel',
       label: 'Label',
       desc: `
-        A short label describing the manuscript
+        A short label describing the manuscript.
       `,
       valueType: 'object',
       makeLink: true,
@@ -69,7 +69,7 @@ export const INITIAL_STATE = {
     {
       id: 'work',
       label: 'Work',
-      desc: 'Work description',
+      desc: 'The intellectual content (works) contained in the manuscript.',
       valueType: 'object',
       makeLink: true,
       externalLink: false,
@@ -81,7 +81,7 @@ export const INITIAL_STATE = {
     {
       id: 'expression',
       label: 'Expression',
-      desc: 'Expression description',
+      desc: 'The linguistic versions of the works contained in the manuscript.',
       valueType: 'object',
       makeLink: true,
       externalLink: false,
@@ -94,11 +94,12 @@ export const INITIAL_STATE = {
       id: 'productionPlace',
       label: 'Production place',
       desc: `
-        The location where the manuscript was written. Multiple places of
-        production may appear for a single manuscript, when there are
-        discrepancies between the contributing data sources or when the
-        precise location is uncertain, or when the production indeed took
-        place in several places (e.g. for composite manuscripts).
+        The location where the manuscript was written. Multiple production places
+        may appear for a single manuscript due to the following reasons:  1) there
+        are discrepancies in the contributing data source,  2) there are discrepancies
+        between several contributing data sources, 3) the precise date is uncertain,
+        4) the production indeed took place on several occasions (e.g. for composite
+        manuscripts).
       `,
       valueType: 'object',
       makeLink: true,
@@ -111,10 +112,11 @@ export const INITIAL_STATE = {
       id: 'productionTimespan',
       label: 'Production date',
       desc: `
-        The date when the manuscript was written. Multiple production dates
-        may appear for a single manuscript, when there are discrepancies
-        between the contributing data sources or when the precise date is
-        uncertain.
+        The date when the manuscript was written. Multiple production dates may appear
+        for a single manuscript due to the following reasons:  1) there are discrepancies
+        in the contributing data source,  2) there are discrepancies between several
+        contributing data sources, 3) the precise date is uncertain, 4) the production
+        indeed took place on several occasions (e.g. for composite manuscripts).
       `,
       valueType: 'object',
       makeLink: false,
@@ -136,6 +138,19 @@ export const INITIAL_STATE = {
       numberedList: false,
       minWidth: 250,
     },
+    {
+      id: 'language',
+      label: 'Language',
+      desc: `
+        The language(s) in which the manuscript was written.
+      `,
+      valueType: 'object',
+      makeLink: true,
+      externalLink: true,
+      sortValues: true,
+      numberedList: false,
+      minWidth: 150,
+    },
     {
       id: 'event',
       label: 'Event',
@@ -150,57 +165,172 @@ export const INITIAL_STATE = {
       minWidth: 280,
     },
     {
-      id: 'language',
-      label: 'Language',
+      id: 'owner',
+      label: 'Owner',
       desc: `
-        The language(s) in which the manuscript was written.
+        Former or current owners (individual or institutional).
       `,
       valueType: 'object',
       makeLink: true,
-      externalLink: true,
+      externalLink: false,
       sortValues: true,
       numberedList: false,
-      minWidth: 150,
+      minWidth: 250
     },
     {
-      id: 'material',
-      label: 'Material',
+      id: 'collection',
+      label: 'Collection',
       desc: `
-        Material info.
+        The collection(s) that the manuscript has been part of at some point in time.
       `,
       valueType: 'object',
       makeLink: true,
-      externalLink: true,
+      externalLink: false,
       sortValues: true,
       numberedList: false,
-      minWidth: 150,
+      minWidth: 200
     },
     {
-      id: 'owner',
-      label: 'Owner',
+      id: 'transferOfCustodyPlace',
+      label: 'Transfer of Custody Place',
       desc: `
-        Former or current owners (individual or institutional).
+        The locations of “Transfer of Custody” events related to the manuscript.
       `,
       valueType: 'object',
       makeLink: true,
       externalLink: false,
       sortValues: true,
       numberedList: false,
-      minWidth: 250
+      minWidth: 200,
     },
     {
-      id: 'collection',
-      label: 'Collection',
+      id: 'transferOfCustodyTimespan',
+      label: 'Transfer of Custody Date',
       desc: `
-        Collection
+        The dates of “Transfer of Custody” events related to the manuscript.
       `,
       valueType: 'object',
       makeLink: true,
       externalLink: false,
       sortValues: true,
       numberedList: false,
-      minWidth: 200
+      minWidth: 200,
+    },
+    {
+      id: 'material',
+      label: 'Material',
+      desc: `
+        The physical material on which the text is written.
+      `,
+      valueType: 'object',
+      makeLink: true,
+      externalLink: true,
+      sortValues: true,
+      numberedList: false,
+      minWidth: 150,
+    },
+    {
+      id: 'height',
+      label: 'Height',
+      desc: `
+        The height of the manuscript in millimeters.
+      `,
+      valueType: 'string',
+      makeLink: false,
+      externalLink: false,
+      sortValues: true,
+      numberedList: false,
+      minWidth: 150,
     },
+    {
+      id: 'width',
+      label: 'Width',
+      desc: `
+        The width of the manuscript in millimeters.
+      `,
+      valueType: 'string',
+      makeLink: false,
+      externalLink: false,
+      sortValues: true,
+      numberedList: false,
+      minWidth: 150,
+    },
+    {
+      id: 'folios',
+      label: 'Folios',
+      desc: `
+        The number of folios (leaves).
+      `,
+      valueType: 'string',
+      makeLink: false,
+      externalLink: false,
+      sortValues: true,
+      numberedList: false,
+      minWidth: 150,
+    },
+    {
+      id: 'lines',
+      label: 'Lines',
+      desc: `
+        The number of lines in a text block. Left blank if the number of lines
+        occurring throughout the manuscript is too irregular to be a useful
+        descriptor for searching.
+      `,
+      valueType: 'string',
+      makeLink: false,
+      externalLink: false,
+      sortValues: true,
+      numberedList: false,
+      minWidth: 150,
+    },
+    {
+      id: 'columns',
+      label: 'Columns',
+      desc: `
+        The number of columns. Left blank if the number of columns
+        occurring throughout the manuscript is too irregular to be a useful
+        descriptor for searching.
+      `,
+      valueType: 'string',
+      makeLink: false,
+      externalLink: false,
+      sortValues: true,
+      numberedList: false,
+      minWidth: 150,
+    },
+    {
+      id: 'miniatures',
+      label: 'Miniatures',
+      desc: `
+        The number of miniatures.
+      `,
+      valueType: 'string',
+      makeLink: false,
+      externalLink: false,
+      sortValues: true,
+      numberedList: false,
+      minWidth: 150,
+    },
+    {
+      id: 'decoratedInitials',
+      label: 'Decorated initials',
+      desc: `
+        The number of miniatures.
+      `,
+      valueType: 'string',
+      makeLink: false,
+      externalLink: false,
+      sortValues: true,
+      numberedList: false,
+      minWidth: 150,
+    },
+
+
+
+
+
+
+
     {
       id: 'source',
       label: 'Source',
diff --git a/src/client/reducers/manuscriptsFacets.js b/src/client/reducers/manuscriptsFacets.js
index fa05313c..553d97b8 100644
--- a/src/client/reducers/manuscriptsFacets.js
+++ b/src/client/reducers/manuscriptsFacets.js
@@ -148,6 +148,7 @@ export const INITIAL_STATE = {
       uriFilter: null,
       priority: 5
     },
+    // event only as a result table column
     owner: {
       id: 'owner',
       label: 'Owner',
@@ -186,7 +187,7 @@ export const INITIAL_STATE = {
     },
     transferOfCustodyPlace: {
       id: 'transferOfCustodyPlace',
-      label: 'Transfer of custody place',
+      label: 'Transfer of Custody Place',
       //predicate: defined in backend
       distinctValueCount: 0,
       values: [],
@@ -206,7 +207,7 @@ export const INITIAL_STATE = {
     },
     transferOfCustodyTimespan: {
       id: 'transferOfCustodyTimespan',
-      label: 'Transfer of custody date',
+      label: 'Transfer of Custody Date',
       //predicate: defined in backend
       distinctValueCount: 0,
       values: [],
-- 
GitLab