Repository navigation
add:通知显示新增位置底栏上方 - #695
add:通知显示新增位置底栏上方#695chenjei2011 wants to merge 3 commits into
Conversation
🤖 Augment PR SummarySummary: Adds the “Above bottom bar” dynamic-notification placement option requested in #691. 🤖 Was this summary useful? React with 👍 or 👎 |
![augmentcode[bot]](https://avatars.githubusercontent.com/in/1027498?s=60&v=4){kind=small}
| var bottomRight = BlackboardCenterSide.TranslatePoint( | ||
| new Point(BlackboardCenterSide.ActualWidth, BlackboardCenterSide.ActualHeight), this); | ||
| double notificationWidth = DynamicNotification.ActualWidth > 0 ? DynamicNotification.ActualWidth : DynamicNotification.Width; | ||
| double notificationHeight = DynamicNotification.ActualHeight > 0 ? DynamicNotification.ActualHeight : 72; |
There was a problem hiding this comment.
At Ink Canvas/MainWindow_cs/MW_Notification.cs:256, consider a ForcePopup notification or clicking to expand one in whiteboard mode: placement runs before DynamicNotification.Show populates the control, and subsequent size changes never recalculate this margin. Using the previous message's height or 72 means expanded content can overlap the bottom toolbar and be clipped at the window's lower edge.
Severity: medium
🤖 Was this useful? React with 👍 or 👎, or 🚀 if it prevented an incident/outage.
| ApplyDynamicNotificationFloatingBarPlacement(); | ||
| break; | ||
| case "BottomBarAbove": | ||
| ApplyDynamicNotificationBottomBarPlacement(); |
There was a problem hiding this comment.
At Ink Canvas/MainWindow_cs/MW_Notification.cs:188, consider entering whiteboard mode while a notification is already visible: ApplyDynamicNotificationPlacement runs only when a notification is requested, and SwitchBackground never recalculates its margin. The notification therefore remains at the desktop floating-toolbar coordinates instead of moving above BlackboardCenterSide, and similarly retains the whiteboard position on exit.
Severity: medium
🤖 Was this useful? React with 👍 or 👎, or 🚀 if it prevented an incident/outage.
|
我建议将相关逻辑合并入浮动栏上方，而不是新增选项 |
|
ok,等我有空就改 |
2 participants
需求动机
#691
功能描述
在设置->通知->通知设置->灵动通知位置新增选项底栏上方，若启用此选项，浮动栏情况下与浮动栏上方选项一致，进入白板后若有灵动通知显示在白板工具栏上方
前端更改(看样式)
后端更改
新增
ApplyDynamicNotificationBottomBarPlacement函数用来响应底栏上方这个选项