From 6b277fce74c336d69d718d4c8ec02c9474e2f499 Mon Sep 17 00:00:00 2001
From: Rustem Mussabekov <winlinx@ya.ru>
Date: Fri, 6 Dec 2024 15:01:43 +0300
Subject: [PATCH] Fix preview button

---
 package.json                               |  2 +-
 src/co/bookmarks/item/actions.js           |  4 ++--
 src/co/bookmarks/item/contextmenu.js       |  4 ++--
 src/routes/my/item/tab/preview.js          | 16 ++++++++--------
 src/routes/my/item/tab/preview.module.styl |  3 +--
 src/target/extension/manifest/index.js     |  2 ++
 6 files changed, 16 insertions(+), 15 deletions(-)

diff --git a/package.json b/package.json
index 53bee255..873d27bc 100755
--- a/package.json
+++ b/package.json
@@ -1,6 +1,6 @@
 {
 	"name": "app",
-	"version": "5.6.63",
+	"version": "5.6.64",
 	"description": "All-in-one bookmark manager",
 	"author": "Rustem Mussabekov",
 	"license": "MIT",
diff --git a/src/co/bookmarks/item/actions.js b/src/co/bookmarks/item/actions.js
index 30de0d40..fd2eb929 100644
--- a/src/co/bookmarks/item/actions.js
+++ b/src/co/bookmarks/item/actions.js
@@ -7,7 +7,7 @@ import Button from '~co/common/button'
 import Icon from '~co/common/icon'
 
 export default function BookmarkItemActions(props) {
-    const { _id, link, access, important } = props
+    const { _id, link, access, important, type } = props
     const { className='' } = props
     const { onCopyLinkClick, onImportantClick, onRemoveClick } = props
 
@@ -38,7 +38,7 @@ export default function BookmarkItemActions(props) {
             <Button 
                 data-button='preview'
                 as={Link}
-                to={`item/${_id}/preview`}
+                to={`item/${_id}/${type == 'link' ? 'web' : 'preview'}`}
                 variant='outline'
                 title={t.s('preview')}>
                 <Icon name='show' />
diff --git a/src/co/bookmarks/item/contextmenu.js b/src/co/bookmarks/item/contextmenu.js
index 310bd1e1..30888bcf 100644
--- a/src/co/bookmarks/item/contextmenu.js
+++ b/src/co/bookmarks/item/contextmenu.js
@@ -5,7 +5,7 @@ import Popover, { Menu, MenuItem, MenuSeparator } from '~co/overlay/popover'
 import { target } from '~target'
 
 export default function BookmarksItemContextmenu({
-    _id, link, important, access, reparse, cache, fileType,
+    _id, link, important, access, reparse, cache, fileType, type,
     onContextMenuClose, onRemoveClick, onCopyLinkClick,
     onSelectClick, onImportantClick, onReparseClick
 }) {
@@ -23,7 +23,7 @@ export default function BookmarksItemContextmenu({
                 <MenuSeparator />
 
                 {target != 'extension' && (
-                    <MenuItem to={`item/${_id}`}>
+                    <MenuItem to={`item/${_id}/${type == 'link' ? 'web' : 'preview'}`}>
                         {t.s('preview')}
                     </MenuItem>
                 )}
diff --git a/src/routes/my/item/tab/preview.js b/src/routes/my/item/tab/preview.js
index 3e2fb3e9..8253b130 100644
--- a/src/routes/my/item/tab/preview.js
+++ b/src/routes/my/item/tab/preview.js
@@ -1,10 +1,10 @@
 import s from './preview.module.styl'
-import React, { useState, useCallback } from 'react'
+import React, { useState, useCallback, useEffect } from 'react'
 import { useSelector } from 'react-redux'
 import { API_ENDPOINT_URL } from '~data/constants/app'
 
 import useWithWebView from '~co/highlights/useWithWebView'
-import Preloader from '~co/common/preloader'
+import Preloader from '~co/common/webview/preloader'
 
 function getPreviewLink(_id) {
     return `${API_ENDPOINT_URL}raindrop/preview/${_id}`
@@ -27,10 +27,16 @@ export default function PageMyItemTabPreview({ item: { _id }, webViewRef }) {
     //loading state
     const [loading, setLoading] = useState(true)
     const onLoad = useCallback(()=>{ setLoading(false) }, [setLoading])
+    useEffect(()=>{ setLoading(true) }, [_id])
 
     return (
         <>
+            {loading && (
+                <Preloader className={s.preloader} />
+            )}
+            
             <iframe 
+                key={_id}
                 ref={webViewRef}
                 role='article'
                 className={s.embed}
@@ -39,12 +45,6 @@ export default function PageMyItemTabPreview({ item: { _id }, webViewRef }) {
                 referrerPolicy='no-referrer'
                 src={`${getPreviewLink(_id)}#solid-bg=false&theme=${font_color||theme.app}&font-family=${font_family}&font-size=${font_size}`}
                 onLoad={onLoad} />
-
-            {loading && (
-                <div className={s.preloader}>
-                    <Preloader enlarge='1.5' />
-                </div>
-            )}
         </>
     )
 }
\ No newline at end of file
diff --git a/src/routes/my/item/tab/preview.module.styl b/src/routes/my/item/tab/preview.module.styl
index 20e8a85f..133fa00c 100644
--- a/src/routes/my/item/tab/preview.module.styl
+++ b/src/routes/my/item/tab/preview.module.styl
@@ -11,8 +11,7 @@
     position: absolute
     left: 0
     right: 0
-    bottom: 0
-    top: 0
+    top: auto !important;
     display: flex
     align-items: center
     justify-content: center
diff --git a/src/target/extension/manifest/index.js b/src/target/extension/manifest/index.js
index 2b29ea6e..f9b38458 100644
--- a/src/target/extension/manifest/index.js
+++ b/src/target/extension/manifest/index.js
@@ -126,6 +126,8 @@ module.exports = ({ vendor, production=false }, l) => {
 		// },
 
 		commands: {
+			_execute_action: {
+			},
 			save_page: {
 				suggested_key: (vendor == 'chrome' || vendor == 'safari') ? {
 					default: 'Ctrl+Shift+S'