fix: qApp set fixed font not work

linuxdeepin / qt5integration

Qt platform theme integration plugins for DDE.

Other

36 stars 47 forks source link

fix: qApp set fixed font not work #199

Closed kegechen closed 11 months ago

kegechen commented 11 months ago

onFontChanged will send ApplicationFontChange event to all windows. we stop this opration if qApp set a font

dtk菜单显示了就无效了原因如下： sub menu ==> draw arrow icon ==> QIcon::fromtheme ==> QIconLoader ==> themeHint(SystemIconThemeName) ==> appTheme() ==> connect ==> fontNameChanged

子菜单需要绘制右箭头图标，用了 QIcon::fromtheme 。其实只要那个信号槽连上了就会强制更新。

Issue: https://github.com/linuxdeepin/dtk/issues/124

deepin-ci-robot commented 11 months ago

[APPROVALNOTIFIER] This PR is NOT APPROVED

This pull-request has been approved by: kegechen, zccrs

The full list of commands accepted by this bot can be found here.

Needs approval from an approver in each of these files: - **[debian/deepin/OWNERS](https://github.com/linuxdeepin/qt5integration/blob/master/debian/deepin/OWNERS)** Approvers can indicate their approval by writing `/approve` in a comment Approvers can cancel approval by writing `/approve cancel` in a comment