Replace the current static page managing by object incubator
Affects | Status | Importance | Assigned to | Milestone | |
---|---|---|---|---|---|
Canonical System Image |
In Progress
|
Medium
|
Bill Filler | ||
messaging-app (Ubuntu) |
In Progress
|
Medium
|
Gustavo Pichorim Boiko |
Bug Description
Depending on the contents of the page, its asynchronous creation might take a really long time because of the internal behavior (or bug?) of QtQuick.
Loading pages synchronously fixes the problem, but the way it is implemented in messaging-app, there is a risk of leaking memory due to pages not being deleted properly when switching views.
There is another way to workaround this QtQuick limitation which is to force the page creation to finish before returning to the event loop, in the AdaptivePageLayout context, it would look like this:
var incubator = layout.
incubator.
In order for this to be fixed bug #1544745 in AdaptivePageLayout first needs to be fixed.
Related branches
- system-apps-ci-bot: Needs Fixing (continuous-integration)
- PS Jenkins bot: Needs Fixing (continuous-integration)
- Ubuntu Phablet Team: Pending requested
-
Diff: 296 lines (+48/-83)8 files modifiedsrc/qml/MMS/Previewer.qml (+2/-2)
src/qml/MainPage.qml (+23/-16)
src/qml/Messages.qml (+1/-1)
src/qml/MessagingContactEditorPage.qml (+1/-1)
src/qml/MessagingContactViewPage.qml (+2/-2)
src/qml/MessagingPageLayout.qml (+9/-47)
src/qml/NewRecipientPage.qml (+2/-3)
src/qml/messaging-app.qml (+8/-11)
Changed in messaging-app (Ubuntu): | |
importance: | Undecided → Medium |
assignee: | nobody → Gustavo Pichorim Boiko (boiko) |
Changed in canonical-devices-system-image: | |
assignee: | nobody → Bill Filler (bfiller) |
importance: | Undecided → Medium |
description: | updated |
Changed in canonical-devices-system-image: | |
status: | New → In Progress |
Changed in messaging-app (Ubuntu): | |
status: | New → In Progress |