qt5base-lts/tests/auto/widgets/kernel/qstackedlayout/tst_qstackedlayout.cpp
Jan Arve Saether 9f96bf2023 Fix a regression with regards to hfw behavior in QStackedWidget
This is a regression because we made QStackedWidget actually
support heightForWidth in 4.8. This was done with change
4780f94e391b5e881497c5228661dead42c821fa.

The problem was that heightForWidth was not calculated correctly
because some of the pages were hidden. The hidden pages were
actually not contributing to the hfw of the QStackedWidget at all.
This again caused the QStackedWidget to change its heightForWidth()
value when the current tab changed, which again could cause "jumps"
in the UI when switching tabs (as demonstrated in the task).

The problem was that the patch relied on calling
QWidgetItem::heightForWidth(), and this function would return -1
if the widget was hidden. However, QWidget::heightForWidth() does
not have this magic and returns the proper hfw value regardless
of its visibility.

One could argue about the correctness of this patch, but since
QStackedLayout::sizeHint() disregards QWidgetItem::sizeHint() (it
asks the widget directly), we do the same in
QStackedLayoutHFW::heightForWidth() for consistency.

In addition, QStackedLayout enforces that only widgets can be added
to it, and you cannot add your own QLayoutItem subclasses to it:

  qWarning("QStackedLayout::addItem: Only widgets can be added");

Task-id: QTBUG-24758

Change-Id: I349cf8f4215e4581ea237ef773d53dcdf3db176b
Reviewed-by: Paul Olav Tvete <paul.tvete@digia.com>
2012-10-10 14:22:58 +02:00

400 lines
12 KiB
C++

