Skip to content

feat: PPT按钮预览#567

Open
AstrZero wants to merge 6 commits into
net6from
feat/PPT-Button-Preview
Open

feat: PPT按钮预览#567
AstrZero wants to merge 6 commits into
net6from
feat/PPT-Button-Preview

Conversation

@AstrZero

Copy link
Copy Markdown
Collaborator

No description provided.

@augmentcode

augmentcode Bot commented Jun 29, 2026

Copy link
Copy Markdown
🤖 Augment PR Summary

Summary: This PR adds a dedicated “PPT 翻页按钮预览” flow to configure and preview PPT page-flip navigation bars.

Changes:

  • Refactors PPTUIManager to read per-position settings (page number visibility, opacity, black background theme) from PowerPointSettings.
  • Adds new persisted settings fields for each bar position (LS/RS/LB/RB) with lazy fallback to legacy PPTSButtonsOption/PPTBButtonsOption.
  • Introduces PPTPageFlipPreviewWindow (fullscreen, non-activating) to visually preview all four navigation bars.
  • Adds PPTPageFlipPreviewPage to edit enabled state, page number, theme, offset, opacity, and global scale with real-time preview.
  • Simplifies the existing PowerPoint settings page by replacing the old per-position expander UI with a single entry card that navigates to the new preview page.
  • Registers the new page in SettingsWindow navigation and adjusts pane display mode while previewing.
  • Adds a minimal package.json (Express/CORS) to the repo root.

Technical Notes: Preview mode temporarily forces PPT presentation UI state, disables AvoidFullScreenHelper, and blocks MainWindow input while the Settings window stays focused.

🤖 Was this summary useful? React with 👍 or 👎

@augmentcode augmentcode Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Review completed. 1 suggestion posted.

Fix All in Augment

Comment augment review to trigger a new review at any time.


<ui:SettingsCard Header="{x:Static props:PPTStrings.SideGroupTitle}"
Visibility="{Binding IsOn, ElementName=CardShowPPTButton, Converter={StaticResource BooleanToVisibilityConverter}}">
<ui:SettingsCard x:Name="CardPPTPageFlipPreviewEntry"

@augmentcode augmentcode Bot Jun 29, 2026

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Ink Canvas/Windows/SettingsViews/Pages/PowerPointPage.xaml still has Visibility bindings referencing CardShowPPTButton later in the file (e.g., around line ~110), but CardShowPPTButton was deleted in this PR; that dangling ElementName will likely cause a XAML load/binding error when the page is created.

Severity: high

Fix This in Augment

🤖 Was this useful? React with 👍 or 👎, or 🚀 if it prevented an incident/outage.

AstrZero and others added 4 commits June 29, 2026 20:48
将原本单独的PPTPageFlipSettingsSubPage的所有设置逻辑和UI整合到PPTPageFlipPreviewPage中,删除冗余的子页面文件,简化设置页面结构
将静态页面类型字典从实例字段改为静态只读字段,移除重复的初始化代码,统一页面类型注册位置
实现PPT翻页工具栏全局统一设置功能,支持:
1. 新增全局默认配置项与单位置独立配置开关
2. 全局设置变更时自动同步所有位置的有效配置
3. 重构设置UI,新增全局设置标签页
4. 统一管理各按钮的缩放、偏移、透明度等样式参数
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants