ListItem.ItemSelector, scrolling doesn't work by default
Bug #1256356 reported by
Sebastien Bacher
This bug affects 1 person
Affects | Status | Importance | Assigned to | Milestone | |
---|---|---|---|---|---|
Ubuntu UI Toolkit |
Fix Released
|
Undecided
|
Nicolas d'Offay | ||
ubuntu-ui-toolkit (Ubuntu) |
Fix Released
|
Undecided
|
Nicolas d'Offay |
Bug Description
Using current trusty, that example doesn't scroll
"import QtQuick 2.0
import Ubuntu.Components 0.1
import Ubuntu.
MainView {
height: 300
width: 150
ListItem.
expanded: true
model: ["list", "of", "options", "needing", "some", "vertical", "scrolling"]
}
}"
It works if you add "containerHeight: height"... shouldn't that be the default?
Related branches
lp:~nicolas-doffay/ubuntu-ui-toolkit/fix-1256356
- Florian Boucault (community): Approve
- PS Jenkins bot: Approve (continuous-integration)
-
Diff: 125 lines (+34/-10)3 files modifiedexamples/ubuntu-ui-toolkit-gallery/OptionSelectors.qml (+6/-6)
modules/Ubuntu/Components/ListItems/ItemSelector.qml (+15/-2)
modules/Ubuntu/Components/OptionSelector.qml (+13/-2)
Changed in ubuntu-ui-toolkit: | |
status: | New → Confirmed |
Changed in ubuntu-ui-toolkit: | |
status: | Confirmed → In Progress |
Changed in ubuntu-ui-toolkit (Ubuntu): | |
status: | New → Fix Committed |
assignee: | nobody → Nicolas d'Offay (nicolas-doffay) |
Changed in ubuntu-ui-toolkit: | |
status: | Fix Committed → Fix Released |
To post a comment you must log in.
that example has no "text" in use, the default should probably be "containerHeight: height - text_height - spacing" when it has text