/****************************************************************************
**
** Copyright (C) 2012 Digia Plc and/or its subsidiary(-ies).
** Contact: http://www.qt-project.org/legal
**
** This file is part of the test suite of the Qt Toolkit.
**
** $QT_BEGIN_LICENSE:LGPL$
** Commercial License Usage
** Licensees holding valid commercial Qt licenses may use this file in
** accordance with the commercial license agreement provided with the
** Software or, alternatively, in accordance with the terms contained in
** a written agreement between you and Digia. For licensing terms and
** conditions see http://qt.digia.com/licensing. For further information
** use the contact form at http://qt.digia.com/contact-us.
**
** GNU Lesser General Public License Usage
** Alternatively, this file may be used under the terms of the GNU Lesser
** General Public License version 2.1 as published by the Free Software
** Foundation and appearing in the file LICENSE.LGPL included in the
** packaging of this file. Please review the following information to
** ensure the GNU Lesser General Public License version 2.1 requirements
** will be met: http://www.gnu.org/licenses/old-licenses/lgpl-2.1.html.
**
** In addition, as a special exception, Digia gives you certain additional
** rights. These rights are described in the Digia Qt LGPL Exception
** version 1.1, included in the file LGPL_EXCEPTION.txt in this package.
**
** GNU General Public License Usage
** Alternatively, this file may be used under the terms of the GNU
** General Public License version 3.0 as published by the Free Software
** Foundation and appearing in the file LICENSE.GPL included in the
** packaging of this file. Please review the following information to
** ensure the GNU General Public License version 3.0 requirements will be
** met: http://www.gnu.org/copyleft/gpl.html.
**
**
** $QT_END_LICENSE$
**
****************************************************************************/
#include <QtTest/QtTest>
#include <QLineEdit>
#include <QLabel>
#include <QStackedLayout>
#include <qapplication.h>
#include <qwidget.h>
#include <QPushButton>
class tst_QStackedLayout : public QObject
{
Q_OBJECT
public:
tst_QStackedLayout();
virtual ~tst_QStackedLayout();
public slots:
void initTestCase();
void cleanupTestCase();
void init();
void cleanup();
private slots:
void getSetCheck();
void testCase();
void deleteCurrent();
void removeWidget();
void keepFocusAfterSetCurrent();
void heigthForWidth();
private:
QWidget *testWidget;
};
// Testing get/set functions
void tst_QStackedLayout::getSetCheck()
{
QStackedLayout obj1;
// int QStackedLayout::currentIndex()
// void QStackedLayout::setCurrentIndex(int)
obj1.setCurrentIndex(0);
QCOMPARE(-1, obj1.currentIndex());
obj1.setCurrentIndex(INT_MIN);
QCOMPARE(-1, obj1.currentIndex());
obj1.setCurrentIndex(INT_MAX);
QCOMPARE(-1, obj1.currentIndex());
// QWidget * QStackedLayout::currentWidget()
// void QStackedLayout::setCurrentWidget(QWidget *)
QWidget *var2 = new QWidget();
obj1.addWidget(var2);
obj1.setCurrentWidget(var2);
QCOMPARE(var2, obj1.currentWidget());
obj1.setCurrentWidget((QWidget *)0);
QCOMPARE(obj1.currentWidget(), var2);
delete var2;
}
tst_QStackedLayout::tst_QStackedLayout()
: testWidget(0)
{
}
tst_QStackedLayout::~tst_QStackedLayout()
{
}
void tst_QStackedLayout::initTestCase()
{
}
void tst_QStackedLayout::cleanupTestCase()
{
}
void tst_QStackedLayout::init()
{
if (testWidget) {
delete testWidget;
testWidget = 0;
}
testWidget = new QWidget(0);
testWidget->resize( 200, 200 );
testWidget->show();
// make sure the tests work with focus follows mouse
QCursor::setPos(testWidget->geometry().center());
testWidget->activateWindow();
QApplication::syncX();
QTest::qWait(250);
}
void tst_QStackedLayout::cleanup()
{
delete testWidget;
testWidget = 0;
}
void tst_QStackedLayout::testCase()
{
QStackedLayout onStack(testWidget);
QStackedLayout *testLayout = &onStack;
testWidget->setLayout(testLayout);
QSignalSpy spy(testLayout,SIGNAL(currentChanged(int)));
// Nothing in layout
QCOMPARE(testLayout->currentIndex(), -1);
QCOMPARE(testLayout->currentWidget(), static_cast<QWidget*>(0));
QCOMPARE(testLayout->count(), 0);
// One widget added to layout
QWidget *w1 = new QWidget(testWidget);
testLayout->addWidget(w1);
QCOMPARE(spy.count(), 1);
QCOMPARE(spy.at(0).at(0).toInt(), 0);
spy.clear();
QCOMPARE(testLayout->currentIndex(), 0);
QCOMPARE(testLayout->currentWidget(), w1);
QCOMPARE(testLayout->count(), 1);
// Another widget added to layout
QWidget *w2 = new QWidget(testWidget);
testLayout->addWidget(w2);
QCOMPARE(testLayout->currentIndex(), 0);
QCOMPARE(testLayout->currentWidget(), w1);
QCOMPARE(testLayout->indexOf(w2), 1);
QCOMPARE(testLayout->count(), 2);
// Change the current index
testLayout->setCurrentIndex(1);
QCOMPARE(spy.count(), 1);
QCOMPARE(spy.at(0).at(0).toInt(), 1);
spy.clear();
QCOMPARE(testLayout->currentIndex(), 1);
QCOMPARE(testLayout->currentWidget(), w2);
// First widget removed from layout
testLayout->removeWidget(w1);
QCOMPARE(testLayout->currentIndex(), 0);
QCOMPARE(testLayout->currentWidget(), w2);
QCOMPARE(testLayout->count(), 1);
// Second widget removed from layout; back to nothing
testLayout->removeWidget(w2);
QCOMPARE(spy.count(), 1);
QCOMPARE(spy.at(0).at(0).toInt(), -1);
spy.clear();
QCOMPARE(testLayout->currentIndex(), -1);
QCOMPARE(testLayout->currentWidget(), static_cast<QWidget*>(0));
QCOMPARE(testLayout->count(), 0);
// Another widget inserted at current index.
// Current index should become current index + 1, but the
// current widget should stay the same
testLayout->addWidget(w1);
QCOMPARE(testLayout->currentIndex(), 0);
QCOMPARE(testLayout->currentWidget(), w1);
testLayout->insertWidget(0, w2);
QCOMPARE(testLayout->currentIndex(), 1);
QCOMPARE(testLayout->currentWidget(), w1);
QVERIFY(w1->isVisible());
QVERIFY(!w2->isVisible());
testLayout->setCurrentWidget(w2);
// Another widget added, so we have: w2, w1, w3 with w2 current
QWidget *w3 = new QWidget(testWidget);
testLayout->addWidget(w3);
QCOMPARE(testLayout->indexOf(w2), 0);
QCOMPARE(testLayout->indexOf(w1), 1);
QCOMPARE(testLayout->indexOf(w3), 2);
// Set index to 1 and remove that widget (w1).
// Then, current index should still be 1, but w3
// should be the new current widget.
testLayout->setCurrentIndex(1);
testLayout->removeWidget(w1);
QCOMPARE(testLayout->currentIndex(), 1);
QCOMPARE(testLayout->currentWidget(), w3);
QVERIFY(w3->isVisible());
// Remove the current widget (w3).
// Since it's the last one in the list, current index should now
// become 0 and w2 becomes the current widget.
testLayout->removeWidget(w3);
QCOMPARE(testLayout->currentIndex(), 0);
QCOMPARE(testLayout->currentWidget(), w2);
QVERIFY(w2->isVisible());
// Make sure index is decremented when we remove a widget at index < current index
testLayout->addWidget(w1);
testLayout->addWidget(w3);
testLayout->setCurrentIndex(2);
testLayout->removeWidget(w2); // At index 0
QCOMPARE(testLayout->currentIndex(), 1);
QCOMPARE(testLayout->currentWidget(), w3);
QVERIFY(w3->isVisible());
testLayout->removeWidget(w1); // At index 0
QCOMPARE(testLayout->currentIndex(), 0);
QCOMPARE(testLayout->currentWidget(), w3);
QVERIFY(w3->isVisible());
testLayout->removeWidget(w3);
QCOMPARE(testLayout->currentIndex(), -1);
QCOMPARE(testLayout->currentWidget(), static_cast<QWidget*>(0));
}
void tst_QStackedLayout::deleteCurrent()
{
QStackedLayout *testLayout = new QStackedLayout(testWidget);
QWidget *w1 = new QWidget;
testLayout->addWidget(w1);
QWidget *w2 = new QWidget;
testLayout->addWidget(w2);
QCOMPARE(testLayout->currentWidget(), w1);
delete testLayout->currentWidget();
QCOMPARE(testLayout->currentWidget(), w2);
}
void tst_QStackedLayout::removeWidget()
{
if (testWidget->layout()) delete testWidget->layout();
QVBoxLayout *vbox = new QVBoxLayout(testWidget);
QPushButton *top = new QPushButton("top", testWidget); //add another widget that can receive focus
top->setObjectName("top");
vbox->addWidget(top);
QStackedLayout *testLayout = new QStackedLayout();
QPushButton *w1 = new QPushButton("1st", testWidget);
w1->setObjectName("1st");
testLayout->addWidget(w1);
QPushButton *w2 = new QPushButton("2nd", testWidget);
w2->setObjectName("2nd");
testLayout->addWidget(w2);
vbox->addLayout(testLayout);
top->setFocus();
QTest::qWait(100);
top->activateWindow();
QTest::qWait(100);
int i =0;
for (;;) {
if (QApplication::focusWidget() == top)
break;
else if (i >= 5)
QSKIP("Can't get focus");
QTest::qWait(100);
++i;
}
QCOMPARE(QApplication::focusWidget(), static_cast<QWidget *>(top));
// focus should stay at the 'top' widget
testLayout->removeWidget(w1);
QCOMPARE(QApplication::focusWidget(), static_cast<QWidget *>(top));
}
class LineEdit : public QLineEdit
{
public:
LineEdit() : hasFakeEditFocus(false)
{ }
bool hasFakeEditFocus;
protected:
bool isSingleFocusWidget() const
{
const QWidget *w = this;
while ((w = w->nextInFocusChain()) != this) {
if (w->isVisible() && static_cast<const QWidget*>(w->focusProxy()) != this
&& w->focusPolicy() & Qt::TabFocus) {
return false;
}
}
return true;
}
void focusInEvent(QFocusEvent *event)
{
QLineEdit::focusInEvent(event);
hasFakeEditFocus = isSingleFocusWidget();
}
void focusOutEvent(QFocusEvent *event)
{
hasFakeEditFocus = false;
QLineEdit::focusOutEvent(event);
}
};
void tst_QStackedLayout::keepFocusAfterSetCurrent()
{
if (testWidget->layout()) delete testWidget->layout();
QStackedLayout *stackLayout = new QStackedLayout(testWidget);
testWidget->setFocusPolicy(Qt::NoFocus);
LineEdit *edit1 = new LineEdit;
LineEdit *edit2 = new LineEdit;
stackLayout->addWidget(edit1);
stackLayout->addWidget(edit2);
stackLayout->setCurrentIndex(0);
testWidget->show();
QApplication::setActiveWindow(testWidget);
QVERIFY(QTest::qWaitForWindowActive(testWidget));
edit1->setFocus();
edit1->activateWindow();
QTRY_VERIFY(edit1->hasFocus());
stackLayout->setCurrentIndex(1);
QVERIFY(!edit1->hasFocus());
QVERIFY(edit2->hasFocus());
QVERIFY(edit2->hasFakeEditFocus);
}
void tst_QStackedLayout::heigthForWidth()
{
if (testWidget->layout()) delete testWidget->layout();
QStackedLayout *stackLayout = new QStackedLayout(testWidget);
QLabel *shortLabel = new QLabel("This is a short text.");
shortLabel->setWordWrap(true);
stackLayout->addWidget(shortLabel);
QLabel *longLabel = new QLabel("Write code once to target multiple platforms\n"
"Qt allows you to write advanced applications and UIs once, "
"and deploy them across desktop and embedded operating systems "
"without rewriting the source code saving time and development cost.\n\n"
"Create amazing user experiences\n"
"Whether you prefer C++ or JavaScript, Qt provides the building blocks - "
"a broad set of customizable widgets, graphics canvas, style engine "
"and more that you need to build modern user interfaces. "
"Incorporate 3D graphics, multimedia audio or video, visual effects, "
"and animations to set your application apart from the competition.");
longLabel->setWordWrap(true);
stackLayout->addWidget(longLabel);
stackLayout->setCurrentIndex(0);
int hfw_index0 = stackLayout->heightForWidth(200);
stackLayout->setCurrentIndex(1);
QCOMPARE(stackLayout->heightForWidth(200), hfw_index0);
}
QTEST_MAIN(tst_QStackedLayout)
#include "tst_qstackedlayout.moc"