✨ enhance selection loading logic to check for bookmarkBriefInfo before loading (#137)

Author: Whyjsee

apps/slax-reader-extensions/src/components/Selection/manager.ts

@@ -411,7 +411,7 @@ export class MarkManager extends Base {
   }
 
   getApproxText(range: Range): HighlightRangeInfo | undefined {
-    if (!range) {
+    if (!range || range.collapsed || range.toString().trim().length === 0) {
       return undefined
     }
 

apps/slax-reader-extensions/src/components/SidePanel.vue

@@ -177,7 +177,7 @@ watch(
     }
 
     loadBriefDetail().then(() => {
-      loadSelection()
+      bookmarkBriefInfo.value && loadSelection()
     })
   }
 )
@@ -211,7 +211,7 @@ props.browser.runtime.onMessage.addListener(
 onMounted(() => {
   updateBookmarkStatus()
   loadBriefDetail().then(() => {
-    loadSelection()
+    bookmarkBriefInfo.value && loadSelection()
   })
 
   tryGetUserInfo(true).then(res => {

apps/slax-reader-extensions/src/entrypoints/content/index.ts

@@ -58,6 +58,12 @@ export default defineContentScript({
       html{
         z-index: 99999999999 !important;
       }
+
+      @media print {
+        :host, html {
+          display: none !important;
+        }
+      }
     `,
       onMount: container => {
         const app = createApp(SidePanel, {