Merge branch 'fix-570'

Author: benel

features/item_consult.feature

@@ -11,3 +11,23 @@ Scénario:
   Et une des rubriques de l'item est "Sacrifice d'Abraham"
   Et une des rubriques de l'item est "Isaac"
   Et une des rubriques de l'item est "1er quart XVIe"
+
+Scénario: l’utilisateur sélectionne un item sur un point de vue sur mobile
+
+  Soit "SJ 000" l'item affiché
+  Et l'utilisateur est sur mobile
+  Alors le titre de l'item affiché est "SJ 000"
+  Et la valeur de l'attribut "spatial" est "Église Saint-Jean-au-Marché, Troyes"
+  Et l'attribut "creator" est absent
+  Et l'attribut "created" est absent
+  Et la légende de l’image est "© Aurélien Bénel, 2016"
+
+Scénario: l’utilisateur sélectionne une photo en noir et blanc sur un point de vue sur mobile
+
+  Soit "AXN 009 B&W" l'item affiché
+  Et l'utilisateur est sur mobile
+  Alors le titre de l'item affiché est "AXN 009"
+  Et la valeur de l'attribut "spatial" est "Église Saint-Loup, Auxon"
+  Et l'attribut "creator" est absent
+  Et l'attribut "created" est absent
+  Et la légende de l’image est "1907/1914"

features/step_definitions/context.rb

@@ -81,3 +81,7 @@
   expect(page).to have_content "Modification du point de vue"
 end
 
+Soit("l'utilisateur est sur mobile") do
+  page.current_window.resize_to(320, 480)
+end
+

features/step_definitions/outcome.rb

@@ -60,3 +60,12 @@
 Alors('le point de vue {string} ne fait plus partie du portfolio') do |viewpoint|
   expect(page).not_to have_content viewpoint
 end
+
+Alors("la légende de l’image est {string}") do |legend|
+  expect(find(".Copyright")).to have_content legend
+end
+
+Alors("l'attribut {string} est absent") do |attribute|
+  expect(find(".Attributes")).not_to have_content attribute
+end
+

features/support/env.rb

@@ -27,6 +27,10 @@ def getURI(name)
       uri = "/item/Vitraux%20-%20Bénel/493f2a3702be0e9a67a6fd29700fd4dff8f6986a"
     when "SJ 001"
       uri = "/item/Vitraux%20-%20Bénel/8d27115e20f9439b627ac511ade9a05bbe509130"
+    when "SJ 000"
+      uri = "/item/Vitraux%20-%20Bénel/358d1c9f168311a9fe7bd7e2737c181514652e9d"
+    when "AXN 009 B&W"
+      uri = "/item/Vitraux - Recensement/d4dd4b9af975a07ecd0331307db8ca24daad7535"
   end
   return uri
 end

src/components/itemPage/Attribute.jsx

