PageHeadState doens't correctly update head.contents
Bug #1400297 reported by
Giulio Collura
This bug report is a duplicate of:
Bug #1341814: Using search in the header can sometimes have a text field from a different tab.
Edit
Remove
This bug affects 1 person
Affects | Status | Importance | Assigned to | Milestone | |
---|---|---|---|---|---|
ubuntu-ui-toolkit (Ubuntu) |
Fix Released
|
Undecided
|
Giulio Collura | ||
Vivid |
New
|
Undecided
|
Unassigned | ||
ubuntu-ui-toolkit (Ubuntu RTM) |
New
|
Undecided
|
Unassigned |
Bug Description
See this small example code: http://
As you can see, I set both PageHeadState to change their 'contents' property when changing state. The expected behavior would be to have in the page head a Label when state is "default" and a TextEntry when in "search" state. However after some interactions (switching state 2 times) both the Label and the Entry are shown.
As this doesn't happen with other properties, this should not happen with contents.
I can set a 'visible' property to each component as workaround though.
Thanks,
Giulio
Related branches
lp:~gcollura/ubuntu-ui-toolkit/fix-1341814-and-1400297
Superseded
for merging
into
lp:ubuntu-ui-toolkit/staging
- Zoltan Balogh: Pending requested
- Riccardo Padovani: Pending requested
-
Diff: 1081 lines (+161/-142)55 files modifiedexamples/ubuntu-ui-toolkit-gallery/po/es.po (+1/-1)
modules/Ubuntu/Components/PageHeadConfiguration.qml (+11/-0)
po/am.po (+2/-2)
po/ar.po (+2/-2)
po/ast.po (+2/-2)
po/az.po (+2/-2)
po/be.po (+2/-2)
po/bs.po (+2/-2)
po/ca.po (+3/-3)
po/ca@valencia.po (+2/-2)
po/ce.po (+2/-2)
po/cs.po (+2/-2)
po/da.po (+2/-2)
po/de.po (+2/-2)
po/el.po (+2/-2)
po/en_AU.po (+2/-2)
po/en_GB.po (+2/-2)
po/es.po (+1/-1)
po/eu.po (+2/-2)
po/fa.po (+1/-1)
po/fi.po (+2/-2)
po/fr.po (+4/-4)
po/fr_CA.po (+2/-2)
po/ga.po (+23/-16)
po/gd.po (+2/-2)
po/gl.po (+4/-4)
po/he.po (+2/-2)
po/hr.po (+2/-2)
po/hu.po (+2/-2)
po/id.po (+2/-2)
po/it.po (+2/-2)
po/ja.po (+2/-2)
po/km.po (+2/-2)
po/kn.po (+2/-2)
po/ko.po (+4/-4)
po/lv.po (+2/-2)
po/ms.po (+2/-2)
po/my.po (+2/-2)
po/nb.po (+4/-4)
po/nl.po (+2/-2)
po/oc.po (+2/-2)
po/pa.po (+2/-2)
po/pl.po (+2/-2)
po/pt.po (+4/-4)
po/pt_BR.po (+2/-2)
po/ro.po (+2/-2)
po/ru.po (+2/-2)
po/sq.po (+2/-2)
po/sr.po (+2/-2)
po/sv.po (+2/-2)
po/tr.po (+12/-11)
po/ug.po (+3/-3)
po/uk.po (+4/-4)
po/zh_CN.po (+2/-2)
po/zh_TW.po (+2/-2)
Superseded
for merging
into
lp:ubuntu-ui-toolkit
- Zoltan Balogh: Needs Fixing
- Riccardo Padovani (community): Approve
-
Diff: 23 lines (+11/-0)1 file modifiedmodules/Ubuntu/Components/PageHeadConfiguration.qml (+11/-0)
lp:~gcollura/ubuntu-ui-toolkit/staging-fix-1341814-and-1400297
- PS Jenkins bot: Approve (continuous-integration)
- Tim Peeters: Approve
- Zoltan Balogh: Pending requested
- Riccardo Padovani: Pending requested
-
Diff: 25 lines (+15/-0)1 file modifiedmodules/Ubuntu/Components/PageHeadConfiguration.qml (+15/-0)
Changed in ubuntu-ui-toolkit (Ubuntu): | |
assignee: | nobody → Giulio Collura (gcollura) |
status: | New → Fix Committed |
To post a comment you must log in.
This seems similar to bug 1341814 where the head contents are not correctly handled when switching page head state.