Use default value when page label is empty

Fixes zotero/zotero#3409
zotero · Oct 20, 2023 · 57db414 · 57db414
1 parent 647195f
commit 57db414
Showing 1 changed file with 12 additions and 16 deletions.
diff --git a/src/common/components/common/preview.js b/src/common/components/common/preview.js
@@ -65,12 +65,10 @@ export function PopupPreview(props) {
 							|| annotation.type === 'text' && <IconText/>
 						}
 					</div>
-					{annotation.pageLabel && (
-						<div className="page" onDoubleClick={handlePageLabelDoubleClick}>
-							<div><FormattedMessage id="pdfReader.page"/></div>
-							<div className="label">{annotation.pageLabel}</div>
-						</div>
-					)}
+					<div className="page" onDoubleClick={handlePageLabelDoubleClick}>
+						<div><FormattedMessage id="pdfReader.page"/></div>
+						<div className="label">{annotation.pageLabel || '-'}</div>
+					</div>
 				</div>
 				<div className="right">
 					{annotation.authorName && (
@@ -273,16 +271,14 @@ export function SidebarPreview(props) {
 							|| annotation.type === 'text' && <IconText/>
 						}
 					</div>
-					{annotation.pageLabel && (
-						<div
-							className="page"
-							onClick={handlePageLabelClick}
-							onDoubleClick={handlePageLabelDoubleClick}
-						>
-							<div><FormattedMessage id="pdfReader.page"/></div>
-							<div className="label">{annotation.pageLabel}</div>
-						</div>
-					)}
+					<div
+						className="page"
+						onClick={handlePageLabelClick}
+						onDoubleClick={handlePageLabelDoubleClick}
+					>
+						<div><FormattedMessage id="pdfReader.page"/></div>
+						<div className="label">{annotation.pageLabel || '-'}</div>
+					</div>
 				</div>
 				<div className="right">
 					{annotation.authorName && (