[MainView] positioning & clipping don't take into account ListView.header's height at startup

Bug #1202277 reported by Michael Spencer
14
This bug affects 3 people
Affects Status Importance Assigned to Milestone
Ubuntu File Manager App
Fix Released
High
Unassigned
Ubuntu UI Toolkit
Fix Released
Critical
Tim Peeters
ubuntu-ui-toolkit (Ubuntu)
Fix Released
Undecided
Tim Peeters

Bug Description

When starting an application with a ListView, the list's header not aligned with the MainView's header. Instead, the MainView's header is aligned with the first list item in the ListView. When updating the list model, the MainView then aligns the list's header correctly. This problem only seems to occur when first starting an application.

Related branches

Revision history for this message
Michael Spencer (ibelieve) wrote :
Changed in ubuntu-filemanager-app:
importance: Undecided → High
status: New → Triaged
Changed in ubuntu-ui-toolkit:
importance: Undecided → Critical
Changed in ubuntu-ui-toolkit:
assignee: nobody → Tim Peeters (tpeeters)
Revision history for this message
Tim Peeters (tpeeters) wrote :

Confirmed. This code easily shows the issue:

import QtQuick 2.0
import Ubuntu.Components 0.1

MainView {
    width: 400
    height: 600

    Page {
        title: "listview"

        ListView {
            anchors.fill: parent

            header: Rectangle {
                color: "red"
                width: parent.width
                height: units.gu(5)
            }
            model: 500
            delegate: Label {
                text: "number " +index
            }
        }
    }
}

Changed in ubuntu-ui-toolkit:
status: New → Confirmed
Revision history for this message
Tim Peeters (tpeeters) wrote :

I will be on holidays until October 6. Feel free to work on this, or to re-assign.

Changed in ubuntu-ui-toolkit:
assignee: Tim Peeters (tpeeters) → Florian Boucault (fboucault)
Changed in ubuntu-ui-toolkit:
assignee: Florian Boucault (fboucault) → Tim Peeters (tpeeters)
Tim Peeters (tpeeters)
Changed in ubuntu-ui-toolkit:
status: Confirmed → In Progress
Tim Peeters (tpeeters)
Changed in ubuntu-ui-toolkit:
status: In Progress → Confirmed
Tim Peeters (tpeeters)
Changed in ubuntu-ui-toolkit:
status: Confirmed → In Progress
Revision history for this message
Tim Peeters (tpeeters) wrote :

I don't see this bug on maguro in the filemanager app. Was it fixed in the app already?

I can reproduce it with the code I posted above, and a fix for the bug is ready to go in the UITK.

Revision history for this message
PS Jenkins bot (ps-jenkins) wrote :

Fix committed into lp:ubuntu-ui-toolkit at revision None, scheduled for release in ubuntu-ui-toolkit, milestone Unknown

Changed in ubuntu-ui-toolkit:
status: In Progress → Fix Committed
Revision history for this message
Michael Spencer (ibelieve) wrote :

Tim, I've added a desktop/tablet mode with a sidebar, and the problem does not occur when the sidebar is visible. It only occurs when the sidebar is hidden (when the app's with is small enough).

I'm still able to reproduce this with the sidebar hidden.

Revision history for this message
Launchpad Janitor (janitor) wrote :

This bug was fixed in the package ubuntu-ui-toolkit - 0.1.46+14.04.20131126-0ubuntu1

---------------
ubuntu-ui-toolkit (0.1.46+14.04.20131126-0ubuntu1) trusty; urgency=low

  [ Loïc Molinari ]
  * [UbuntuShape] Informed renderer of index and vertex data patterns.
  * [UbuntuShape] Transformed coordinates of textures packed in atlases.
    (LP: #1252736)

  [ Jani Monoses ]
  * Avoid reference errors in OrientationHelper. (LP: #1244631)

  [ Albert Astals ]
  * Do not include the QtQml catch-all include header Include the
    headers we need instead .
  * Compile properly with make -j10 Otherwise tries to compile the tests
    before the components.so is there and fails.

  [ Christian Dywan ]
  * Only inspect left hand for keywords and omit identifier. (LP:
    #1250653)
  * Merge pot update scripts and tweak 'make pot'. (LP: #1186250)
  * Add dependencies and indexes to link Qt5 and Ubuntu.Unity.Action
    docs. (LP: #1202561)
  * Implement BUILTINS in qmlapicheck to exclude built-in types.
  * Move ubuntu-ui-toolkit-doc dep from -examples to Recommends This way
    it's possible to run autopilot tests without docs.

  [ Zsombor Egri ]
  * Tabs are controlled independently by TabBar from Tabs component.
    (LP: #1246758)

  [ Florian Boucault ]
  * Conditionally disable v8-private dependency if Qt's version >= 5.2.
  * TextArea: added missing MathUtils import. (LP: #1248731)
  * Homogenized the icon related APIs. Everywhere where an icon can be
    set 2 properties are defined: url iconSource: url of any image file
    that can be used as the icon. string iconName: name of the icon in
    the theme. Redundant APIs are now deprecated. Action, ActionItem,
    OptionSelectorDelegate, ListItems.Base and ListItems.Standard have
    been modified. (LP: #1191054)

  [ Leo Arias ]
  * Add the .py files to the qtcreator project. (LP: #1244615)
  * Added logging for all the public methods on the autopilot emulators.

  [ Robert Bruce Park ]
  * Fix indicator tabs ordering.

  [ Michael Zanetti ]
  * fix typo in ListItems.Empty doc string.

  [ tpeeters ]
  * Align ListView header with MainView header. (LP: #1202403, #1202277)
  * Support initial value of selectionMode that is not equal to false.
    (LP: #1250194)

  [ Ubuntu daily release ]
  * Automatic snapshot from revision 859
 -- Ubuntu daily release <email address hidden> Tue, 26 Nov 2013 01:11:34 +0000

Changed in ubuntu-ui-toolkit (Ubuntu):
status: New → Fix Released
Tim Peeters (tpeeters)
Changed in ubuntu-ui-toolkit:
status: Fix Committed → Fix Released
Changed in ubuntu-ui-toolkit (Ubuntu):
assignee: nobody → Tim Peeters (tpeeters)
David Planella (dpm)
Changed in ubuntu-filemanager-app:
status: Triaged → Fix Released
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Duplicates of this bug

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.