Merge pull request #82 from coolpace/next

Next

Author: Codennnn

extension/manifest-firefox.json

@@ -9,7 +9,7 @@
 
   "name": "V2EX Polish",
 
-  "version": "1.9.25",
+  "version": "1.10.0",
 
   "description": "专为 V2EX 用户设计，提供了丰富的扩展功能。",
 

extension/manifest.json

@@ -3,7 +3,7 @@
 
   "name": "V2EX Polish",
 
-  "version": "1.9.25",
+  "version": "1.10.0",
 
   "description": "专为 V2EX 用户设计，提供了丰富的扩展功能。",
 

src/components/model.ts src/components/modal.tssrc/components/model.ts renamed to src/components/modal.ts

@@ -1,6 +1,6 @@
 import { createButton } from './button'
 
-interface ModelElements {
+interface ModalElements {
   $mask: JQuery
   $main: JQuery
   $container: JQuery
@@ -9,23 +9,23 @@ interface ModelElements {
   $content: JQuery
 }
 
-interface ModelControl extends ModelElements {
+interface ModalControl extends ModalElements {
   open: () => void
   close: () => void
 }
 
-interface CreateModelProps {
+interface CreateModalProps {
   root?: JQuery
   title?: string
-  onMount?: (elements: ModelElements) => void
-  onOpen?: (elements: ModelElements) => void
-  onClose?: (elements: ModelElements) => void
+  onMount?: (elements: ModalElements) => void
+  onOpen?: (elements: ModalElements) => void
+  onClose?: (elements: ModalElements) => void
 }
 
 /**
- * 创建 model 框。
+ * 创建 modal 框。
  */
-export function createModel(props: CreateModelProps): ModelControl {
+export function createModal(props: CreateModalProps): ModalControl {
   const { root, title, onOpen, onClose, onMount } = props
 
   const $mask = $('<div class="v2p-modal-mask">')
@@ -51,7 +51,7 @@ export function createModel(props: CreateModelProps): ModelControl {
 
   const $container = $mask.append($main).hide()
 
-  const modelElements = {
+  const modalElements = {
     $mask,
     $main,
     $container,
@@ -63,9 +63,11 @@ export function createModel(props: CreateModelProps): ModelControl {
   // 用于判定是否已经绑定了事件, 避免重复绑定。
   let boundEvent = false
 
-  const maskClickHandler = () => {
-    // eslint-disable-next-line @typescript-eslint/no-use-before-define
-    handleModalClose()
+  const maskClickHandler = (ev: JQuery.MouseUpEvent) => {
+    if (ev.currentTarget === $mask.get(0) && ev.currentTarget === ev.target) {
+      // eslint-disable-next-line @typescript-eslint/no-use-before-define
+      handleModalClose()
+    }
   }
 
   const keyupHandler = (ev: JQuery.KeyDownEvent) => {
@@ -76,21 +78,21 @@ export function createModel(props: CreateModelProps): ModelControl {
   }
 
   const handleModalClose = () => {
-    $mask.off('click', maskClickHandler)
+    $mask.off('mouseup', maskClickHandler)
     $(document).off('keydown', keyupHandler)
     boundEvent = false
 
     $container.fadeOut('fast')
     document.body.classList.remove('v2p-modal-open')
 
-    onClose?.(modelElements)
+    onClose?.(modalElements)
   }
 
   const handleModalOpen = () => {
     // Hack: 为了防止 open 点击事件提前冒泡到 document 上，需要延迟绑定事件。
     setTimeout(() => {
       if (!boundEvent) {
-        $mask.on('click', maskClickHandler)
+        $mask.on('mouseup', maskClickHandler)
         $(document).on('keydown', keyupHandler)
         boundEvent = true
       }
@@ -99,16 +101,16 @@ export function createModel(props: CreateModelProps): ModelControl {
     $container.fadeIn('fast')
     document.body.classList.add('v2p-modal-open')
 
-    onOpen?.(modelElements)
+    onOpen?.(modalElements)
   }
 
   $closeBtn.on('click', handleModalClose)
 
-  onMount?.(modelElements)
+  onMount?.(modalElements)
 
   if (root) {
     root.append($container)
   }
 
-  return { ...modelElements, open: handleModalOpen, close: handleModalClose }
+  return { ...modalElements, open: handleModalOpen, close: handleModalClose }
 }

src/contents/home/index.ts

@@ -52,7 +52,7 @@ void (async () => {
       const bgImg = ($cell.prop('style') as CSSStyleDeclaration).backgroundImage
 
       if (bgImg.includes('/static/img/corner_star.png')) {
-        $cell.find('.count_livid').append(' | 置顶').addClass('count_orange')
+        $cell.find('.count_livid').append(' | 推荐').addClass('count_orange')
       }
     })
   }