fix: review piero

RedTurtle · Nov 14, 2023 · e47a156 · e47a156
1 parent db12d09
commit e47a156
Show file tree

Hide file tree

Showing 3 changed files with 8 additions and 11 deletions.
diff --git a/src/components/ItaliaTheme/Blocks/Listing/CardWithImageTemplate.jsx b/src/components/ItaliaTheme/Blocks/Listing/CardWithImageTemplate.jsx
@@ -1,6 +1,6 @@
 /*
  * componente per visulizzare un CT "Persona" nei Listing o in aclune pagine
-*/
+ */
 import React from 'react';
 import PropTypes from 'prop-types';
 import cx from 'classnames';
@@ -129,8 +129,6 @@ const CardWithImageTemplate = (props) => {
                         className={cx('img-responsive-wrapper', {
                           'natural-image-size': natural_image_size,
                         })}
-                        // TODO: spostare in scss ?
-                        style={{width: 'inherit'}}
                       >
                         <div className="img-responsive img-responsive-panoramic">
                           <figure className="img-wrapper">{image}</figure>

diff --git a/src/components/ItaliaTheme/View/Commons/Events.jsx b/src/components/ItaliaTheme/View/Commons/Events.jsx
@@ -1,6 +1,3 @@
-/*
- * TODO: ....
- */
 import PropTypes from 'prop-types';
 import { defineMessages, useIntl } from 'react-intl';
 import {
@@ -122,4 +119,3 @@ Events.propTypes = {
   title: PropTypes.string,
   folder_name: PropTypes.string,
 };
-
diff --git a/src/components/ItaliaTheme/View/Commons/PageHeader/PageHeader.jsx b/src/components/ItaliaTheme/View/Commons/PageHeader/PageHeader.jsx
@@ -1,9 +1,5 @@
 /*
  * Page Header Component used in most of the views
- * props:
- * - content: content object
- * - imageinheader: boolean, if true, show image in header (eg. in PersonaView)
- * - imageinheader_field: string, field name of the image to show in header (eg. 'foto_persona')
  */
 import cx from 'classnames';
 import PropTypes from 'prop-types';
@@ -37,9 +33,16 @@ const messages = defineMessages({
 });
 
 /**
+ * PageHeader view component class.
+ * @function PageHeader
  * @params content {object} content object
  * @params imageinheader {boolean} if true, show image in header (eg. in PersonaView)
  * @params imageinheader_field {string} field name of the image to show in header (eg. 'foto_persona')
+ * @params readingtime {number} reading time in minutes
+ * @params showreadingtime {boolean} show or hide reading time
+ * @params showdates {boolean} show or hide dates in header
+ * @params showtassonomiaargomenti {boolean} show or hide argomenti in header
+ * @returns {string} Markup of the component.
  */
 const PageHeader = (props) => {
   const {