Merge lp://qastaging/~mterry/dialer-app/no-ubuntu-colors into lp://qastaging/dialer-app

Proposed by Michael Terry
Status: Needs review
Proposed branch: lp://qastaging/~mterry/dialer-app/no-ubuntu-colors
Merge into: lp://qastaging/dialer-app
Diff against target: 378 lines (+31/-38)
17 files modified
src/qml/DialerPage/CallButton.qml (+2/-2)
src/qml/DialerPage/CustomButton.qml (+1/-1)
src/qml/DialerPage/DialerPage.qml (+2/-2)
src/qml/DialerPage/KeypadButton.qml (+2/-2)
src/qml/DialerPage/KeypadEntry.qml (+3/-3)
src/qml/Dialogs/DisableFlightModeDialog.qml (+1/-2)
src/qml/Dialogs/NoDefaultSIMCardDialog.qml (+1/-3)
src/qml/Dialogs/NotificationDialog.qml (+0/-1)
src/qml/Dialogs/SetDefaultSIMCardDialog.qml (+1/-2)
src/qml/Dialogs/SimLockedDialog.qml (+3/-3)
src/qml/HistoryPage/HistoryDelegate.qml (+1/-1)
src/qml/HistoryPage/HistoryDetailsPage.qml (+2/-2)
src/qml/HistoryPage/SwipeItemDemo.qml (+2/-2)
src/qml/LiveCallPage/ConferenceCallDisplay.qml (+3/-3)
src/qml/LiveCallPage/HangupButton.qml (+2/-4)
src/qml/LiveCallPage/LiveCall.qml (+4/-4)
src/qml/LiveCallPage/MultiCallDisplay.qml (+1/-1)
To merge this branch: bzr merge lp://qastaging/~mterry/dialer-app/no-ubuntu-colors
Reviewer Review Type Date Requested Status
PS Jenkins bot continuous-integration Pending
Ubuntu Phablet Team Pending
Review via email: mp+297412@code.qastaging.launchpad.net

Commit message

Fix colors to use theme colors more consistently.

Description of the change

See bug 1581047 for full rationale.

Here is a list of potentially noticeable changes:

- Orange buttons are now green.
- Some less important buttons that were orange are now neutral grey: DisableFlightModeDialog cancel, NoDefaultSIMCardDialog no and later, NotificationDialog close, and SetDefaultSIMCardDialog no.
- Call button went from a dark green to theme green.
- Hangup button no longer changes to half-opacity orange when pressed. Instead it simply changes to half-opacity theme red. Didn't seem necessary to also change color on top of opacity when pressing.
- KeypadEntry cursor went from orange to blue, the new theme color for text cursors.
- Unrelated to color, but I was right there, so... SimLockedDialog's Ok button is now spelled OK. (OK and Okay are words, Ok is not.)

And some less noticeable changes:

- Simply switching hard-coded names to theme equivalents (like darkGrey to backgroundSecondaryColor; same color, new name)
- Small changes like off-brand "red" to theme red

To post a comment you must log in.
532. By Michael Terry

Don't change color of button sublabels

Unmerged revisions

532. By Michael Terry

Don't change color of button sublabels

531. By Michael Terry

Don't use UbuntuColors

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