| Commit message (Collapse) | Author | Age | Files | Lines |
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
The initial drag threshold is a necessary evil on touch to avoid
conflicting with flickables, but leads to bad experience (QTBUG-47081)
when using a mouse. Now that we have separate mouse and touch handling,
we can apply immediate moves when using a mouse, but keep the old
behavior on touch.
Changing the behavior is simple, but auto tests need quite many tweaks
so RangeSlider and Dial will be done separate follow up changes.
[ChangeLog][Important Behavior Changes] Sliders and Dials now react
immediately when using a mouse. Now the initial drag threshold applies
only on touch, to avoid conflicting with flickables.
Task-number: QTBUG-59920
Change-Id: Ifc1e6ed74c7894c3c854c975dff5238278827590
Reviewed-by: Mitch Curtis <mitch.curtis@qt.io>
|
|
|
|
|
|
|
|
|
|
|
|
| |
For example, if you have a floating button on top of a text editor,
hovering the button must change the cursor from the editor's ibeam
cursor to an arrow cursor. This applies to all interactive controls
that call setAcceptedMouseButtons(). If a control blocks mouse events,
it should not use some random cursor from another control underneath.
Task-number: QTBUG-59629
Change-Id: I8a6ae306bbc76a9b22377361cb19cf9c3a872d31
Reviewed-by: Mitch Curtis <mitch.curtis@qt.io>
|
|
|
|
|
|
| |
Task-number: QTBUG-59309
Change-Id: I3c19d55431dd4831c4510acb5ef0e9fffa8f478f
Reviewed-by: Mitch Curtis <mitch.curtis@qt.io>
|
|
|
|
|
|
|
|
| |
Scrollbar over TextArea had Qt::IBeamCursor, although the textarea would not be activated when clicked.
Task-number: QTBUG-59629
Change-Id: Ie96ba03360fcfb5872f5bb6345e168c987978aca
Reviewed-by: J-P Nurmi <jpnurmi@qt.io>
|
|
|
|
|
|
|
|
| |
Random values of the m_elapsed could make the time in updateCurrentTime invalid.
Task-number: QTBUG-59911
Change-Id: I8d36f50f01679f56300df9bae00a8d5de6528cc3
Reviewed-by: J-P Nurmi <jpnurmi@qt.io>
|
|
|
|
|
|
|
| |
The signal was added in Qt Quick 2.9 and has revision 9.
Change-Id: Id1bb0f17ac5d8c9684ab5127782d030b54756e6f
Reviewed-by: Mitch Curtis <mitch.curtis@qt.io>
|
|
|
|
|
|
|
|
|
| |
The signals have been there since the first release. The docs were
just missing.
Change-Id: I5f5340859136b8924832da4ddb2a18d4a812cca3
Task-numer: QTBUG-59400
Reviewed-by: Mitch Curtis <mitch.curtis@qt.io>
|
|
|
|
|
|
|
|
| |
To use the same "was introduced in Qt 5.7" phrasing already used at the
top of the page.
Change-Id: I3736a1c3e87aa91e7fa3b8a41370f4b2252e793d
Reviewed-by: Mitch Curtis <mitch.curtis@qt.io>
|
|
|
|
|
|
|
| |
To make it more clear what the controls are and how they look.
Change-Id: I6ce9a56a1692d90900952be798d7b254bc067dbb
Reviewed-by: Mitch Curtis <mitch.curtis@qt.io>
|
|
|
|
|
|
|
|
|
|
| |
The version table added in c9cd624 looked fine in locally generated
docs, but it seems to get a bit too wide in the online snapshots (the
table becomes horizontally scrollable). => Merge the controls and
styles columns that all have the same version.
Change-Id: Icb8387f95211ddee030cb410594adc9a6fad5c97
Reviewed-by: Mitch Curtis <mitch.curtis@qt.io>
|
|
|
|
|
|
|
|
|
| |
This was mentioned in the detailed description of DialogButtonBox,
but could easily be missed by users looking at Dialog's standardButtons
documentation, because that doesn't mention DialogButtonBox.
Change-Id: If3229589e24992d3071388a044246906e739a918
Reviewed-by: J-P Nurmi <jpnurmi@qt.io>
|
|
|
|
|
|
|
|
| |
qtquickcontrols2-customize.qdoc:28: Can't link to 'Creating Qt Quick Projects'
qtquickcontrols2-customize.qdoc:28: warning: Can't link to 'Creating C++ Classes'
Change-Id: I5ddc57215fd65fa6a80819ee694b5e031676f5a7
Reviewed-by: Mitch Curtis <mitch.curtis@qt.io>
|
|
|
|
|
|
|
| |
The property was added in 4a97498.
Change-Id: I0f21f499e74450780db85d5c2c92aa4a0367c3b5
Reviewed-by: Mitch Curtis <mitch.curtis@qt.io>
|
|
|
|
|
| |
Change-Id: I5bce83d66615f11588988947d088d4b6ce5240b2
Reviewed-by: Mitch Curtis <mitch.curtis@qt.io>
|
|
|
|
|
|
|
| |
The Window attached property has not been used since 3bbd90c7.
Change-Id: I09e0c620aa83533c98316f50d6bfe3fd292761d4
Reviewed-by: Mitch Curtis <mitch.curtis@qt.io>
|
|
|
|
|
|
|
|
| |
Native text rendering was removed in 3f5a66eb at the same time
SwipeDelegate was being developed.
Change-Id: I14eb1f335963abb9ead9450d62262990a3abc7d9
Reviewed-by: Mitch Curtis <mitch.curtis@qt.io>
|
|
|
|
|
|
|
|
|
|
|
| |
Dial::valueAt() does not exist.
qquickdial.cpp:515: warning: Can't link to 'valueAt()'
The broken see also link was accidentally copied from Slider.
Change-Id: I781b7d0bcf15d3b34dd2b8784a273d5cd7218469
Reviewed-by: Mitch Curtis <mitch.curtis@qt.io>
|
|
|
|
|
|
|
|
| |
To avoid crashes in the future.
Task-number: QTBUG-59532
Change-Id: Ie81f95939fc5655567f9d987cf040daf3a32df43
Reviewed-by: J-P Nurmi <jpnurmi@qt.io>
|
|
|
|
|
|
|
|
|
| |
Check for valid URLs and test object types. Throw warnings when
appropriate instead of crashing later.
Task-number: QTBUG-59634
Change-Id: Ia269dc8afd31b618f1ff7aec94d684029cb78244
Reviewed-by: Mitch Curtis <mitch.curtis@qt.io>
|
|
|
|
|
|
|
|
|
| |
Store the current operation name internally to allow throwing
informative warnings when component creation fails.
Task-number: QTBUG-59634
Change-Id: I7506c5eb4be35878c8abd73a637357e0d8ae08a2
Reviewed-by: Mitch Curtis <mitch.curtis@qt.io>
|
|
|
|
|
|
|
|
|
|
| |
Previously it just dumped the error from the component, without telling
that it was anyhow related to StackView. Now it uses the standard QML
warning syntax and includes the name of the operation (push/replace).
Task-number: QTBUG-59634
Change-Id: Iea8df51f587c243ca065627e59bb0e5b5b526664
Reviewed-by: Mitch Curtis <mitch.curtis@qt.io>
|
|
|
|
|
|
|
|
|
| |
This fixes a crash in QQuickSwipeDelegatePriavte's destructor when
using QML debugging.
Task-number: QTBUG-59532
Change-Id: I55f97039eb723652fec24e4c3a8ce5d1a8fb6c41
Reviewed-by: J-P Nurmi <jpnurmi@qt.io>
|
|
|
|
|
|
|
| |
Task-number: QTBUG-59661
Change-Id: I35627bafc57266cb7b2842fe02765536ca7eecba
Reviewed-by: Mitch Curtis <mitch.curtis@qt.io>
Reviewed-by: Qt CI Bot <qt_ci_bot@qt-project.org>
|
|
|
|
|
|
|
|
|
| |
The documentation body was clearly copied from Control::font. Locale
doesn't have mutable properties. Instead, one has to create a locale
using Qt.locale(). Change the wording to match that.
Change-Id: I637b00808c29c16ee768820be327e3c780cfee64
Reviewed-by: Mitch Curtis <mitch.curtis@qt.io>
|
|
|
|
|
| |
Change-Id: I799eb73a05842b1990b5615fd01ad5204c554f20
Reviewed-by: Mitch Curtis <mitch.curtis@qt.io>
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
If QQuickComboBox was destroyed while its popup was open, it emitted
highlightedIndexChanged() during the destruction due to the popup's
visibility change. Use setPopup(nullptr) to destroy the popup in a
controlled manner, that is, disconnecting the relevant signals first.
This way deleting the popup doesn't have weird side-effects during
the destruction of QQuickComboBox.
Task-number: QTBUG-57650
Change-Id: I1c55707be68a24e115276ffb6dfa1c89051a45ea
Reviewed-by: Mitch Curtis <mitch.curtis@qt.io>
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
qmlRegisterRevision() registers a nameless type, so using it would make
AbstractButton throw confusingly formatted warnings:
QML : <warning>
instead of
QML AbstractButton: <warning>
Change-Id: I6b51824ad019ed9451079cf8a2595e908ef8187e
Reviewed-by: Mitch Curtis <mitch.curtis@qt.io>
|
|
|
|
|
| |
Change-Id: Ie371dced41ed86373aeb0a4600cb0245b4550b7e
Reviewed-by: J-P Nurmi <jpnurmi@qt.io>
|
|
|
|
|
| |
Change-Id: I0352376201a1efd318f89a758dd8dc5a269742f4
Reviewed-by: J-P Nurmi <jpnurmi@qt.io>
|
|
|
|
|
| |
Change-Id: Idcd36f81abf0968c7e85a04c0faf340e4309ef18
Reviewed-by: Thomas Hartmann <thomas.hartmann@qt.io>
|
|
|
|
|
|
|
|
| |
The snippet was removed in b026ebf67 when Slider::live was added, but
it was not noticed that the snippet was also used in Tooltip docs.
Change-Id: I8dac3bb90bc44ca447f9e34d0015443e6245d894
Reviewed-by: Nico Vertriest <nico.vertriest@qt.io>
|
|
|
|
|
|
| |
Change-Id: If141c5dd6a0126c49f508590a5addb7f8e4b2159
Reviewed-by: J-P Nurmi <jpnurmi@qt.io>
Reviewed-by: Tim Jenssen <tim.jenssen@qt.io>
|
|
|
|
|
| |
Change-Id: I869b25ff2c9efe1c21def0baf4de0606c7c9baa2
Reviewed-by: J-P Nurmi <jpnurmi@qt.io>
|
|
|
|
|
| |
Change-Id: If991e2311f37c1977cdf2a913b3f022e5f999f5f
Reviewed-by: J-P Nurmi <jpnurmi@qt.io>
|
|\
| |
| |
| |
| |
| |
| |
| |
| | |
Conflicts:
src/imports/controls/universal/qtquickcontrols2universalstyleplugin.cpp
src/quicktemplates2/qquicktextarea.cpp
src/quicktemplates2/qquicktextfield.cpp
Change-Id: Ie80a2d3bcbb961a38e3dffa247547d3d860c231a
|
| |
| |
| |
| |
| |
| |
| |
| | |
For the internal Radio and CheckIndicator types.
Task-number: QTBUG-59026
Change-Id: I7e34f29e33ebd3d748892df0ba1592fe8ae5ed44
Reviewed-by: Mitch Curtis <mitch.curtis@qt.io>
|
| |
| |
| |
| |
| |
| |
| |
| | |
Read-only editors should not use the IBeam cursor, but switch to
a plain arrow cursor.
Change-Id: If579571ceff086af6b8259ade769b5efb27565db
Reviewed-by: Mitch Curtis <mitch.curtis@qt.io>
|
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| | |
Because QQuickApplicationWindow connected
SIGNAL(activeFocusItemChanged())
to
SLOT(_q_updateActiveFocus()));
it would enter _q_updateActiveFocus() from the dtor of QQuickWindow
(because dtor of QQuickWindow would emit activeFocusItemChanged()).
At that point the QQuickApplicationWindow object for the member function
_q_updateActiveFocus() was already destroyed, so we would crash.
Instead, make sure we clear the focus as early as possible, and then
disconnect in case activeFocusItemChanged() is emitted after
QQuickApplicationWindow is destroyed.
Task-number: QTBUG-57846
Change-Id: I4b1999e647b970394436a2d462b7f352f1c8a811
Reviewed-by: J-P Nurmi <jpnurmi@qt.io>
|
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| | |
When visible has been set true, don't attempt to start the enter
transition from componentComplete(), but wait until the popup is
associated with a window. This makes it possible to share a single
QQuickView instance for all data rows in tst_snippets.
Task-number: QTBUG-58196
Change-Id: I9204e0d2e6468a21ee36f313daedc43d2e919ec6
Reviewed-by: Mitch Curtis <mitch.curtis@qt.io>
|
| |
| |
| |
| |
| | |
Change-Id: I89c037629037a73fd481d29b28bd15b1ba57af09
Reviewed-by: J-P Nurmi <jpnurmi@qt.io>
|
| |
| |
| |
| |
| | |
Change-Id: I5cd37b78c18847c9535584d0452c8214873db5f2
Reviewed-by: J-P Nurmi <jpnurmi@qt.io>
|
| |
| |
| |
| |
| | |
Change-Id: I5e8afbf0aa1704a13d359e7ddf867cd791b3f0db
Reviewed-by: J-P Nurmi <jpnurmi@qt.io>
|
| |
| |
| |
| |
| | |
Change-Id: I606e0e68c16faceaddbddacb37eec7d557e184ce
Reviewed-by: J-P Nurmi <jpnurmi@qt.io>
|
| |
| |
| |
| |
| |
| |
| |
| |
| | |
Implement all the missing things and fix the type of the cast in
QWidgetPlatformMenuItem::setMenu() to make sub menus work with the
widgets fallbacks.
Change-Id: I19bb010a5c639a77b76d1d92a47ad03576c06d2f
Reviewed-by: Mitch Curtis <mitch.curtis@qt.io>
|
| |
| |
| |
| |
| |
| |
| |
| | |
This makes the range that the user has selected clear,
and is already done by other styles.
Change-Id: Ib8f2c81e823193fa9934587d04fc011aafc0bd65
Reviewed-by: J-P Nurmi <jpnurmi@qt.io>
|
| |
| |
| |
| |
| | |
Change-Id: Ie417f6377016e1a28122efefcc9fcef5e642d027
Reviewed-by: J-P Nurmi <jpnurmi@qt.io>
|
| |
| |
| |
| |
| | |
Change-Id: Iaf3159b993ada20a30a8208f7cfe92731966a3f7
Reviewed-by: J-P Nurmi <jpnurmi@qt.io>
|
| |
| |
| |
| |
| |
| |
| |
| | |
The background didn't show up unless setHighQuality(true) was used.
Task-number: QTBUG-59233
Change-Id: Ief04c48b7c82d5c6be2e63de5c515cdf763a2064
Reviewed-by: J-P Nurmi <jpnurmi@qt.io>
|
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| | |
While debugging QTBUG-59026, I found out that QLocale::name() is empty
with the "C" locale. Because the empty locale name was used as a selector
in QQuickStyleSelector, it ended up registering QML types using URLs
that contained a double slash (empty locale selector matched):
QtQuick/Controls.2/Material//RadioButton.qml
At the same time, the QML engine imported implicitly internal QML types
constructing the URLs directly based on the location on the file system
without a double slash:
QtQuick/Controls.2/Material/RadioIndicator.qml
As a result, the same QML module ended up having types registered from
two different URLs, which is not allowed.
This change simply prevents both problems. First of all, if the locale
name is empty for any reason, it won't be added anymore to the list of
selectors. Furthermore, the final selected URL is normalized to avoid
redundant dashes.
Task-number: QTBUG-59026
Change-Id: I19b890451aaddfe4277bb6b26bc3c1394a75a704
Reviewed-by: Mitch Curtis <mitch.curtis@qt.io>
|
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| | |
This seems like a mistake that slipped in when popTransition(),
pushTransition() and replaceTransition() were merged in acecb06.
Only immediate push/pop/replace operations were affected, where
the animations were force-completed by hand. In this scenario it
was always passing the pop transitions for completion, even if
the current operation was push or replace.
Change-Id: I6a940953feaefe6643ddae26c070856a66bf6c10
Reviewed-by: Mitch Curtis <mitch.curtis@qt.io>
|