AuroraMiddleware/qt5base-lts
1
0
Fork 0
Code Issues Pull Requests Packages Projects Releases Wiki Activity

Mac style: Get proper ThemeButtonKind for QDateTimeEdit

Browse Source 
When it gets 'calendarPopup' set, that is.

The reason lies, indirectly, in the extra margins we add to
QComboBoxes and other QPushButtons in the Mac style. However,
these extra margins do not apply to spinboxes, which is what
QDateTimeEdit is.

In particular, this causes Designer to show QDateTimeEdit at
the wrong size when it's not in a layout.

Task-number: QTBUG-38615
Change-Id: I3dd52e10db8c8970a6dc40b39fbe29dd86167daa
Reviewed-by: Jake Petroules <jake.petroules@petroules.com>
Reviewed-by: J-P Nurmi <jpnurmi@digia.com>
Reviewed-by: Morten Johan Sørvig <morten.sorvig@digia.com>
This commit is contained in:
Gabriel de Dietrich 2014-04-29 19:29:59 +02:00 committed by The Qt Project
parent 721543e464
commit a006ede6e7
1 changed files with 24 additions and 6 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

30
src/widgets/styles/qmacstyle_mac.mm
View File

@ -1345,12 +1345,30 @@ void QMacStylePrivate::initComboboxBdi(const QStyleOptionComboBox *combo, HIThem
// an extra check here before using the mini and small buttons.
int h = combo->rect.size().height();
if (combo->editable){
if (h < 21)
bdi->kind = kThemeComboBoxMini;
else if (h < 26)
bdi->kind = kThemeComboBoxSmall;
else
bdi->kind = kThemeComboBox;
if (qobject_cast<const QDateTimeEdit *>(widget)) {
// Except when, you know, we get a QDateTimeEdit with calendarPopup
// enabled. And then things get weird, basically because it's a
// transvestite spinbox with editable combobox tendencies. Meaning
// that it wants to look a combobox, except that it isn't one, so it
// doesn't get all those extra free margins around. (Don't know whose
// idea those margins were, but now it looks like we're stuck with
// them forever). So anyway, the height threshold should be smaller
// in this case, or the style gets confused when it needs to render
// or return any subcontrol size of the poor thing.
if (h < 9)
bdi->kind = kThemeComboBoxMini;
else if (h < 22)
bdi->kind = kThemeComboBoxSmall;
else
bdi->kind = kThemeComboBox;
} else {
if (h < 21)
bdi->kind = kThemeComboBoxMini;
else if (h < 26)
bdi->kind = kThemeComboBoxSmall;
else
bdi->kind = kThemeComboBox;
}
} else {
// Even if we specify that we want the kThemePopupButton, Carbon
// will use the kThemePopupButtonSmall if the size matches. So we