Details
-
Bug
-
Resolution: Done
-
P2: Important
-
5.5.0 RC
-
None
Description
We start to support Box model in QTBUG-41559, then we totally added 5 qreal and 4 bool like:
qreal padding; qreal topPadding; qreal leftPadding; qreal rightPadding; qreal bottomPadding; bool explicitTopPadding; bool explicitLeftPadding; bool explicitRightPadding; bool explicitBottomPadding;
It could hurt performance a lot. We should move them into extraData.
Current fact:
- Text: most in extraData except padding
- TextEdit: all not in extraData yet
- TextInput: all not in extraData yet
- Positioner(Column, Row, Grid, Flow): all not in extraData yet
- Flickable: leftMargin/topMargin/rightMargin/bottomMargin are in AxisData now, special case.
Attachments
Issue Links
- relates to
-
QTBUG-41559 Support for Qt Quick Box model
- Open
For Gerrit Dashboard: QTBUG-46529 | ||||||
---|---|---|---|---|---|---|
# | Subject | Branch | Project | Status | CR | V |
113898,6 | QQuickText: move padding into extraData | dev | qt/qtdeclarative | Status: MERGED | +2 | 0 |
113904,6 | QQuickTextInput: move paddings into extraData | dev | qt/qtdeclarative | Status: MERGED | +2 | 0 |
113938,7 | QQuickTextEdit: move paddings into extraData | dev | qt/qtdeclarative | Status: MERGED | -2 | 0 |
113957,5 | QQuickPositioner: move paddings into extraData | dev | qt/qtdeclarative | Status: MERGED | +2 | 0 |
114258,3 | WIP: Rectangle: add support for padding | dev | qt/qtdeclarative | Status: ABANDONED | -2 | 0 |