Merge lp://qastaging/~fboucault/address-book-app/multicolumns into lp://qastaging/address-book-app

Proposed by Florian Boucault
Status: Merged
Approved by: Renato Araujo Oliveira Filho
Approved revision: 474
Merged at revision: 490
Proposed branch: lp://qastaging/~fboucault/address-book-app/multicolumns
Merge into: lp://qastaging/address-book-app
Prerequisite: lp://qastaging/~fboucault/address-book-app/toolkit13
Diff against target: 349 lines (+63/-59)
11 files modified
src/imports/ABContactListPage.qml (+25/-18)
src/imports/ABContactViewPage.qml (+14/-9)
src/imports/MainWindow.qml (+8/-16)
src/imports/Settings/SettingsPage.qml (+1/-1)
src/imports/Ubuntu/AddressBook/ContactEditor/ContactEditorPage.qml (+3/-4)
src/imports/Ubuntu/AddressBook/ContactShare/ContactSharePage.qml (+2/-2)
src/imports/Ubuntu/Contacts/ContactListView.qml (+4/-4)
src/imports/Ubuntu/Contacts/ContactSimpleListView.qml (+2/-1)
src/imports/Ubuntu/Contacts/ListItemWithActions.qml (+1/-1)
src/imports/Ubuntu/Contacts/PageWithBottomEdge.qml (+1/-1)
src/imports/Ubuntu/Contacts/SIMCardImportPage.qml (+2/-2)
To merge this branch: bzr merge lp://qastaging/~fboucault/address-book-app/multicolumns
Reviewer Review Type Date Requested Status
Renato Araujo Oliveira Filho (community) Approve
PS Jenkins bot continuous-integration Needs Fixing
Review via email: mp+270113@code.qastaging.launchpad.net

Commit message

Use AdaptivePageLayout instead of PageStack to support multi column layouts.

To post a comment you must log in.
Revision history for this message
PS Jenkins bot (ps-jenkins) wrote :
review: Needs Fixing (continuous-integration)
472. By Florian Boucault

Merged from toolkit13

Revision history for this message
PS Jenkins bot (ps-jenkins) wrote :
review: Needs Fixing (continuous-integration)
Revision history for this message
Renato Araujo Oliveira Filho (renatofilho) wrote :

- Is correct that the edit page has a small header than other pages?
- Is correct that when opening the edit page from the bottom edge it fills only the right side page? Keeping the left side page empty?

473. By Florian Boucault

Merged from toolkit13

Revision history for this message
Florian Boucault (fboucault) wrote :

> - Is correct that the edit page has a small header than other pages?

I'm not sure why the new contact page has a smaller header. That will probably be fixed anyway with the next MR that changes the bottom edge.

> - Is correct that when opening the edit page from the bottom edge it fills
> only the right side page? Keeping the left side page empty?

I intend to fix that in the next merge request as this is not good UX wise but not a regression.

Revision history for this message
PS Jenkins bot (ps-jenkins) wrote :
review: Needs Fixing (continuous-integration)
474. By Florian Boucault

Merged from toolkit13

Revision history for this message
PS Jenkins bot (ps-jenkins) wrote :
review: Needs Fixing (continuous-integration)
Revision history for this message
Renato Araujo Oliveira Filho (renatofilho) wrote :

Looks good.

review: Approve

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
The diff is not available at this time. You can reload the page or download it.

Subscribers

People subscribed via source and target branches