@@ -41,7 +41,7 @@ class Attribute extends React.Component {
 
   render() {
     return (
-      <div className="Attribute">
+      <div className={`Attribute ${this.props.className}`}>
         <div className="Key">
           {this.props.name}
         </div>

src/components/itemPage/Copyright.jsx

@@ -0,0 +1,30 @@
+import { Trans } from '@lingui/macro';
+
+/**
+ * A simple copyright notice.
+ */
+const Copyright = ({ creator, created }) => {
+  let copyrightText;
+  // Black and white pictures have a different date format and no author,
+  // which means we must handle them separately.
+  if (!creator) {
+    copyrightText = created;
+  } else {
+    let yearOrDate;
+    // If created is not a valid date
+    // (pictures with a creator may have a weird created field too).
+    if (isNaN(Date.parse(created))) {
+      yearOrDate = created;
+    } else {
+      yearOrDate = new Date(created).getFullYear().toString();
+    }
+    copyrightText = <Trans>© {creator}, {yearOrDate}</Trans>;
+  }
+  return (
+    <p className="Copyright pb-3">
+      {copyrightText}
+    </p>
+  );
+};
+
+export default Copyright;

src/components/itemPage/Item.jsx

@@ -4,15 +4,22 @@ import Hypertopic from 'hypertopic';
 import groupBy from 'json-groupby';
 import { Helmet } from 'react-helmet';
 import conf from '../../config.js';
-import { Items } from '../../model.js';
+import { HIDDEN_ON_MOBILE, Items } from '../../model.js';
 import Viewpoint from './Viewpoint.jsx';
 import Attribute from './Attribute.jsx';
 import Resource from './Resource.jsx';
 import Header from '../Header.jsx';
 import SameNameBlock from './SameNameBlock.jsx';
 import { DiscussionEmbed } from 'disqus-react';
 import { t, Trans } from '@lingui/macro';
+import Copyright from './Copyright.jsx';
 
+/**
+ * Gets a string representation of an object. If the object is an array,
+ * joins the representations of its elements with commas.
+ * @param {any} obj object to convert
+ * @returns {string} the string representation
+ */
 function getString(obj) {
   if (Array.isArray(obj)) {
     return obj.map(val => getString(val)).join(', ');
@@ -40,6 +47,7 @@ class Item extends Component {
   render() {
     let name = getString(this.state.item.name);
     let attributes = this._getAttributes();
+    const { creator, created } = this.state.item;
     let viewpoints = this._getViewpoints();
     let sameNameBlock = this._getSameNameBlock();
     const { visitMap } = this.state;
@@ -98,6 +106,9 @@ class Item extends Component {
               <div className="Subject">
                 <h2 className="h4 font-weight-bold text-center">{name}</h2>
                 <Resource href={this.state.item.resource} />
+                <div className="d-block d-sm-none">
+                  <Copyright creator={creator} created={created} />
+                </div>
               </div>
               <Comments appId={this.state.disqus} item={this.state.item} />
             </div>
@@ -109,10 +120,19 @@ class Item extends Component {
 
   _getAttributes() {
     return new Items([this.state.item]).getAttributes()
-      .map(x => (
-        <Attribute key={x[0]} name={x[0]} value={x[1]}
-          setAttribute={this._setAttribute} deleteAttribute={this._deleteAttribute}/>
-      ));
+      .map(attribute => {
+        const className = HIDDEN_ON_MOBILE.includes(attribute[0]) ? 'd-none d-sm-table-row' : '';
+        return (
+          <Attribute
+            className={className}
+            key={attribute[0]}
+            name={attribute[0]}
+            value={attribute[1]}
+            setAttribute={this._setAttribute}
+            deleteAttribute={this._deleteAttribute}
+          />
+        );
+      });
   }
 
   _getViewpoints() {

src/locales/en/messages.po

@@ -25,7 +25,7 @@ msgstr ""
 msgid "Ajouter"
 msgstr "Add"
 
-#: src/components/itemPage/Item.jsx:261
+#: src/components/itemPage/Item.jsx:279
 msgid "Ajouter un attribut et une valeur..."
 msgstr "Add an attribute and a value..."
 
@@ -37,7 +37,7 @@ msgstr "Add a topic..."
 msgid "Attention : Le point de vue sera définitivement supprimé."
 msgstr "Warning: The viewpoint will be permanently deleted."
 
-#: src/components/itemPage/Item.jsx:85
+#: src/components/itemPage/Item.jsx:92
 msgid "Attributs du document"
 msgstr "Document attributes"
 
@@ -57,7 +57,7 @@ msgstr "Create a viewpoint"
 msgid "Créer"
 msgstr ""
 
-#: src/components/itemPage/Item.jsx:82
+#: src/components/itemPage/Item.jsx:89
 msgid "Description"
 msgstr "Description"
 
@@ -115,16 +115,16 @@ msgstr "Viewpoints"
 msgid "Pseudonyme"
 msgstr "Pseudonym"
 
-#: src/components/portfolioPage/SearchBar.jsx:47
+#: src/components/portfolioPage/SearchBar.jsx:55
 msgid "Rechercher..."
 msgstr "Search..."
 
-#: src/components/itemPage/Item.jsx:75
+#: src/components/itemPage/Item.jsx:82
 #: src/components/viewpointPage/Outliner.jsx:37
 msgid "Retour à l'accueil"
 msgstr "Go back to home page"
 
-#: src/components/itemPage/Item.jsx:75
+#: src/components/itemPage/Item.jsx:82
 msgid "Retour à la visite"
 msgstr ""
 
@@ -156,11 +156,11 @@ msgstr "Delete"
 msgid "Tous les items"
 msgstr "All items"
 
-#: src/components/itemPage/Item.jsx:352
+#: src/components/itemPage/Item.jsx:370
 msgid "Voulez-vous réellement que l'item affiché ne soit plus décrit à l'aide de cette rubrique ?"
 msgstr "Remove this topic for this item?"
 
-#: src/components/itemPage/Item.jsx:263
+#: src/components/itemPage/Item.jsx:281
 msgid "attribut : valeur"
 msgstr "attribute: value"
 
@@ -179,3 +179,7 @@ msgstr "value"
 #: src/components/viewpointPage/Outliner.jsx:48
 msgid "Êtes-vous sûr de vouloir supprimer le point de vue ?"
 msgstr "Are you sure you want to delete the viewpoint?"
+
+#: src/components/itemPage/Copyright.jsx:21
+msgid "© {creator}, {yearOrDate}"
+msgstr "© {creator}, {yearOrDate}"

src/locales/fr/messages.po

@@ -25,7 +25,7 @@ msgstr "Adresse email"
 msgid "Ajouter"
 msgstr "Ajouter"
 
-#: src/components/itemPage/Item.jsx:261
+#: src/components/itemPage/Item.jsx:279
 msgid "Ajouter un attribut et une valeur..."
 msgstr "Ajouter un attribut et une valeur..."
 
@@ -37,7 +37,7 @@ msgstr "Ajouter une rubrique..."
 msgid "Attention : Le point de vue sera définitivement supprimé."
 msgstr "Attention : Le point de vue sera définitivement supprimé."
 
-#: src/components/itemPage/Item.jsx:85
+#: src/components/itemPage/Item.jsx:92
 msgid "Attributs du document"
 msgstr "Attributs du document"
 
@@ -57,7 +57,7 @@ msgstr "Création du point de vue"
 msgid "Créer"
 msgstr "Créer"
 
-#: src/components/itemPage/Item.jsx:82
+#: src/components/itemPage/Item.jsx:89
 msgid "Description"
 msgstr "Description"
 
@@ -115,16 +115,16 @@ msgstr "Points de vue"
 msgid "Pseudonyme"
 msgstr "Pseudonyme"
 
-#: src/components/portfolioPage/SearchBar.jsx:47
+#: src/components/portfolioPage/SearchBar.jsx:55
 msgid "Rechercher..."
 msgstr "Rechercher..."
 
-#: src/components/itemPage/Item.jsx:75
+#: src/components/itemPage/Item.jsx:82
 #: src/components/viewpointPage/Outliner.jsx:37
 msgid "Retour à l'accueil"
 msgstr "Retour à l'accueil"
 
-#: src/components/itemPage/Item.jsx:75
+#: src/components/itemPage/Item.jsx:82
 msgid "Retour à la visite"
 msgstr "Retour à la visite"
 
@@ -156,11 +156,11 @@ msgstr "Supprimer"
 msgid "Tous les items"
 msgstr "Tous les items"
 
-#: src/components/itemPage/Item.jsx:352
+#: src/components/itemPage/Item.jsx:370
 msgid "Voulez-vous réellement que l'item affiché ne soit plus décrit à l'aide de cette rubrique ?"
 msgstr "Voulez-vous réellement que l'item affiché ne soit plus décrit à l'aide de cette rubrique ?"
 
-#: src/components/itemPage/Item.jsx:263
+#: src/components/itemPage/Item.jsx:281
 msgid "attribut : valeur"
 msgstr "attribut : valeur"
 
@@ -179,3 +179,7 @@ msgstr "valeur"
 #: src/components/viewpointPage/Outliner.jsx:48
 msgid "Êtes-vous sûr de vouloir supprimer le point de vue ?"
 msgstr "Êtes-vous sûr de vouloir supprimer le point de vue ?"
+
+#: src/components/itemPage/Copyright.jsx:21
+msgid "© {creator}, {yearOrDate}"
+msgstr "© {creator}, {yearOrDate}"

src/model.js

@@ -23,6 +23,13 @@ class Topics {
 
 }
 
+/**
+ * When on mobile,
+ * the attributes in this list shouldn't be displayed in the left sidebar,
+ * because they are already displayed elsewhere in the page.
+ */
+const HIDDEN_ON_MOBILE = ['creator', 'created'];
+
 class Items {
 
   constructor(items) {
@@ -51,4 +58,4 @@ class Items {
     .map(([_, value]) => value);
 }
 
-export {Topics, Items};
+export {Topics, Items, HIDDEN_ON_MOBILE};