Added global.cfg file that lists the includes needed to create the
binary compatibility test file. Also added the binary compatibility file
to test 5.10 content.
Change-Id: I9ddf9000991452c63d491695fb30c5c27199164d
Reviewed-by: J-P Nurmi <jpnurmi@qt.io>
Reviewed-by: Sergio Ahumada <sahumada@texla.cl>
Action caused checked state of the button to be reset back to its
original value if checked status had been change through click or key
press. To avoid this, separate private function has been added to that
does not change the status.
Task-number: QTBUG-65108
Change-Id: I8a5aaa9aab739db3ba10d0202f5fb718cc7195bd
Reviewed-by: J-P Nurmi <jpnurmi@qt.io>
Currently, there's no explicit mention of being able to customize the
font via the qtquickcontrols2.conf file.
Change-Id: I0c62e19b721a2d652021c303f876aaaa1098b880
Reviewed-by: J-P Nurmi <jpnurmi@qt.io>
Currently, there's no explicit mention of being able to customize the
palette via the qtquickcontrols2.conf file.
Change-Id: I11f16a4e7dd759de2121e986df17ae9fdc651180
Reviewed-by: J-P Nurmi <jpnurmi@qt.io>
QQuickIconLabel is a wrapper around QQuickText & QQuickImage. While
the alignment is mostly handled by layouting the internal QQuickText
and QQuickImage instances, it must also propagate the alignment so
that the text and image elements themselves also apply the correct
content alignment.
Task-number: QTBUG-64492
Change-Id: Ieb9d26fce703247bc6e2e61fa38e44c29937d1ad
Reviewed-by: Liang Qi <liang.qi@qt.io>
A regression introduced by 0504150. A wrong binding for the x-coordinate
was chosen while merging CheckIndicator back to CheckDelegate.
Task-number: QTBUG-64430
Change-Id: Ia3b07e7537cc888406076c700ecd495a958cb7c1
Reviewed-by: Liang Qi <liang.qi@qt.io>
To make them correctly available in QML so that they also work as
signal arguments.
Task-number: QTBUG-64259
Change-Id: I5fa00871f96357d924661c4b8af323a0f1748c9d
Reviewed-by: Liang Qi <liang.qi@qt.io>
We've come to realize that even though it's tempting to group similar
properties together, organizing the API so that revisions are grouped
together makes future maintenance more pleasant. It's a lot easier to
to see what was added and when.
The same was done for earlier revisions in 5.9 in commit 430fe83.
Change-Id: I738d7fdadd348c21737228c37d0f31e39b37f8e7
Reviewed-by: J-P Nurmi <jpnurmi@qt.io>
We've come to realize that even though it's tempting to group similar
properties together, organizing the API so that revisions are grouped
together makes future maintenance more pleasant. It's a lot easier to
to see what was added and when.
Change-Id: I47ba7725260f2c259048848cc2a9b17bce2f01c7
Reviewed-by: J-P Nurmi <jpnurmi@qt.io>
QQuickTextField wants to emit pressed(), released(), and pressAndHold()
signals not only on left, but on all mouse button clicks. Therefore it
explicitly sets accepted mouse buttons to all. However, the base class,
QQuickTextInput, has only been prepared to handle left (selection) and
middle (paste) mouse buttons. Don't pass right mouse button -only
events to the base class to avoid it clearing selection on right click.
Task-number: QTBUG-64048
Change-Id: I015820c78995cc8dd7cbeb16fca0f9b991040bad
Reviewed-by: Mitch Curtis <mitch.curtis@qt.io>
This reverts commit 80f6b146e6, because
the optimizations caused a binding loop on the TabBar page in the
Gallery example.
qrc:/pages/TabBarPage.qml:90:13: QML TabBar:
Binding loop detected for property "implicitHeight"
As a result, the tabs are looking broken at least when running with
the Imagine style.
Change-Id: I231bf5b21ecf92000bac94782567301214b64fec
Reviewed-by: Mitch Curtis <mitch.curtis@qt.io>
Make sure dynamically created QML components are destroyed when
going out of scope, and print out the error when creation fails.
Change-Id: Ie0ed785780cb861dd057adda8f4da399dd4986d2
Reviewed-by: Mitch Curtis <mitch.curtis@qt.io>
Unfortunately this comes a bit late in the 5.10 cycle, but this should
be released together with the rest of the mnemonics stuff going out in
5.10, because it affects the value of AbstractButton/Menu(Bar)Item::text.
As the removed TODO comments and altered tests indicate, previously the
ampersand was blatantly stripped out. It would be worse to change it
later once people have already started using mnemonics and rely on the
behavior in custom styles.
The necessary modifications to QQuickText were added in qtdeclarative
commit 65ef4ba.
Task-number: QTBUG-61422
Change-Id: Iaa73da8c012e9a6019743cf98f5bdc02527064e5
Reviewed-by: Mitch Curtis <mitch.curtis@qt.io>
The handle can be positioned "between" pixels, leading to unpleasant
results:
https://imgur.com/a/GKItw
Round the position to avoid this.
Doesn't seem to be an issue for other styles that use Qt Quick
primitives rather than images.
Change-Id: Iddb21e37ff777de433b3fa2a68e640db9a4b369a
Reviewed-by: J-P Nurmi <jpnurmi@qt.io>
Control's padding is supposed to be content padding, not background
padding. The background has a fixed height, so it's enough to set the
Y-coordinate to keep it vertically center aligned.
running: qmlbench/benchmarks/auto/creation/quick.controls2/delegates_progressbar.qml
[...]
Average: 205.8 frames; using samples; MedianAll=206; StdDev=1.30384, CoV=0.00633547
After:
running: qmlbench/benchmarks/auto/creation/quick.controls2/delegates_progressbar.qml
[...]
Average: 214.4 frames; using samples; MedianAll=214; StdDev=3.20936, CoV=0.014969
Change-Id: I4900bf496effa52e9291192e05ed95f4a44fbea4
Reviewed-by: Mitch Curtis <mitch.curtis@qt.io>
A binding to "<id>.<property>" is a tiny bit faster than a binding to
"parent.<property>". ID is a direct reference, like a pointer in C++,
whereas "parent" is a property so the binding is effectively depending
on two properties: "<property>.<property>".
The gain is not huge, but enough to show up in qmlbench. ;)
Before:
running: qmlbench/benchmarks/auto/creation/quick.controls2/delegates_menuitem.qml
[...] (10 repeats, all results between 65-66)
Average: 65.5 frames; using samples; MedianAll=66; StdDev=0.527046, CoV=0.00804651
After:
running: qmlbench/benchmarks/auto/creation/quick.controls2/delegates_menuitem.qml
[...] (10 repeats, all results between 66-67)
Average: 66.6 frames; using samples; MedianAll=67; StdDev=0.516398, CoV=0.00775372
Change-Id: Ib161fad3b11e19a4bb0f39900639131f36838d2a
Reviewed-by: Mitch Curtis <mitch.curtis@qt.io>
The internal CheckIndicator helper was introduced together with
CheckDelegate in 1acb34a, because we naturally wanted to share the
indicator instead of duplicating it. This change is controversial,
because it leads to duplicate code, but keeping the indicator
definitions inline is clearly faster. This is not seen as a huge
problem for the Default style, because the indicator is not too
complicated. Basically, this fixes a ~5% performance regression
introduced by 1acb34a.
Before:
running: qmlbench/benchmarks/auto/creation/quick.controls2/delegates_checkbox.qml
[...]
Average: 72.8 frames; using samples; MedianAll=73; StdDev=1.48324, CoV=0.0203742
After:
running: qmlbench/benchmarks/auto/creation/quick.controls2/delegates_checkbox.qml
[...]
Average: 77 frames; using samples; MedianAll=77; StdDev=1.41421, CoV=0.0183664
Change-Id: Ibee0e29e83a64ee4a6a772a90b1784a9c8c715bb
Reviewed-by: Mitch Curtis <mitch.curtis@qt.io>
Listen to the relevant view changes once in QQuickTumbler and cache
the offset/contentY instead of every QQuickTumblerAttached instance
doing its own geometry and child tracking, and cache the view offset
or contentY so that they are readily available while calculating
displacements for attached property objects.
This gives a 5% boost (40->42 frames in qmlbench) on TX1 in release
mode.
Change-Id: If1a77468e812e65bc07f32216ff9bf2e1dc5b935
Reviewed-by: Mitch Curtis <mitch.curtis@qt.io>
Add an internal CheckLabel helper that simply initializes a few
QQuickText properties with defaults that are suitable for CheckBox,
Switch, and RadioButton. This gives a 10% boost on desktop in debug
mode, and a 5% boost on a TX1 in release mode.
Change-Id: I82fead9ca22b6aa74f24924d240c924b2a42a912
Reviewed-by: Mitch Curtis <mitch.curtis@qt.io>
Mitch Curtis
Oswald Buddenhagen
J-P Nurmi
Milla Pohjanheimo
Joni Poikelin
Liang Qi
Antti Kokko
Konstantin Ritt
Friedemann Kleint