feat: 跟进 v3.3.6 菜单的 data-name 属性变更

siyuan-note/siyuan#16141

Author: TCOTC

src/modules/googleAnalytics.ts

@@ -146,7 +146,7 @@ export class GoogleAnalytics {
             });
         }
 
-        logging.log('Sending Google Analytics data. https://github.com/TCOTC/Whisper/issues/11'); // TODO: 修改输出日志为“Whisper 主题正在发送 Google Analytics 数据（在外观模式菜单中可以禁用）”
+        logging.log('Sending Google Analytics data. https://github.com/TCOTC/Whisper/issues/11'); // TODO 修改输出日志为“Whisper 主题正在发送 Google Analytics 数据（在外观模式菜单中可以禁用）”
 
         // 收集并发送基本信息
         this.sendInfo();

src/modules/menuHandler.ts

@@ -52,43 +52,39 @@ export class MenuHandler implements ThemeModule {
         this.commonMenu.insertAdjacentElement('beforebegin', whisperCommonMenu);
         this.whisperCommonMenu = whisperCommonMenu;
 
-        this.commonMenuObserver = new MutationObserver((mutations) => {
-            // 使用一个标志位来确保只处理一次
-            let processed = false;
-
-            mutations.forEach(() => {
-                if (processed) return; // 如果已经处理过，直接返回
+        this.commonMenuObserver = new MutationObserver(() => {
+            this.whisperCommonMenu?.removeAttribute('data-name');
 
+            if (this.commonMenu) {
                 // 先卸载监听再添加，避免重复添加
-                if (this.commonMenu) {
-                    this.commonMenu.removeEventListener('click', this.handleMenuClick, true);
-                    this.commonMenu.removeEventListener('click', this.handleCloseClick, true);
-                }
-                
-                if (this.whisperCommonMenu) {
-                    this.whisperCommonMenu.dataset.name = '';
-                }
+                this.commonMenu.removeEventListener('click', this.handleMenuClick, true);
+                this.commonMenu.removeEventListener('click', this.handleCloseClick, true);
 
-                if (this.commonMenu?.getAttribute('data-name') === 'barmode') {
+                const menuName = this.commonMenu.getAttribute('data-name');
+                if (menuName === 'barmode') {
                     // 外观模式菜单
                     this.commonMenu.addEventListener('click', this.handleMenuClick, true);
-                } else if ( // TODO功能 需要给原生 PR 一个菜单的 data-name="tab-header" 属性来简化判断逻辑，然后提升主题最低版本号
-                    this.commonMenu?.querySelector('[data-id="close"]') &&
-                    this.commonMenu?.querySelector('[data-id="split"]') &&
-                    this.commonMenu?.querySelector('[data-id="copy"]')
+                } else if (menuName === 'tab' || // v3.3.6 给页签菜单增加 data-name="tab" 属性 https://github.com/siyuan-note/siyuan/pull/16141
+                    // TODO废弃 以下6行代码是兼容 v3.3.6 以前的版本，版本号提升到 v3.3.6 时即可移除。并且修改之后需要验证是否在平板上正常工作
+                    (
+                        this.commonMenu.querySelector('[data-id="close"]') &&
+                        this.commonMenu.querySelector('[data-id="split"]') &&
+                        this.commonMenu.querySelector('[data-id="copy"]')
+                    )
                     // && this.commonMenu?.querySelector('[data-id="tabToWindow"]') // 平板上没有“移动到新窗口”选项
                 ) {
                     // 页签菜单
+                    // TODO废弃 以下3行代码是兼容 v3.3.6 以前的版本，版本号提升到 v3.3.6 时即可移除，并且需要同步修改对应的使用 [data-name="tab-header"] 的 CSS
                     if (this.whisperCommonMenu) {
                         this.whisperCommonMenu.dataset.name = 'tab-header';
                     }
                     this.handleTabClose();
                 }
+            }
 
-                processed = true; // 标记为已处理
-            });
         });
 
+        // TODO跟进 试试通过 Add plugin event bus to #commonMenu 来实现更准确的监听 https://github.com/TCOTC/Whisper/issues/16 https://github.com/siyuan-note/siyuan/issues/16171
         // 监听菜单的属性变化
         this.commonMenuObserver.observe(this.commonMenu, { attributes: true });
     }

src/modules/tooltipHandler.ts

@@ -55,6 +55,8 @@ export class TooltipHandler implements ThemeModule {
         // 文档树
         // TODO跟进 文档信息显示在左下角的问题还是没解决，估计是思源本体的问题：鼠标划过笔记本之后 tooltip 不隐藏 https://github.com/siyuan-note/siyuan/issues/14823
         //  到时候把这部分代码注释掉测试看看还会不会有问题
+
+        // TODO跟进 试试通过 Add plugin event bus to tooltip 来实现更准确的监听 https://github.com/TCOTC/Whisper/issues/15 https://github.com/siyuan-note/siyuan/issues/16151
         const doc = e.closest('[data-type="navigation-file"]');
         if (doc) {
             this.removeTooltipData();

styles/modules/_menu.scss

@@ -4,7 +4,7 @@
 
 // 工具栏
 .protyle-toolbar {
-	transition: none; // 禁用工具栏平移效果 // TODO跟进: v3.5.5 之后就不需要覆盖这个样式了，以后提升版本号达到 3.5.5 再移除 https://github.com/siyuan-note/siyuan/commit/a1f275e3c329e8bb93863d353741c1d106faaf55
+	transition: none; // 禁用工具栏平移效果 // TODO废弃 v3.5.5 之后就不需要覆盖这个样式了，以后提升版本号达到 3.5.5 再移除 https://github.com/siyuan-note/siyuan/commit/a1f275e3c329e8bb93863d353741c1d106faaf55
 }
 
 // 超链接菜单，链接和锚文本输入框默认两行高（不显示滚动条就需要 50px）
@@ -58,7 +58,7 @@ label:hover .b3-switch:checked:not(:disabled)::before, .b3-switch:checked:hover:
 	}
 }
 // 页签菜单
-// TODO功能 需要给原生 PR 一个菜单的 data-name="tab-header" 属性来简化判断逻辑，类似外观模式的 data-name="barmode"
+// TODO废弃 以下选择器是兼容 v3.3.6 以前的版本，版本号提升到 v3.3.6 时即可修改为 #commonMenu[data-name="tab"] > .b3-menu__items  ，并且需要同步修改 menuHandler.ts 的对应逻辑
 #whisperCommonMenu[data-name="tab-header"] + #commonMenu > .b3-menu__items {
 	display: flex;
 	flex-direction: column;