diff options
author | Gabriel de Dietrich <gabriel.dedietrich@qt.io> | 2016-12-20 19:12:36 -0800 |
---|---|---|
committer | Gabriel de Dietrich <gabriel.dedietrich@qt.io> | 2017-01-06 19:41:54 +0000 |
commit | 0e810e27a5e5a6c2e244ca439fbdf4f4ccc5b3da (patch) | |
tree | 18e1e9ce5c7351dc5515321155d2b8b2892bdf32 /src/corelib/thread | |
parent | 64f4108c3a18c3e72da98b74efbf36dc3b8e08e9 (diff) |
QMacStyle: Fix drawing of PE_IndicatorMenuCheckMark
... And refactor to use it for drawing CE_MenuItem.
This removes a couple calls to HITheme. Instead, we
use CoreText to render the checkmark character. Notice
that this is what we were doing with HITheme for
CE_MenuItem. Drawing PE_IndicatorMenuCheckMark was
done manually, and in a way that has not been updated.
Notice also that the choice of color from the palette
has also been fixed to look for the State_Selected state
instead of the State_On state, in accordance with
QCommonStyle. In fact, on macOS, the checkmark is never
rendered when the State_On bit is clear. Moreover, we
systematically pick the checkmark color from the style
option palette.
[ChangeLog][QtWidgets][QMacStyle] PE_IndicatorMenuCheckMark
looks for State_On instead of the State_Selected to set its
highlighted state. Its color is picked from the style option
palette.
Change-Id: Ib033df357fd83080cb4320c43949f75bb079c8a5
Task-number: QTBUG-57470
Reviewed-by: Jake Petroules <jake.petroules@qt.io>
Diffstat (limited to 'src/corelib/thread')
0 files changed, 0 insertions, 0 deletions