Skip to content

Commit

Permalink
tst_QDialogButtonBox::hideAndShowButton: Check for Qt::TabFocusAllCon…
Browse files Browse the repository at this point in the history
…trols

The test requires Qt::TabFocusAllControls, so Qt::TabFocusTextControls
and/or Qt::TabFocusListControls, which is the default on macOS, is not
sufficient.

Pick-to: 6.8
Change-Id: Ideb673f570125c0d73c67a408f3d144f29052ace
Reviewed-by: Volker Hilsheimer <[email protected]>
  • Loading branch information
torarnv committed Sep 19, 2024
1 parent c1ee648 commit 0a1c5d6
Showing 1 changed file with 2 additions and 2 deletions.
Original file line number Diff line number Diff line change
Expand Up @@ -386,8 +386,8 @@ void tst_QDialogButtonBox::removeButton()
#ifdef QT_BUILD_INTERNAL
void tst_QDialogButtonBox::hideAndShowButton()
{
if (QGuiApplication::styleHints()->tabFocusBehavior() == Qt::NoTabFocus)
QSKIP("Test skipped with Qt::NoTabFocus");
if (QGuiApplication::styleHints()->tabFocusBehavior() != Qt::TabFocusAllControls)
QSKIP("Test requires Qt::TabFocusAllControls tab focus behavior");

QDialogButtonBox buttonBox;
QDialogButtonBoxPrivate *d = static_cast<QDialogButtonBoxPrivate *>(QObjectPrivate::get(&buttonBox));
Expand Down

0 comments on commit 0a1c5d6

Please sign in to comment.