address-book-app

Merge lp://qastaging/~renatofilho/address-book-app/fix-button-colors into lp://qastaging/~phablet-team/address-book-app/staging

  1. fix-button-colors
  2. Merge into staging
Proposed by Renato Araujo Oliveira Filho
Status: Merged
Merged at revision: 295
Proposed branch: lp://qastaging/~renatofilho/address-book-app/fix-button-colors
Merge into: lp://qastaging/~phablet-team/address-book-app/staging
Diff against target: 47 lines (+4/-3)
3 files modified
src/imports/Common/RemoveContactsDialog.qml (+2/-1)
src/imports/ContactEdit/ContactDetailSyncTargetEditor.qml (+1/-1)
src/imports/ContactEdit/ContactEditor.qml (+1/-1)
To merge this branch: bzr merge lp://qastaging/~renatofilho/address-book-app/fix-button-colors
Reviewer Review Type Date Requested Status
PS Jenkins bot continuous-integration Approve
Ubuntu Phablet Team Pending
Review via email: mp+233254@code.qastaging.launchpad.net

Commit message

- Used pre-defined "red" color from UbuntuColors.
- Use green and red colors for buttons on delete contact dialog.
- Fixed "Add field" button size.

To post a comment you must log in.
lp://qastaging/~renatofilho/address-book-app/fix-button-colors updated
296. By Renato Araujo Oliveira Filho

Do not use "gradient" color for "no" button.

297. By Renato Araujo Oliveira Filho

Use the button default size for ComboButton and SyncTarget field.

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

PASSED: Continuous integration, rev:295
http://jenkins.qa.ubuntu.com/job/phablet-team-address-book-app-staging-ci/313/
Executed test runs:
    SUCCESS: http://jenkins.qa.ubuntu.com/job/generic-deb-autopilot-utopic-touch/4411
    SUCCESS: http://jenkins.qa.ubuntu.com/job/generic-mediumtests-utopic/3301
    SUCCESS: http://jenkins.qa.ubuntu.com/job/phablet-team-address-book-app-staging-utopic-i386-ci/313
    SUCCESS: http://jenkins.qa.ubuntu.com/job/generic-deb-autopilot-runner-mako/4190
    SUCCESS: http://jenkins.qa.ubuntu.com/job/generic-mediumtests-builder-utopic-armhf/5663
        deb: http://jenkins.qa.ubuntu.com/job/generic-mediumtests-builder-utopic-armhf/5663/artifact/work/output/*zip*/output.zip
    SUCCESS: http://s-jenkins.ubuntu-ci:8080/job/touch-flash-device/12581
    SUCCESS: http://jenkins.qa.ubuntu.com/job/autopilot-testrunner-otto-utopic/2709
    SUCCESS: http://jenkins.qa.ubuntu.com/job/generic-mediumtests-builder-utopic-amd64/3595
        deb: http://jenkins.qa.ubuntu.com/job/generic-mediumtests-builder-utopic-amd64/3595/artifact/work/output/*zip*/output.zip

Click here to trigger a rebuild:
http://s-jenkins.ubuntu-ci:8080/job/phablet-team-address-book-app-staging-ci/313/rebuild

review: Approve (continuous-integration)
Revision history for this message
PS Jenkins bot (ps-jenkins) wrote :

PASSED: Continuous integration, rev:297
http://jenkins.qa.ubuntu.com/job/phablet-team-address-book-app-staging-ci/314/
Executed test runs:
    SUCCESS: http://jenkins.qa.ubuntu.com/job/generic-deb-autopilot-utopic-touch/4413
    SUCCESS: http://jenkins.qa.ubuntu.com/job/generic-mediumtests-utopic/3303
    SUCCESS: http://jenkins.qa.ubuntu.com/job/phablet-team-address-book-app-staging-utopic-i386-ci/314
    SUCCESS: http://jenkins.qa.ubuntu.com/job/generic-deb-autopilot-runner-mako/4192
    SUCCESS: http://jenkins.qa.ubuntu.com/job/generic-mediumtests-builder-utopic-armhf/5665
        deb: http://jenkins.qa.ubuntu.com/job/generic-mediumtests-builder-utopic-armhf/5665/artifact/work/output/*zip*/output.zip
    SUCCESS: http://s-jenkins.ubuntu-ci:8080/job/touch-flash-device/12583
    SUCCESS: http://jenkins.qa.ubuntu.com/job/autopilot-testrunner-otto-utopic/2711
    SUCCESS: http://jenkins.qa.ubuntu.com/job/generic-mediumtests-builder-utopic-amd64/3597
        deb: http://jenkins.qa.ubuntu.com/job/generic-mediumtests-builder-utopic-amd64/3597/artifact/work/output/*zip*/output.zip

Click here to trigger a rebuild:
http://s-jenkins.ubuntu-ci:8080/job/phablet-team-address-book-app-staging-ci/314/rebuild

review: Approve (continuous-integration)

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