[Merge] lp:~mterry/dialer-app/no-ubuntu-colors into lp:dialer-app
Michael Terry
michael.terry at canonical.com
Tue Jun 14 22:25:45 UTC 2016
Michael Terry has proposed merging lp:~mterry/dialer-app/no-ubuntu-colors into lp:dialer-app.
Commit message:
Fix colors to use theme colors more consistently.
Requested reviews:
Ubuntu Phablet Team (phablet-team)
Related bugs:
Bug #1581047 in ubuntu-system-settings (Ubuntu): "UbuntuColors considered harmful"
https://bugs.launchpad.net/ubuntu/+source/ubuntu-system-settings/+bug/1581047
For more details, see:
https://code.launchpad.net/~mterry/dialer-app/no-ubuntu-colors/+merge/297412
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
--
Your team Ubuntu Phablet Team is requested to review the proposed merge of lp:~mterry/dialer-app/no-ubuntu-colors into lp:dialer-app.
-------------- next part --------------
A non-text attachment was scrubbed...
Name: review-diff.txt
Type: text/x-diff
Size: 14621 bytes
Desc: not available
URL: <https://lists.ubuntu.com/archives/ubuntu-reviews/attachments/20160614/b2e0c3fe/attachment.diff>
More information about the Ubuntu-reviews
mailing list