Uploaded image for project: 'Qt'
  1. Qt
  2. QTBUG-46529

move padding to extraData for padding support

XMLWordPrintable

      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.

        No reviews matched the request. Check your Options in the drop-down menu of this sections header.

            liaqi Liang Qi
            liaqi Liang Qi
            Votes:
            0 Vote for this issue
            Watchers:
            1 Start watching this issue

              Created:
              Updated:
              Resolved:

                There are no open Gerrit changes