linuxdeepin / qt5integration

Qt platform theme integration plugins for DDE.
Other
36 stars 47 forks source link

fix: 玲珑应用托盘无法共存 #136

Closed kamiyadm closed 1 year ago

kamiyadm commented 1 year ago

close linuxdeepin/developer-center#3649

black-desk commented 1 year ago

他commit message没写清楚, 大概情况是这样的:

  1. 托盘图标是个平台相关的实现, kde和gnome都有自己的平台相关插件;
  2. deepin目前似乎是没有专门的托盘图标接口, 目前相关业务基本在使用kde的已经弃用的接口;
  3. 使用kde的老接口是因为qt提供的默认的实现用了kde的接口;
  4. 这套接口目前看来没有大问题, 但是其一些实现上的细节导致了应用如果跑在容器中, 托盘图标会有bug.

所以这里将qtbase中的实现抄了过来进行了一个简单的修复.

zccrs commented 1 year ago

他commit message没写清楚, 大概情况是这样的:

  1. 托盘图标是个平台相关的实现, kde和gnome都有自己的平台相关插件;
  2. deepin目前似乎是没有专门的托盘图标接口, 目前相关业务基本在使用kde的已经弃用的接口;
  3. 使用kde的老接口是因为qt提供的默认的实现用了kde的接口;
  4. 这套接口目前看来没有大问题, 但是其一些实现上的细节导致了应用如果跑在容器中, 托盘图标会有bug.

所以这里将qtbase中的实现抄了过来进行了一个简单的修复.

commit message可以更新下

zccrs commented 1 year ago

对于已经处理了的评论,点一下 Resolve