cozy-viewer 7.0.2 → 8.0.0

package/CHANGELOG.md

@@ -3,6 +3,33 @@
 All notable changes to this project will be documented in this file.
 See [Conventional Commits](https://conventionalcommits.org) for commit guidelines.
 
+# [8.0.0](https://github.com/cozy/cozy-libs/compare/cozy-viewer@7.0.2...cozy-viewer@8.0.0) (2024-12-05)
+
+
+### Bug Fixes
+
+* **Viewer:** It crashed for file without metadata ([85a25dd](https://github.com/cozy/cozy-libs/commit/85a25dd14d929f5326ba8d963e5929a2a3126771))
+* **Viewer:** Last qualification divider wasn't displayed correctly ([2c24870](https://github.com/cozy/cozy-libs/commit/2c2487066866510f2352cb3a8aac32a0a730beb1))
+* **Viewer:** Now hide qualification list if not relevant ([8deaa4a](https://github.com/cozy/cozy-libs/commit/8deaa4a4dd64c0cddeabde9218c2f8e356dbc52e))
+* **Viewer:** Now qualification divider use correct tag ([09f29d1](https://github.com/cozy/cozy-libs/commit/09f29d1e6d35fc759e3b45ebc2c25f2e7e4f7c9a))
+* **Viewer:** Now show correctly the owner in sharing block ([b88e24a](https://github.com/cozy/cozy-libs/commit/b88e24a707f619e24c6bb174107ffdb6d6f28668))
+* **Viewer:** Sharing item style in panel wasn't correct ([7f76212](https://github.com/cozy/cozy-libs/commit/7f76212bfaf7a2f4b51c4001597367592c141456))
+
+
+### Features
+
+* **Viewer:** Show always panel, not just for certified/connected files ([9bc42e7](https://github.com/cozy/cozy-libs/commit/9bc42e76735ff4da9e10c3a71c5c0e4443f2bfe5))
+* **Viewer:** Upgrade peerDep ([1e199ec](https://github.com/cozy/cozy-libs/commit/1e199ecbfd9dacc59224b70106963a212ef8b2b1))
+
+
+### BREAKING CHANGES
+
+* **Viewer:** You must have `cozy-sharing >= 16.11.0` and `cozy-harvest-lib >= 31.0.5`
+
+
+
+
+
 ## [7.0.2](https://github.com/cozy/cozy-libs/compare/cozy-viewer@7.0.1...cozy-viewer@7.0.2) (2024-12-05)
 
 **Note:** Version bump only for package cozy-viewer

package/dist/Footer/FooterContent.d.ts

@@ -4,7 +4,7 @@ declare function FooterContent({ file, toolbarRef, children, isPublic }: {
     toolbarRef: any;
     children: any;
     isPublic: any;
-}): JSX.Element | null;
+}): JSX.Element;
 declare namespace FooterContent {
     namespace propTypes {
         const file: PropTypes.Validator<object>;

package/dist/Footer/FooterContent.js

@@ -23,8 +23,6 @@ var _helpers = require("./helpers");
 
 var _PrintButton = _interopRequireDefault(require("../components/PrintButton"));
 
-var _helpers2 = require("../helpers");
-
 function _getRequireWildcardCache(nodeInterop) { if (typeof WeakMap !== "function") return null; var cacheBabelInterop = new WeakMap(); var cacheNodeInterop = new WeakMap(); return (_getRequireWildcardCache = function _getRequireWildcardCache(nodeInterop) { return nodeInterop ? cacheNodeInterop : cacheBabelInterop; })(nodeInterop); }
 
 function _interopRequireWildcard(obj, nodeInterop) { if (!nodeInterop && obj && obj.__esModule) { return obj; } if (obj === null || typeof obj !== "object" && typeof obj !== "function") { return { default: obj }; } var cache = _getRequireWildcardCache(nodeInterop); if (cache && cache.has(obj)) { return cache.get(obj); } var newObj = {}; var hasPropertyDescriptor = Object.defineProperty && Object.getOwnPropertyDescriptor; for (var key in obj) { if (key !== "default" && Object.prototype.hasOwnProperty.call(obj, key)) { var desc = hasPropertyDescriptor ? Object.getOwnPropertyDescriptor(obj, key) : null; if (desc && (desc.get || desc.set)) { Object.defineProperty(newObj, key, desc); } else { newObj[key] = obj[key]; } } } newObj.default = obj; if (cache) { cache.set(obj, newObj); } return newObj; }
@@ -79,34 +77,20 @@ var FooterContent = function FooterContent(_ref2) {
     isOpenMin: isPublic ? true : false,
     mediumHeightRatio: isPublic ? undefined : 0.5
   };
-
-  if ((0, _helpers2.isValidForPanel)({
-    file: file
-  })) {
-    return /*#__PURE__*/_react.default.createElement(_BottomSheet.default, {
-      toolbarProps: toolbarProps,
-      portalProps: {
-        disablePortal: true
-      },
-      settings: bottomSheetSettings
-    }, /*#__PURE__*/_react.default.createElement(_BottomSheet.BottomSheetHeader, {
-      className: (0, _classnames.default)('u-ph-1 u-pb-1', styles.bottomSheetHeader)
-    }, /*#__PURE__*/_react.default.createElement(FooterButtons, {
-      file: file,
-      FooterActionButtonsWithFile: FooterActionButtonsWithFile
-    })), /*#__PURE__*/_react.default.createElement(_BottomSheetContent.default, {
-      file: file,
-      isPublic: isPublic
-    }));
-  } // If `FooterActionButtons` hasn't children
-
-
-  if (!FooterActionButtonsWithFile) return null;
-  return /*#__PURE__*/_react.default.createElement("div", {
-    className: styles.footer
+  return /*#__PURE__*/_react.default.createElement(_BottomSheet.default, {
+    toolbarProps: toolbarProps,
+    portalProps: {
+      disablePortal: true
+    },
+    settings: bottomSheetSettings
+  }, /*#__PURE__*/_react.default.createElement(_BottomSheet.BottomSheetHeader, {
+    className: (0, _classnames.default)('u-ph-1 u-pb-1', styles.bottomSheetHeader)
   }, /*#__PURE__*/_react.default.createElement(FooterButtons, {
     file: file,
     FooterActionButtonsWithFile: FooterActionButtonsWithFile
+  })), /*#__PURE__*/_react.default.createElement(_BottomSheetContent.default, {
+    file: file,
+    isPublic: isPublic
   }));
 };
 

package/dist/Panel/Qualification.js

@@ -31,10 +31,14 @@ var _QualificationListItemOther = _interopRequireDefault(require("./Qualificatio
 
 var _QualificationListItemQualification = _interopRequireDefault(require("./QualificationListItemQualification"));
 
+var _helpers = require("./helpers");
+
 var _ExpirationAlert = _interopRequireDefault(require("../components/ExpirationAlert"));
 
 var _withViewerLocales = require("../hoc/withViewerLocales");
 
+var _useReferencedContactName = _interopRequireDefault(require("../hooks/useReferencedContactName"));
+
 function _getRequireWildcardCache(nodeInterop) { if (typeof WeakMap !== "function") return null; var cacheBabelInterop = new WeakMap(); var cacheNodeInterop = new WeakMap(); return (_getRequireWildcardCache = function _getRequireWildcardCache(nodeInterop) { return nodeInterop ? cacheNodeInterop : cacheBabelInterop; })(nodeInterop); }
 
 function _interopRequireWildcard(obj, nodeInterop) { if (!nodeInterop && obj && obj.__esModule) { return obj; } if (obj === null || typeof obj !== "object" && typeof obj !== "function") { return { default: obj }; } var cache = _getRequireWildcardCache(nodeInterop); if (cache && cache.has(obj)) { return cache.get(obj); } var newObj = {}; var hasPropertyDescriptor = Object.defineProperty && Object.getOwnPropertyDescriptor; for (var key in obj) { if (key !== "default" && Object.prototype.hasOwnProperty.call(obj, key)) { var desc = hasPropertyDescriptor ? Object.getOwnPropertyDescriptor(obj, key) : null; if (desc && (desc.get || desc.set)) { Object.defineProperty(newObj, key, desc); } else { newObj[key] = obj[key]; } } } newObj.default = obj; if (cache) { cache.set(obj, newObj); } return newObj; }
@@ -70,6 +74,11 @@ var Qualification = function Qualification(_ref) {
       optionFile = _useState2[0],
       setOptionFile = _useState2[1];
 
+  var _useReferencedContact = (0, _useReferencedContactName.default)(file),
+      contacts = _useReferencedContact.contacts;
+
+  var formattedContactValue = (0, _paper.formatContactValue)(contacts);
+
   var hideActionsMenu = function hideActionsMenu() {
     setOptionFile({
       id: '',
@@ -121,6 +130,8 @@ var Qualification = function Qualification(_ref) {
       return (_actionBtnRef$current = actionBtnRef.current[idx]) !== null && _actionBtnRef$current !== void 0 ? _actionBtnRef$current : /*#__PURE__*/(0, _react.createRef)();
     });
   }, [formattedMetadataQualification]);
+  var showMetadataList = formattedMetadataQualification.length !== 2 || formattedContactValue; // we have at minimum "qualification" and "contact" item
+
   return /*#__PURE__*/_react.default.createElement(_react.default.Fragment, null, (0, _paper.isExpiringSoon)(file) && !isExpirationAlertHidden(file) && /*#__PURE__*/_react.default.createElement(_ExpirationAlert.default, {
     file: file
   }), /*#__PURE__*/_react.default.createElement(_QualificationListItemQualification.default, {
@@ -130,12 +141,12 @@ var Qualification = function Qualification(_ref) {
     toggleActionsMenu: function toggleActionsMenu(val) {
       return _toggleActionsMenu(0, formattedMetadataQualification[0].name, val);
     }
-  }), /*#__PURE__*/_react.default.createElement(_List.default, null, formattedMetadataQualification.map(function (meta, idx) {
+  }), showMetadataList && /*#__PURE__*/_react.default.createElement(_List.default, null, formattedMetadataQualification.map(function (meta, idx) {
     var name = meta.name;
     if (name === 'qualification') return null;
     var metadataQualificationType = (0, _paper.getMetadataQualificationType)(name);
     var QualificationListItemComp = ComponentFromMetadataQualificationType[metadataQualificationType];
-    var isLastItem = idx === formattedMetadataQualification.length - 1;
+    var hideDivider = (0, _helpers.makeHideDivider)(formattedMetadataQualification, idx);
     return /*#__PURE__*/_react.default.createElement(_react.Fragment, {
       key: idx
     }, /*#__PURE__*/_react.default.createElement(QualificationListItemComp, {
@@ -145,7 +156,8 @@ var Qualification = function Qualification(_ref) {
       toggleActionsMenu: function toggleActionsMenu(val) {
         return _toggleActionsMenu(idx, name, val);
       }
-    }), !isLastItem && /*#__PURE__*/_react.default.createElement(_Divider.default, {
+    }), !hideDivider && /*#__PURE__*/_react.default.createElement(_Divider.default, {
+      component: "li",
       variant: "inset"
     }));
   }), optionFile.name && /*#__PURE__*/_react.default.createElement(_ActionMenuWrapper.default, {

package/dist/Panel/QualificationListItemContact.js

@@ -59,6 +59,26 @@ var QualificationListItemContact = function QualificationListItemContact(_ref) {
       optionFile = _useState2[0],
       setOptionFile = _useState2[1];
 
+  var _useReferencedContact = (0, _useReferencedContactName.default)(file),
+      contacts = _useReferencedContact.contacts,
+      isLoadingContacts = _useReferencedContact.isLoadingContacts;
+
+  if (isLoadingContacts) {
+    return /*#__PURE__*/_react.default.createElement(_ListItem.default, null, /*#__PURE__*/_react.default.createElement(_Spinner.default, {
+      color: "var(--secondaryTextColor)"
+    }));
+  }
+
+  var formattedValue = (0, _paper.formatContactValue)(contacts);
+
+  if (!formattedValue) {
+    return null;
+  }
+
+  var formattedTitle = (0, _paper.getTranslatedNameForContact)({
+    lang: lang
+  });
+
   var hideActionsMenu = function hideActionsMenu() {
     return setOptionFile({
       name: '',
@@ -79,25 +99,6 @@ var QualificationListItemContact = function QualificationListItemContact(_ref) {
     });
   };
 
-  var _useReferencedContact = (0, _useReferencedContactName.default)(file),
-      contacts = _useReferencedContact.contacts,
-      isLoadingContacts = _useReferencedContact.isLoadingContacts;
-
-  if (isLoadingContacts) {
-    return /*#__PURE__*/_react.default.createElement(_ListItem.default, null, /*#__PURE__*/_react.default.createElement(_Spinner.default, {
-      color: "var(--secondaryTextColor)"
-    }));
-  }
-
-  var formattedTitle = (0, _paper.getTranslatedNameForContact)({
-    lang: lang
-  });
-  var formattedValue = (0, _paper.formatContactValue)(contacts);
-
-  if (!isLoadingContacts && !formattedValue) {
-    return null;
-  }
-
   return /*#__PURE__*/_react.default.createElement(_react.default.Fragment, null, /*#__PURE__*/_react.default.createElement(_ListItem.default, null, /*#__PURE__*/_react.default.createElement(_ListItemIcon.default, null, /*#__PURE__*/_react.default.createElement(_Icon.default, {
     icon: _People.default
   })), /*#__PURE__*/_react.default.createElement(_QualificationListItemText.default, {

package/dist/Panel/Sharing.js

@@ -15,8 +15,6 @@ var _react = _interopRequireWildcard(require("react"));
 
 var _cozySharing = require("cozy-sharing");
 
-var _Divider = _interopRequireDefault(require("cozy-ui/transpiled/react/Divider"));
-
 var _Icon = _interopRequireDefault(require("cozy-ui/transpiled/react/Icon"));
 
 var _Right = _interopRequireDefault(require("cozy-ui/transpiled/react/Icons/Right"));
@@ -56,7 +54,9 @@ var Sharing = function Sharing(_ref) {
   var recipients = getRecipients(file._id);
   var permissions = getDocumentPermissions(file._id);
   var link = getSharingLink(file._id);
-  return /*#__PURE__*/_react.default.createElement(_react.default.Fragment, null, /*#__PURE__*/_react.default.createElement(_List.default, null, /*#__PURE__*/_react.default.createElement(_ListItem.default, {
+  return /*#__PURE__*/_react.default.createElement(_react.default.Fragment, null, /*#__PURE__*/_react.default.createElement(_ListItem.default, {
+    size: "large",
+    divider: true,
     button: true,
     onClick: function onClick() {
       return setShowModal(true);
@@ -71,18 +71,19 @@ var Sharing = function Sharing(_ref) {
     }
   }), /*#__PURE__*/_react.default.createElement(_ListItemIcon.default, null, /*#__PURE__*/_react.default.createElement(_Icon.default, {
     icon: _Right.default
-  })))), /*#__PURE__*/_react.default.createElement(_Divider.default, null), /*#__PURE__*/_react.default.createElement(_List.default, null, /*#__PURE__*/_react.default.createElement(_cozySharing.LinkRecipientLite, {
+  }))), /*#__PURE__*/_react.default.createElement(_List.default, null, /*#__PURE__*/_react.default.createElement(_cozySharing.LinkRecipientLite, {
     permissions: permissions,
     link: link
-  }), recipients.map(function (recipient) {
+  }), recipients.length > 0 ? recipients.map(function (recipient) {
     return /*#__PURE__*/_react.default.createElement(_cozySharing.MemberRecipientLite, {
       key: recipient.index,
       recipient: recipient,
       isOwner: isOwner(file._id)
     });
-  })), showModal && /*#__PURE__*/_react.default.createElement(_cozySharing.ShareModal, {
+  }) : /*#__PURE__*/_react.default.createElement(_cozySharing.OwnerRecipientDefaultLite, null)), showModal && /*#__PURE__*/_react.default.createElement(_cozySharing.ShareModal, {
     document: file,
     documentType: "Files",
+    sharingDesc: "",
     onClose: function onClose() {
       return setShowModal(false);
     }

package/dist/Panel/Summary.js

@@ -56,6 +56,8 @@ function _getRequireWildcardCache(nodeInterop) { if (typeof WeakMap !== "functio
 function _interopRequireWildcard(obj, nodeInterop) { if (!nodeInterop && obj && obj.__esModule) { return obj; } if (obj === null || typeof obj !== "object" && typeof obj !== "function") { return { default: obj }; } var cache = _getRequireWildcardCache(nodeInterop); if (cache && cache.has(obj)) { return cache.get(obj); } var newObj = {}; var hasPropertyDescriptor = Object.defineProperty && Object.getOwnPropertyDescriptor; for (var key in obj) { if (key !== "default" && Object.prototype.hasOwnProperty.call(obj, key)) { var desc = hasPropertyDescriptor ? Object.getOwnPropertyDescriptor(obj, key) : null; if (desc && (desc.get || desc.set)) { Object.defineProperty(newObj, key, desc); } else { newObj[key] = obj[key]; } } } newObj.default = obj; if (cache) { cache.set(obj, newObj); } return newObj; }
 
 var Summary = function Summary(_ref) {
+  var _file$metadata;
+
   var file = _ref.file,
       t = _ref.t;
 
@@ -75,7 +77,7 @@ var Summary = function Summary(_ref) {
       showAlert = _useAlert.showAlert;
 
   var label = t('Viewer.panel.summary');
-  var value = file.metadata.description;
+  var value = (_file$metadata = file.metadata) === null || _file$metadata === void 0 ? void 0 : _file$metadata.description;
   var actions = (0, _Actions.makeActions)([_Actions.copyToClipboard, _Actions.editAttribute]);
 
   var handleClick = /*#__PURE__*/function () {

package/dist/Panel/SummaryDialog.js

@@ -38,11 +38,13 @@ function ownKeys(object, enumerableOnly) { var keys = Object.keys(object); if (O
 function _objectSpread(target) { for (var i = 1; i < arguments.length; i++) { var source = null != arguments[i] ? arguments[i] : {}; i % 2 ? ownKeys(Object(source), !0).forEach(function (key) { (0, _defineProperty2.default)(target, key, source[key]); }) : Object.getOwnPropertyDescriptors ? Object.defineProperties(target, Object.getOwnPropertyDescriptors(source)) : ownKeys(Object(source)).forEach(function (key) { Object.defineProperty(target, key, Object.getOwnPropertyDescriptor(source, key)); }); } return target; }
 
 var SummaryDialog = function SummaryDialog(_ref) {
+  var _file$metadata;
+
   var file = _ref.file,
       t = _ref.t,
       onClose = _ref.onClose;
 
-  var _useState = (0, _react.useState)(file.metadata.description),
+  var _useState = (0, _react.useState)((_file$metadata = file.metadata) === null || _file$metadata === void 0 ? void 0 : _file$metadata.description),
       _useState2 = (0, _slicedToArray2.default)(_useState, 2),
       value = _useState2[0],
       setValue = _useState2[1];

package/dist/Panel/helpers.d.ts

@@ -2,3 +2,4 @@ export function makeFormat(file: any): string;
 export function makeDate(lang: string): string;
 export function makeSize(bytes: number): string;
 export function makePath(file: any): string;
+export function makeHideDivider(formattedMetadataQualification: array, idx: number): boolean;

package/dist/Panel/helpers.js

@@ -3,7 +3,7 @@
 Object.defineProperty(exports, "__esModule", {
   value: true
 });
-exports.makeSize = exports.makePath = exports.makeFormat = exports.makeDate = void 0;
+exports.makeSize = exports.makePath = exports.makeHideDivider = exports.makeFormat = exports.makeDate = void 0;
 
 var _file = require("cozy-client/dist/models/file");
 
@@ -61,5 +61,22 @@ var makePath = function makePath(file) {
 
   return (_file$path = file.path) === null || _file$path === void 0 ? void 0 : _file$path.replace("/".concat(file.name), '');
 };
+/**
+ *
+ * @param {array} formattedMetadataQualification
+ * @param {number} idx
+ * @returns {boolean}
+ */
+
+
+exports.makePath = makePath;
+
+var makeHideDivider = function makeHideDivider(formattedMetadataQualification, idx) {
+  var lastItem = formattedMetadataQualification.at(-1);
+  var isLastItem = idx === formattedMetadataQualification.length - 1;
+  var isSecondLastItem = idx === formattedMetadataQualification.length - 2;
+  var hideDivider = isLastItem || isSecondLastItem && lastItem.name === 'contact';
+  return hideDivider;
+};
 
-exports.makePath = makePath;
+exports.makeHideDivider = makeHideDivider;

package/dist/Panel/helpers.spec.d.ts

@@ -0,0 +1 @@
+export {};

package/dist/ViewerContainer.js

@@ -37,8 +37,6 @@ var _Viewer = _interopRequireDefault(require("./Viewer"));
 
 var _ViewerInformationsWrapper = _interopRequireDefault(require("./ViewerInformationsWrapper"));
 
-var _helpers = require("./helpers");
-
 var _locales = require("./locales");
 
 var _proptypes2 = require("./proptypes");
@@ -83,9 +81,7 @@ var ViewerContainer = function ViewerContainer(props) {
   var fileCount = files.length;
   var hasPrevious = currentIndex > 0;
   var hasNext = currentIndex < fileCount - 1;
-  var validForPanel = (0, _helpers.isValidForPanel)({
-    file: currentFile
-  }) && isDesktop && !disablePanel;
+  var validForPanel = isDesktop && !disablePanel;
 
   var componentsPropsWithDefault = _objectSpread(_objectSpread({}, componentsProps), {}, {
     toolbarProps: _objectSpread({

package/dist/components/Footer.js

@@ -19,7 +19,6 @@ var styles = {
   "viewer-nav--next": "styles__viewer-nav--next___3mBHi",
   "viewer-controls": "styles__viewer-controls___2lf-z",
   "--expanded": "styles__--expanded___1sp4Q",
-  "viewer-controls--display-content-top": "styles__viewer-controls--display-content-top___3wszk",
   "viewer-toolbar": "styles__viewer-toolbar___1GLOA",
   "viewer-toolbar--hidden": "styles__viewer-toolbar--hidden___Zq97m",
   "viewer-footer": "styles__viewer-footer___iGv8J"

package/dist/components/Navigation.js

@@ -27,7 +27,6 @@ var styles = {
   "viewer-nav--next": "styles__viewer-nav--next___3mBHi",
   "viewer-controls": "styles__viewer-controls___2lf-z",
   "--expanded": "styles__--expanded___1sp4Q",
-  "viewer-controls--display-content-top": "styles__viewer-controls--display-content-top___3wszk",
   "viewer-toolbar": "styles__viewer-toolbar___1GLOA",
   "viewer-toolbar--hidden": "styles__viewer-toolbar--hidden___Zq97m",
   "viewer-footer": "styles__viewer-footer___iGv8J"

package/dist/components/Toolbar.js

@@ -55,7 +55,6 @@ var styles = {
   "viewer-nav--next": "styles__viewer-nav--next___3mBHi",
   "viewer-controls": "styles__viewer-controls___2lf-z",
   "--expanded": "styles__--expanded___1sp4Q",
-  "viewer-controls--display-content-top": "styles__viewer-controls--display-content-top___3wszk",
   "viewer-toolbar": "styles__viewer-toolbar___1GLOA",
   "viewer-toolbar--hidden": "styles__viewer-toolbar--hidden___Zq97m",
   "viewer-footer": "styles__viewer-footer___iGv8J"

package/dist/components/ViewerControls.js

@@ -41,8 +41,6 @@ var _Navigation = _interopRequireDefault(require("./Navigation"));
 
 var _Toolbar = _interopRequireDefault(require("./Toolbar"));
 
-var _helpers = require("../helpers");
-
 var _proptypes = require("../proptypes");
 
 function _getRequireWildcardCache(nodeInterop) { if (typeof WeakMap !== "function") return null; var cacheBabelInterop = new WeakMap(); var cacheNodeInterop = new WeakMap(); return (_getRequireWildcardCache = function _getRequireWildcardCache(nodeInterop) { return nodeInterop ? cacheNodeInterop : cacheBabelInterop; })(nodeInterop); }
@@ -61,7 +59,6 @@ var styles = {
   "viewer-nav--next": "styles__viewer-nav--next___3mBHi",
   "viewer-controls": "styles__viewer-controls___2lf-z",
   "--expanded": "styles__--expanded___1sp4Q",
-  "viewer-controls--display-content-top": "styles__viewer-controls--display-content-top___3wszk",
   "viewer-toolbar": "styles__viewer-toolbar___1GLOA",
   "viewer-toolbar--hidden": "styles__viewer-toolbar--hidden___Zq97m",
   "viewer-footer": "styles__viewer-footer___iGv8J"
@@ -200,11 +197,8 @@ var ViewerControls = /*#__PURE__*/function (_Component) {
           toolbarRef = toolbarProps.toolbarRef,
           showFilePath = toolbarProps.showFilePath;
       var hidden = this.state.hidden;
-      var shouldDisplayContentTop = (0, _helpers.isValidForPanel)({
-        file: file
-      });
       return /*#__PURE__*/_react.default.createElement("div", {
-        className: (0, _classnames.default)(styles['viewer-controls'], (_cx = {}, (0, _defineProperty2.default)(_cx, styles['viewer-controls--expanded'], expanded), (0, _defineProperty2.default)(_cx, styles['viewer-controls--display-content-top'], shouldDisplayContentTop), (0, _defineProperty2.default)(_cx, classes.viewerControlsWithInfo, showInfoPanel), _cx)),
+        className: (0, _classnames.default)(styles['viewer-controls'], (_cx = {}, (0, _defineProperty2.default)(_cx, styles['viewer-controls--expanded'], expanded), (0, _defineProperty2.default)(_cx, classes.viewerControlsWithInfo, showInfoPanel), _cx)),
         ref: function ref(wrapped) {
           _this3.wrapped = wrapped;
         }

package/dist/helpers.d.ts

@@ -1,7 +1,4 @@
 export function getCurrentModel(metadataName: any): "information" | "contact" | "page" | undefined;
-export function isValidForPanel({ file }: {
-    file: IOCozyFile;
-}): boolean;
 export function isFileEncrypted(file: any): any;
 export function formatDate({ f, lang, date }: {
     f: any;

package/dist/helpers.js

@@ -5,7 +5,7 @@ var _interopRequireDefault = require("@babel/runtime/helpers/interopRequireDefau
 Object.defineProperty(exports, "__esModule", {
   value: true
 });
-exports.removeFilenameFromPath = exports.normalizeAndSpreadAttributes = exports.makeWebLink = exports.isValidForPanel = exports.isFileEncrypted = exports.isEditableAttribute = exports.getCurrentModel = exports.formatDate = exports.buildEditAttributePath = void 0;
+exports.removeFilenameFromPath = exports.normalizeAndSpreadAttributes = exports.makeWebLink = exports.isFileEncrypted = exports.isEditableAttribute = exports.getCurrentModel = exports.formatDate = exports.buildEditAttributePath = void 0;
 
 var _defineProperty2 = _interopRequireDefault(require("@babel/runtime/helpers/defineProperty"));
 
@@ -33,37 +33,19 @@ var getCurrentModel = function getCurrentModel(metadataName) {
  * @property {string} type - doctype of the document
  */
 
-/**
- * Checks if the file matches one of the following conditions:
- * - Is certified
- * - Is Qualified
- * - From a Connector
- *
- * @param {object} param
- * @param {IOCozyFile} param.file
- * @returns {boolean}
- */
-
 
 exports.getCurrentModel = getCurrentModel;
 
-var isValidForPanel = function isValidForPanel(_ref) {
-  var file = _ref.file;
-  return (0, _file.hasCertifications)(file) || (0, _file.hasQualifications)(file) || (0, _file.isFromKonnector)(file);
-};
-
-exports.isValidForPanel = isValidForPanel;
-
 var isFileEncrypted = function isFileEncrypted(file) {
   return (0, _file.isEncrypted)(file);
 };
 
 exports.isFileEncrypted = isFileEncrypted;
 
-var formatDate = function formatDate(_ref2) {
-  var f = _ref2.f,
-      lang = _ref2.lang,
-      date = _ref2.date;
+var formatDate = function formatDate(_ref) {
+  var f = _ref.f,
+      lang = _ref.lang,
+      date = _ref.date;
 
   if (lang === 'en') {
     return f(date, 'MM/DD/YYYY');
@@ -113,10 +95,10 @@ var normalizeAndSpreadAttributes = function normalizeAndSpreadAttributes(rawFile
 
 exports.normalizeAndSpreadAttributes = normalizeAndSpreadAttributes;
 
-var makeWebLink = function makeWebLink(_ref3) {
-  var client = _ref3.client,
-      slug = _ref3.slug,
-      path = _ref3.path;
+var makeWebLink = function makeWebLink(_ref2) {
+  var client = _ref2.client,
+      slug = _ref2.slug,
+      path = _ref2.path;
 
   try {
     var cozyURL = new URL(client.getStackClient().uri);

package/dist/stylesheet.css

@@ -195,16 +195,16 @@
   width: 100%;
   height: 100%;
 }
-.styles__viewer-controls___2lf-z.styles__--expanded___1sp4Q .styles__viewer-nav___3V03k {
-  margin-top: 0;
-  width: 40%;
-}
 @media (max-width: 63.938rem) {
-  .styles__viewer-controls--display-content-top___3wszk {
+  .styles__viewer-controls___2lf-z {
     justify-content: flex-start;
     padding-top: 4.5rem;
   }
 }
+.styles__viewer-controls___2lf-z.styles__--expanded___1sp4Q .styles__viewer-nav___3V03k {
+  margin-top: 0;
+  width: 40%;
+}
 .styles__viewer-toolbar___1GLOA {
   position: absolute;
   top: 0;

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "cozy-viewer",
-  "version": "7.0.2",
+  "version": "8.0.0",
   "description": "Cozy-Viewer provides a component to show files in a viewer.",
   "main": "dist/index.js",
   "license": "MIT",
@@ -27,7 +27,7 @@
     "babel-preset-cozy-app": "^2.8.1",
     "cozy-client": "50.0.0",
     "cozy-device-helper": "2.0.0",
-    "cozy-harvest-lib": "^31.0.5",
+    "cozy-harvest-lib": "^31.0.6",
     "cozy-intent": "^2.29.1",
     "cozy-logger": "^1.16.1",
     "cozy-sharing": "^16.11.0",
@@ -52,13 +52,13 @@
   "peerDependencies": {
     "cozy-client": ">=50.0.0",
     "cozy-device-helper": ">=2.0.0",
-    "cozy-harvest-lib": ">=30.8.0",
+    "cozy-harvest-lib": ">=31.0.5",
     "cozy-intent": ">=2.26.0",
     "cozy-logger": ">=1.9.0",
-    "cozy-sharing": ">=16.8.0",
+    "cozy-sharing": ">=16.11.0",
     "cozy-ui": ">=113.8.0",
     "react": ">=16.12.0",
     "react-dom": ">=16.12.0"
   },
-  "gitHead": "99210fddf08b4701afcf0390c0042d3f7cfb97ec"
+  "gitHead": "5e8d5bd6ec3ba9f49fdacf8669cf54fb6cd8fc90"
 }

package/src/Footer/FooterContent.jsx

@@ -10,7 +10,6 @@ import { makeStyles } from 'cozy-ui/transpiled/react/styles'
 import BottomSheetContent from './BottomSheetContent'
 import { extractChildrenCompByName } from './helpers'
 import PrintButton from '../components/PrintButton'
-import { isValidForPanel } from '../helpers'
 
 const FooterButtons = ({
   file,
@@ -56,36 +55,22 @@ const FooterContent = ({ file, toolbarRef, children, isPublic }) => {
     mediumHeightRatio: isPublic ? undefined : 0.5
   }
 
-  if (isValidForPanel({ file })) {
-    return (
-      <BottomSheet
-        toolbarProps={toolbarProps}
-        portalProps={{ disablePortal: true }}
-        settings={bottomSheetSettings}
-      >
-        <BottomSheetHeader
-          className={cx('u-ph-1 u-pb-1', styles.bottomSheetHeader)}
-        >
-          <FooterButtons
-            file={file}
-            FooterActionButtonsWithFile={FooterActionButtonsWithFile}
-          />
-        </BottomSheetHeader>
-        <BottomSheetContent file={file} isPublic={isPublic} />
-      </BottomSheet>
-    )
-  }
-
-  // If `FooterActionButtons` hasn't children
-  if (!FooterActionButtonsWithFile) return null
-
   return (
-    <div className={styles.footer}>
-      <FooterButtons
-        file={file}
-        FooterActionButtonsWithFile={FooterActionButtonsWithFile}
-      />
-    </div>
+    <BottomSheet
+      toolbarProps={toolbarProps}
+      portalProps={{ disablePortal: true }}
+      settings={bottomSheetSettings}
+    >
+      <BottomSheetHeader
+        className={cx('u-ph-1 u-pb-1', styles.bottomSheetHeader)}
+      >
+        <FooterButtons
+          file={file}
+          FooterActionButtonsWithFile={FooterActionButtonsWithFile}
+        />
+      </BottomSheetHeader>
+      <BottomSheetContent file={file} isPublic={isPublic} />
+    </BottomSheet>
   )
 }
 

package/src/Panel/Qualification.jsx

@@ -12,6 +12,7 @@ import {
   isExpiringSoon,
   formatMetadataQualification,
   KNOWN_BILLS_ATTRIBUTES_NAMES,
+  formatContactValue,
   getMetadataQualificationType
 } from 'cozy-client/dist/models/paper'
 import Divider from 'cozy-ui/transpiled/react/Divider'
@@ -23,8 +24,10 @@ import QualificationListItemDate from './QualificationListItemDate'
 import QualificationListItemInformation from './QualificationListItemInformation'
 import QualificationListItemOther from './QualificationListItemOther'
 import QualificationListItemQualification from './QualificationListItemQualification'
+import { makeHideDivider } from './helpers'
 import ExpirationAlert from '../components/ExpirationAlert'
 import { withViewerLocales } from '../hoc/withViewerLocales'
+import useReferencedContactName from '../hooks/useReferencedContactName'
 
 const ComponentFromMetadataQualificationType = {
   contact: QualificationListItemContact,
@@ -46,6 +49,8 @@ const Qualification = ({ file }) => {
     name: '',
     value: ''
   })
+  const { contacts } = useReferencedContactName(file)
+  const formattedContactValue = formatContactValue(contacts)
 
   const hideActionsMenu = () => {
     setOptionFile({ id: '', name: '', value: '' })
@@ -83,6 +88,9 @@ const Qualification = ({ file }) => {
     )
   }, [formattedMetadataQualification])
 
+  const showMetadataList =
+    formattedMetadataQualification.length !== 2 || formattedContactValue // we have at minimum "qualification" and "contact" item
+
   return (
     <>
       {isExpiringSoon(file) && !isExpirationAlertHidden(file) && (
@@ -96,38 +104,43 @@ const Qualification = ({ file }) => {
           toggleActionsMenu(0, formattedMetadataQualification[0].name, val)
         }
       />
-      <List>
-        {formattedMetadataQualification.map((meta, idx) => {
-          const { name } = meta
-          if (name === 'qualification') return null
-
-          const metadataQualificationType = getMetadataQualificationType(name)
-          const QualificationListItemComp =
-            ComponentFromMetadataQualificationType[metadataQualificationType]
-
-          const isLastItem = idx === formattedMetadataQualification.length - 1
-
-          return (
-            <Fragment key={idx}>
-              <QualificationListItemComp
-                file={file}
-                ref={actionBtnRef.current[idx]}
-                formattedMetadataQualification={meta}
-                toggleActionsMenu={val => toggleActionsMenu(idx, name, val)}
-              />
-              {!isLastItem && <Divider variant="inset" />}
-            </Fragment>
-          )
-        })}
-        {optionFile.name && (
-          <ActionMenuWrapper
-            onClose={hideActionsMenu}
-            file={file}
-            optionFile={optionFile}
-            ref={actionBtnRef.current[optionFile.id]}
-          />
-        )}
-      </List>
+      {showMetadataList && (
+        <List>
+          {formattedMetadataQualification.map((meta, idx) => {
+            const { name } = meta
+            if (name === 'qualification') return null
+
+            const metadataQualificationType = getMetadataQualificationType(name)
+            const QualificationListItemComp =
+              ComponentFromMetadataQualificationType[metadataQualificationType]
+
+            const hideDivider = makeHideDivider(
+              formattedMetadataQualification,
+              idx
+            )
+
+            return (
+              <Fragment key={idx}>
+                <QualificationListItemComp
+                  file={file}
+                  ref={actionBtnRef.current[idx]}
+                  formattedMetadataQualification={meta}
+                  toggleActionsMenu={val => toggleActionsMenu(idx, name, val)}
+                />
+                {!hideDivider && <Divider component="li" variant="inset" />}
+              </Fragment>
+            )
+          })}
+          {optionFile.name && (
+            <ActionMenuWrapper
+              onClose={hideActionsMenu}
+              file={file}
+              optionFile={optionFile}
+              ref={actionBtnRef.current[optionFile.id]}
+            />
+          )}
+        </List>
+      )}
     </>
   )
 }

package/src/Panel/QualificationListItemContact.jsx

@@ -26,14 +26,6 @@ const QualificationListItemContact = ({ file }) => {
     name: '',
     value: ''
   })
-
-  const hideActionsMenu = () => setOptionFile({ name: '', value: '' })
-  const toggleActionsMenu = (name, value) =>
-    setOptionFile(prev => {
-      if (prev.value) return { name: '', value: '' }
-      return { name, value }
-    })
-
   const { contacts, isLoadingContacts } = useReferencedContactName(file)
 
   if (isLoadingContacts) {
@@ -44,13 +36,21 @@ const QualificationListItemContact = ({ file }) => {
     )
   }
 
-  const formattedTitle = getTranslatedNameForContact({ lang })
   const formattedValue = formatContactValue(contacts)
 
-  if (!isLoadingContacts && !formattedValue) {
+  if (!formattedValue) {
     return null
   }
 
+  const formattedTitle = getTranslatedNameForContact({ lang })
+
+  const hideActionsMenu = () => setOptionFile({ name: '', value: '' })
+  const toggleActionsMenu = (name, value) =>
+    setOptionFile(prev => {
+      if (prev.value) return { name: '', value: '' }
+      return { name, value }
+    })
+
   return (
     <>
       <ListItem>

package/src/Panel/Sharing.jsx

@@ -5,9 +5,9 @@ import {
   ShareModal,
   useSharingContext,
   MemberRecipientLite,
+  OwnerRecipientDefaultLite,
   LinkRecipientLite
 } from 'cozy-sharing'
-import Divider from 'cozy-ui/transpiled/react/Divider'
 import Icon from 'cozy-ui/transpiled/react/Icon'
 import RightIcon from 'cozy-ui/transpiled/react/Icons/Right'
 import List from 'cozy-ui/transpiled/react/List'
@@ -34,37 +34,39 @@ const Sharing = ({ file, t }) => {
 
   return (
     <>
-      <List>
-        <ListItem button onClick={() => setShowModal(true)}>
-          <ListItemText
-            primary={
-              <>
-                {t('Viewer.panel.sharing')}
-                {!allLoaded && <Spinner className="u-ml-half" noMargin />}
-              </>
-            }
-            primaryTypographyProps={{ variant: 'h6' }}
-          />
-          <ListItemIcon>
-            <Icon icon={RightIcon} />
-          </ListItemIcon>
-        </ListItem>
-      </List>
-      <Divider />
+      <ListItem size="large" divider button onClick={() => setShowModal(true)}>
+        <ListItemText
+          primary={
+            <>
+              {t('Viewer.panel.sharing')}
+              {!allLoaded && <Spinner className="u-ml-half" noMargin />}
+            </>
+          }
+          primaryTypographyProps={{ variant: 'h6' }}
+        />
+        <ListItemIcon>
+          <Icon icon={RightIcon} />
+        </ListItemIcon>
+      </ListItem>
       <List>
         <LinkRecipientLite permissions={permissions} link={link} />
-        {recipients.map(recipient => (
-          <MemberRecipientLite
-            key={recipient.index}
-            recipient={recipient}
-            isOwner={isOwner(file._id)}
-          />
-        ))}
+        {recipients.length > 0 ? (
+          recipients.map(recipient => (
+            <MemberRecipientLite
+              key={recipient.index}
+              recipient={recipient}
+              isOwner={isOwner(file._id)}
+            />
+          ))
+        ) : (
+          <OwnerRecipientDefaultLite />
+        )}
       </List>
       {showModal && (
         <ShareModal
           document={file}
           documentType="Files"
+          sharingDesc=""
           onClose={() => setShowModal(false)}
         />
       )}

package/src/Panel/Summary.jsx

@@ -31,7 +31,7 @@ const Summary = ({ file, t }) => {
   const { showAlert } = useAlert()
 
   const label = t('Viewer.panel.summary')
-  const value = file.metadata.description
+  const value = file.metadata?.description
   const actions = makeActions([copyToClipboard, editAttribute])
 
   const handleClick = async () => {

package/src/Panel/SummaryDialog.jsx

@@ -9,7 +9,7 @@ import TextField from 'cozy-ui/transpiled/react/TextField'
 import { withViewerLocales } from '../hoc/withViewerLocales'
 
 const SummaryDialog = ({ file, t, onClose }) => {
-  const [value, setValue] = useState(file.metadata.description)
+  const [value, setValue] = useState(file.metadata?.description)
   const [busy, setBusy] = useState(false)
   const client = useClient()
 

package/src/Panel/helpers.js

@@ -41,3 +41,19 @@ export const makeSize = bytes => {
  * @returns {string}
  */
 export const makePath = file => file.path?.replace(`/${file.name}`, '')
+
+/**
+ *
+ * @param {array} formattedMetadataQualification
+ * @param {number} idx
+ * @returns {boolean}
+ */
+export const makeHideDivider = (formattedMetadataQualification, idx) => {
+  const lastItem = formattedMetadataQualification.at(-1)
+  const isLastItem = idx === formattedMetadataQualification.length - 1
+  const isSecondLastItem = idx === formattedMetadataQualification.length - 2
+  const hideDivider =
+    isLastItem || (isSecondLastItem && lastItem.name === 'contact')
+
+  return hideDivider
+}

package/src/Panel/helpers.spec.js

@@ -0,0 +1,40 @@
+import { makeHideDivider } from './helpers'
+
+describe('makeHideDivider', () => {
+  describe('should hide the divider', () => {
+    it('if contact is the only qualification', () => {
+      const res = makeHideDivider([{ name: 'contact' }], 0)
+
+      expect(res).toBeTruthy()
+    })
+
+    it('for the last qualification', () => {
+      const res = makeHideDivider(
+        [{ name: 'contact' }, { name: 'issueDate' }],
+        1
+      )
+
+      expect(res).toBeTruthy()
+    })
+
+    it('for the first qualificaiton if contact is the second and last one', () => {
+      const res = makeHideDivider(
+        [{ name: 'issueDate' }, { name: 'contact' }],
+        0
+      )
+
+      expect(res).toBeTruthy()
+    })
+  })
+
+  describe('should show the divider', () => {
+    it('for contact if is the first and not only one qualification', () => {
+      const res = makeHideDivider(
+        [{ name: 'contact' }, { name: 'issueDate' }],
+        0
+      )
+
+      expect(res).toBeFalsy()
+    })
+  })
+})

package/src/Readme.md

@@ -58,7 +58,6 @@ import DemoProvider from './providers/DemoProvider'
 import Button from 'cozy-ui/transpiled/react/Buttons'
 import DownloadIcon from 'cozy-ui/transpiled/react/Icons/Download'
 import ShareIcon from 'cozy-ui/transpiled/react/Icons/Share'
-import { isValidForPanel } from 'cozy-ui/transpiled/react/Viewer/helpers'
 import getPanelBlocks, { panelBlocksSpecs } from 'cozy-ui/transpiled/react/Viewer/Panel/getPanelBlocks'
 import Sprite from 'cozy-ui/transpiled/react/Icon/Sprite'
 import IconButton from 'cozy-ui/transpiled/react/IconButton'

package/src/ViewerContainer.jsx

@@ -12,7 +12,6 @@ import { useExtendI18n } from 'cozy-ui/transpiled/react/providers/I18n'
 
 import Viewer from './Viewer'
 import ViewerInformationsWrapper from './ViewerInformationsWrapper'
-import { isValidForPanel } from './helpers'
 import { locales } from './locales'
 import { toolbarPropsPropType } from './proptypes'
 import { ActionMenuProvider } from './providers/ActionMenuProvider'
@@ -37,8 +36,7 @@ const ViewerContainer = props => {
   const fileCount = files.length
   const hasPrevious = currentIndex > 0
   const hasNext = currentIndex < fileCount - 1
-  const validForPanel =
-    isValidForPanel({ file: currentFile }) && isDesktop && !disablePanel
+  const validForPanel = isDesktop && !disablePanel
 
   const componentsPropsWithDefault = {
     ...componentsProps,

package/src/components/ViewerControls.jsx

@@ -11,7 +11,6 @@ import { infoWidth } from './InformationPanel'
 import Navigation from './Navigation'
 import Toolbar from './Toolbar'
 import styles from './styles.styl'
-import { isValidForPanel } from '../helpers'
 import { toolbarPropsPropType } from '../proptypes'
 
 const ACTIONS_HIDE_DELAY = 3000
@@ -125,14 +124,10 @@ class ViewerControls extends Component {
     const { showToolbar, showClose, toolbarRef, showFilePath } = toolbarProps
     const { hidden } = this.state
 
-    const shouldDisplayContentTop = isValidForPanel({ file })
-
     return (
       <div
         className={cx(styles['viewer-controls'], {
           [styles['viewer-controls--expanded']]: expanded,
-          [styles['viewer-controls--display-content-top']]:
-            shouldDisplayContentTop,
           [classes.viewerControlsWithInfo]: showInfoPanel
         })}
         ref={wrapped => {

package/src/components/styles.styl

@@ -50,15 +50,14 @@
     width 100%
     height 100%
 
-.viewer-controls.--expanded .viewer-nav
-    margin-top 0
-    width 40%
-
-.viewer-controls--display-content-top
     +medium-screen()
         justify-content flex-start
         padding-top $toolbarHeightMedium - $viewerMarginTopMedium + 1rem
 
+.viewer-controls.--expanded .viewer-nav
+    margin-top 0
+    width 40%
+
 .viewer-toolbar
     position absolute
     top 0

package/src/helpers.js

@@ -2,8 +2,6 @@ import { generateWebLink } from 'cozy-client'
 import {
   isEncrypted,
   isFromKonnector,
-  hasQualifications,
-  hasCertifications,
   normalize
 } from 'cozy-client/dist/models/file'
 import {
@@ -30,22 +28,6 @@ export const getCurrentModel = metadataName => {
  * @property {string} type - doctype of the document
  */
 
-/**
- * Checks if the file matches one of the following conditions:
- * - Is certified
- * - Is Qualified
- * - From a Connector
- *
- * @param {object} param
- * @param {IOCozyFile} param.file
- * @returns {boolean}
- */
-export const isValidForPanel = ({ file }) => {
-  return (
-    hasCertifications(file) || hasQualifications(file) || isFromKonnector(file)
-  )
-}
-
 export const isFileEncrypted = file => isEncrypted(file)
 
 export const formatDate = ({ f, lang, date }) => {